On 11/12/2018 03:24 PM, Claes Redestad wrote: > Webrev: http://cr.openjdk.java.net/~redestad/8213741/jdk.00/ > Bug: https://bugs.openjdk.java.net/browse/JDK-8213741
Looks good to me. *) Not sure this is significantly better than String.valueOf(String.valueOf(s)), as original comment suggests: 1701 String s; 1702 return (value == null || (s = value.toString()) == null) ? "null" : s; *) Do you really want new "static finals" be "private" as well? It seems to me the classes are private already, and maybe dropping some privates help to avoid some bridge methods? (Probably obsolete by nestmates, but still). It probably does not matter all that much for bootstrap code. -Aleksey
