RFR: 8292758: put support for UNSIGNED5 format into its own header file [v5]

Coleen Phillimore coleenp at openjdk.org
Wed Sep 7 17:46:59 UTC 2022


On Wed, 7 Sep 2022 01:05:38 GMT, John R Rose <jrose at openjdk.org> wrote:

>> Refactor code from inside of CompressedStream into its own unit.
>> 
>> This code is likely to be used in future refactorings, such as JDK-8292818 (replace 96-bit representation for field metadata with variable-sized streams).
>> 
>> Add gtests.
>
> John R Rose has updated the pull request incrementally with one additional commit since the last revision:
> 
>   add missing "this->"

This looks good to me, for the current CompressedStream and for upcoming changes.

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

Marked as reviewed by coleenp (Reviewer).

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


More information about the hotspot-dev mailing list