Oops, missed that one in my search-and-replace. I've updated the online webrev.
Thanks! Mike On Apr 18 2011, at 16:13 , David Schlosnagle wrote: > On Mon, Apr 18, 2011 at 6:24 PM, Mike Duigou <[email protected]> wrote: >> The latest webrev : http://cr.openjdk.java.net/~mduigou/4884238/2/webrev/ >> >> Any other remaining feeback? >> > > Mike, > > You'll need to update the private constructor and AssertionError for > StandardCharset as the existing private Charset() won't compile: > > 35 public final class StandardCharset { > 36 > 37 private Charsets() { > 38 throw new AssertionError("No java.nio.charset.Charsets > instances for you!"); > 39 } > > - Dave
