RFR: 8302325: Wrong comment in java.base/share/native/libjimage/imageFile.hpp [v2]

Severin Gehwolf sgehwolf at openjdk.org
Tue Feb 14 19:30:21 UTC 2023


On Tue, 14 Feb 2023 18:10:11 GMT, Jim Laskey <jlaskey at openjdk.org> wrote:

>> Got it. How about we change this comment from:
>> 
>> //  - Even though ATTRIBUTE_END is used to mark the end of the attribute stream,
>> //      streams will contain zero byte values to represent lesser significant bits.
>> //      Thus, detecting a zero byte is not sufficient to detect the end of an attribute
>> //      stream.
>> 
>> 
>> to:
>> 
>> //  - Even though ATTRIBUTE_END (which might be encoded as a zero byte) is used to
>> //      mark the end of the attribute stream, streams will contain zero byte values in the
>> //      non-header portion of the attribute data. Thus, detecting a zero byte is not
>> //      sufficient to detect the end of an attribute stream.
>> 
>> 
>> ? The phrase "to represent lesser significant bits" and mention of `ATTRIBUTE_END` is throwing me off.
>
> Sounds good to me.

Thanks, updated.

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

PR: https://git.openjdk.org/jdk/pull/12533


More information about the core-libs-dev mailing list