Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8286386: Address possibly lossy conversions in java.net.http #8656

Closed
wants to merge 10 commits into from
+27 −13
Closed

8286386: Address possibly lossy conversions in java.net.http #8656

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
6cba6d7
8286386: Address possibly lossy conversions in java.net.http
dfuch May 11, 2022
6ef906e
Fix comments
dfuch May 11, 2022
bbcf238
Use char constant to avoid casts
dfuch May 11, 2022
ec344ee
Add _MASK suffix to char constants
dfuch May 11, 2022
fbf3c9a
Revert adding char constants
dfuch May 11, 2022
2334b74
Revert adding char constants
dfuch May 11, 2022
deb2299
lengthOfCode() returns int; Removed excessive comments
dfuch May 12, 2022
25c24d6
Pavel's feedback
dfuch May 12, 2022
1f0902e
Move assert
dfuch May 12, 2022
258a589
Typo
dfuch May 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions src/java.net.http/share/classes/jdk/internal/net/http/websocket/Frame.java
View file
Original file line number Diff line number Diff line change
@@ -39,8 +39,6 @@ private Frame() { }

static final int MAX_HEADER_SIZE_BYTES = 2 + 8 + 4;
static final int MAX_CONTROL_FRAME_PAYLOAD_LENGTH = 125;
static final char FIN_BIT = 0b10000000_00000000;
static final char RSV1_BIT = 0b01000000_00000000;

enum Opcode {