A few char-fiddling optimizations for Pattern.java

[Ulf Zibis]

>
> Hi Xueming and Ulf,
>
> Please review a few more optimizations to be added to the
> current crop:
>
> http://cr.openjdk.java.net/~martin/webrevs/openjdk7/Pattern-opt/
>
>    

Have you tried to disassemble? :

     private static final boolean isSupplementary(int cp) {
          returnCharacter.isSurrogate((char)cp)  ||
//  in this case method should be namedisSupplementaryOrHigher():
//                cp>= Character.MIN_SUPPLEMENTARY_CODE_POINT);
                  Character.isSupplementaryCodePoint(cp);

     }

Maybe HotSpot would compile to same code.
Anyway I think, such functionality should belong to class Character.
And don't forget the 8-space indentation rule for line continuation. ;-)


Use Character.codePointCount(...) instead ;-)  :

countCodePoints(CharSequence seq)


-Ulf