Editing glitch.Was there but then it was gone. Thanks Brent. http://cr.openjdk.java.net/~jlaskey/8233116/webrev.04/index.html
> On Nov 15, 2019, at 7:47 PM, Brent Christian <brent.christ...@oracle.com> > wrote: > > Hi, Jim > > > src/java.base/share/classes/java/lang/String.java > > These changes look fine. > > -- > > test/jdk/java/lang/String/TranslateEscapes.java > > I'm missing where the new verifyLineTerminator() method gets called. Perhaps > that's meant to go here: > > 83 static void test4() { > 84 > 85 } > > ? > > -Brent > > On 11/12/19 7:15 AM, Jim Laskey wrote: >> updated >> webrev: http://cr.openjdk.java.net/~jlaskey/8233116/webrev.03/index.html >> <http://cr.openjdk.java.net/~jlaskey/8233116/webrev.03/index.html>