RFR: 8286386: Address possibly lossy conversions in java.net.http [v9]

[Pavel Rappo]

On Thu, 12 May 2022 14:15:43 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>>> OK - I will change codeLengthOf as suggested. It was not immediately obvious to me that the values would fit in the first 31 bits.
>> 
>> In fact, it would even fit into the first 30 bits. There's a top-level comment that explains the layout of `code` elements. Maybe you can improve it somehow or carry over that specific part about the length into `codeLengthOf`. Alternatively, you can always slap `assert` in `codeLengthOf`.
>
> Did I understand correctly that the lower 32 bits is expected to contain a length coded on 5 bits - which is expected to be a value in (5..30)?

Yes, you did understand that correctly.

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

PR: https://git.openjdk.java.net/jdk/pull/8656