remove nocommits, this isnt really evil

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/branches/preflexfixes@967137 13f79535-47bb-0310-9956-ffa450edef68

lucene/src/java/org/apache/lucene/util/BytesRef.java

@@ -331,13 +331,16 @@ public final class BytesRef implements Comparable<BytesRef>, Externalizable {
           // We know the terms are not equal, but, we may
           // have to carefully fixup the bytes at the
           // difference to match UTF16's sort order:
+          
+          // NOTE: instead of moving supplementary code points (0xee and 0xef) to the unused 0xfe and 0xff, 
+          // we move them to the unused 0xfc and 0xfd [reserved for future 6-byte character sequences]
+          // this reserves 0xff for preflex's term reordering (surrogate dance), and if unicode grows such
+          // that 6-byte sequences are needed we have much bigger problems anyway.
           if (aByte >= 0xee && bByte >= 0xee) {
             if ((aByte & 0xfe) == 0xee) {
-              // nocommit
               aByte += 0xe;
             }
             if ((bByte&0xfe) == 0xee) {
-              // nocommit
               bByte += 0xe;
             }
           }