RFR [15] 8245111: Update doc comments for improved processing by the Standard Doclet

Roger Riggs Roger.Riggs at oracle.com
Fri May 15 15:53:11 UTC 2020


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
>>
>



More information about the core-libs-dev mailing list