On Mon, 26 Aug 2024 13:43:49 GMT, Weijun Wang <wei...@openjdk.org> wrote:

>> If I do that, the performance drops a little. So I rather keep it here.
>
> I see. In fact, what if `byteState` is an instance variable outside the 
> method? If a previous squeeze has left some bytes there, there is no need to 
> call `asLittleEndian.set` again next time they are needed. Does it help in 
> any performance gain? especially if the caller only squeeze a few bytes each 
> time?

Also, a comment will be nice here. Otherwise, some people might still want to 
move it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20631#discussion_r1731272923

Reply via email to