On Sat, 25 Mar 2023 11:23:01 GMT, Lance Andersen <lan...@openjdk.org> wrote:

>> I hear you but the option is not to use javadoc comments and use block 
>> comments :-) 
>> 
>> I think we can agree to disagree on impacting readability ;-) this comes 
>> down to personal preferences
>> 
>> Again thank you for making the change.
>
>> I think the PR is ready to be sponsored after this.
> 
> Please see my comment regarding the  end of central directory record.
> 
> I would prefer to tweak that in a fashion similar to what I indicated as I 
> thought your original version was clearer.  I understand what Martin was 
> indicating of the use of EOC and that could have been addressed by adding  
> _(EOC)_ or _ENDHDR_ after _end of central directory record_ to make it clearer
> 
> I will leave it up to you as to whether you want to make the change but it 
> would be clearer as I think we took a small step backwards.
> 
> Either way you should not need a sponsor and should be good to integrated 
> when you are ready

> I hear you but the option is not to use javadoc comments and use block 
> comments :-)

I learned something new today: Javadoc comments and block comments are not the 
same!

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/12231#discussion_r1148357449

Reply via email to