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