RFR: 8302325: Wrong comment in java.base/share/native/libjimage/imageFile.hpp
Alan Bateman
alanb at openjdk.org
Tue Feb 14 10:43:43 UTC 2023
On Mon, 13 Feb 2023 14:12:15 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
> Could I please get a review of this trivial comment-only change? `imageFile.hpp`
> describes some properties of the jimage file `lib/modules`. However, I don't think
> the comment example matches current code in the JDK. [`ATTRIBUTE_OFFSET`](https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/jdk/internal/jimage/ImageLocation.java#L44) get
> written to the image file with value `0x05` while the comment mentions it gets
> written as `0x04`. I propose to fix the comment so that it matches the code.
>
> In passing, I've also fixed a comment related to `ATTRIBUTE_END`. I think the point
> being made there is about byte values of `[0x01..0x07]`, all would represent
> `ATTRIBUTE_END`, as the upper `5` bits would be `0`. Therefore, byte `0x01` would equally
> represent `ATTRIBUTE_END` as would `0x00` and `0x07` or any value in between.
>
> Thoughts?
kind = 5 looks right.
Can you update the end date of the copyright headers as this is the first change in 2023. imageFile.cpp by missed by the other change so it could be done here too.
-------------
Marked as reviewed by alanb (Reviewer).
PR: https://git.openjdk.org/jdk/pull/12533
More information about the core-libs-dev
mailing list