On Tue, 10 Jun 2025 08:34:49 GMT, Johan Sjölen <jsjo...@openjdk.org> wrote:
>> Radim Vansa has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Revert removing FieldInfoReader::next_uint() > > src/hotspot/share/utilities/packedTable.hpp line 69: > >> 67: // by the supplier (when Supplier::next() returns false the whole >> array should >> 68: // be filled). >> 69: void fill(u1* table, size_t table_length, Supplier &supplier) const; > > Let the ampersand hug the type. Also seems like this can be a static method? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24847#discussion_r2137331314
