On Wed, 11 May 2022 18:42:46 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> In relation to 
>> [JDK-8244681](https://bugs.openjdk.java.net/browse/JDK-8244681), please find 
>> here a patch that addresses possibly lossy conversions in java.net.http
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert adding char constants

src/java.net.http/share/classes/jdk/internal/net/http/websocket/Frame.java line 
291:

> 289: 
> 290:         HeaderWriter noMask() {
> 291:             // The negation "~" sets the high order bits

Rubber-stamping this in front of every of the four closely sitting casts seems 
excessive:

                // The negation "~" sets the high order bits
                // so the value is more than 16 bits and the
                // compiler will emit a warning if not cast

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

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

Reply via email to