Request for Review : CR#6924259: Remove String.count/String.offset

Ulf Zibis Ulf.Zibis at gmx.de
Thu May 24 15:56:05 PDT 2012


Am 24.05.2012 22:45, schrieb Mike Duigou:
> A patch of the changes to java.lang.String for JDK 8 are at<http://cr.openjdk.java.net/~mduigou/6924259/0/webrev/>. The changes for JDK 7 have only superficial differences (line offsets in the patch).
Correct Copyright date.

  157     public String(String original) {
  158         this.value = (original.value.length>  0)
  159                 ? original.value
  160                 : EMPTY_STRING_VALUE;  145
  161     }

Does the last case ever occur?, if you have:

  143     public String() {
  144         this.value = EMPTY_STRING_VALUE;
  145     }


-Ulf




More information about the jdk7u-dev mailing list