Fix initCap function to set outbound position correctly.

Project: http://git-wip-us.apache.org/repos/asf/incubator-drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-drill/commit/7fc8d6d4
Tree: http://git-wip-us.apache.org/repos/asf/incubator-drill/tree/7fc8d6d4
Diff: http://git-wip-us.apache.org/repos/asf/incubator-drill/diff/7fc8d6d4

Branch: refs/heads/master
Commit: 7fc8d6d4dfee9fd37327efa205145e75eaf2cde1
Parents: 9187035
Author: Jacques Nadeau <[email protected]>
Authored: Mon Aug 25 22:30:37 2014 -0700
Committer: Jacques Nadeau <[email protected]>
Committed: Mon Aug 25 22:32:16 2014 -0700

----------------------------------------------------------------------
 .../apache/drill/exec/expr/fn/impl/StringFunctionHelpers.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-drill/blob/7fc8d6d4/exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/StringFunctionHelpers.java
----------------------------------------------------------------------
diff --git 
a/exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/StringFunctionHelpers.java
 
b/exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/StringFunctionHelpers.java
index 68f7ca4..d429a28 100644
--- 
a/exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/StringFunctionHelpers.java
+++ 
b/exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/StringFunctionHelpers.java
@@ -32,8 +32,8 @@ public class StringFunctionHelpers {
   // white space is treated as everything else.
   public static void initCap(int start, int end, DrillBuf inBuf, DrillBuf 
outBuf) {
     boolean capNext = true;
-
-    for (int id = start; id < end; id++) {
+    int out = 0;
+    for (int id = start; id < end; id++; out++) {
       byte currentByte = inBuf.getByte(id);
 
       // 'A - Z' : 0x41 - 0x5A
@@ -61,7 +61,7 @@ public class StringFunctionHelpers {
         }
       }
 
-      outBuf.setByte(id, currentByte);
+      outBuf.setByte(out, currentByte);
     } // end of for_loop
   }
 

Reply via email to