RFR: 8351339: WebSocket::sendBinary assume that user supplied buffers are BIG_ENDIAN [v4]
Volkan Yazici
vyazici at openjdk.org
Fri Mar 21 11:41:45 UTC 2025
On Fri, 21 Mar 2025 11:23:51 GMT, Michael McMahon <michaelm at openjdk.org> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Apply review feedback on naming in `Masker`
>>
>> `reset` -> `setMask`
>> `initGallopingMask` -> `initVectorMask`
>> `applyGallopingMask` -> `applyVectorMask`
>
> src/java.net.http/share/classes/jdk/internal/net/http/websocket/Frame.java line 145:
>
>> 143:
>> 144: /**
>> 145: * Positions the {@link #offset} at 0, which is needed for vectorized masking, by masking necessary amount of bytes.
>
> Suggestion:
>
> * Applies up to 3 remaining from the previous pass bytes of the mask and
> * positions the {@link #offset} at 0, which is needed for vectorized masking,
> * by masking necessary amount of bytes.
@Michael-Mc-Mahon, I'm fine with the change, though doesn't this obsolete the `by masking necessary amount of bytes` statement at the end?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24033#discussion_r2007400397
More information about the net-dev
mailing list