Fix string bitwise op semantics.

jnthn · jnthn · commit f4a840698b6a · 2013-06-20T15:15:41.000+02:00
Strings of different lengths want The Other Thing.

With this, all Rakudo sanity tests pass on JVM.
diff --git a/src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java b/src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java
@@ -2851,9 +2851,9 @@ public static String bitor_s(String a, String b) {
         StringBuilder r = new StringBuilder(mlength);
         int apos = 0;
         int bpos = 0;
-        while (apos < alength || bpos < blength) {
-           int cpa = apos < alength ? a.codePointAt(apos) : 0;
-           int cpb = bpos < blength ? b.codePointAt(bpos) : 0;
+        while (apos < alength && bpos < blength) {
+           int cpa = a.codePointAt(apos);
+           int cpb = b.codePointAt(bpos);
            r.appendCodePoint(cpa | cpb);
            apos += Character.charCount(cpa);
            bpos += Character.charCount(cpb);
@@ -2868,9 +2868,9 @@ public static String bitxor_s(String a, String b) {
         StringBuilder r = new StringBuilder(mlength);
         int apos = 0;
         int bpos = 0;
-        while (apos < alength || bpos < blength) {
-           int cpa = apos < alength ? a.codePointAt(apos) : 0;
-           int cpb = bpos < blength ? b.codePointAt(bpos) : 0;
+        while (apos < alength && bpos < blength) {
+           int cpa = a.codePointAt(apos);
+           int cpb = b.codePointAt(bpos);
            r.appendCodePoint(cpa ^ cpb);
            apos += Character.charCount(cpa);
            bpos += Character.charCount(cpb);
@@ -2885,9 +2885,9 @@ public static String bitand_s(String a, String b) {
         StringBuilder r = new StringBuilder(mlength);
         int apos = 0;
         int bpos = 0;
-        while (apos < alength || bpos < blength) {
-           int cpa = apos < alength ? a.codePointAt(apos) : 0;
-           int cpb = bpos < blength ? b.codePointAt(bpos) : 0;
+        while (apos < alength && bpos < blength) {
+           int cpa = a.codePointAt(apos);
+           int cpb = b.codePointAt(bpos);
            r.appendCodePoint(cpa & cpb);
            apos += Character.charCount(cpa);
            bpos += Character.charCount(cpb);
