Hi Pavel,
No problem with the "with" -> "by" changes.
javax/naming/NameNotFoundException.java: 55
"initialized" -> "are initialized"
java/util/jar/Attributes.java:594
The period was in the correct place.
The second sentence is a separate comment about the use (non-use).
"used for the extension mechanism that is no longer supported."
Would read better as:
"This name is obsolete, the extension mechanism is no longer supported."
$.02, Roger
On 5/15/20 10:00 AM, Daniel Fuchs wrote:
Hi Pavel,
This looks good to me - but English is not my native language ;-)
cheers,
-- daniel
On 15/05/2020 13:35, Pavel Rappo wrote:
Hello,
Please review this trivial change for
https://bugs.openjdk.java.net/browse/JDK-8245111:
http://cr.openjdk.java.net/~prappo/8245111/webrev.00/
In addition to fixing the main issue, this includes a blanket change
from "followed with" to "followed by" as the latter seemed idiomatic
to me.
-Pavel
