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.
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
8300236: Use VarHandle access in Data(Input | Output)Stream classes #12076
Closed
Closed
Changes from 14 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
9250ab4
Add a supported API similar to java.io.Bits
minborg 9199268
Add test and fix imports
minborg a4c767e
Rework BigEndianAtZeroBuffer
minborg 1b719c5
Recompose classes and methods
minborg af301d5
Rename methods and parameters
minborg 3e02361
Fix typo
minborg 3e0d367
Prepare for review
minborg 0ee9bb6
Reformat javadoc
minborg a8cd6b4
Update copyright years
minborg 432a6b0
Fix jcheck issues
minborg 9e158a0
Remove white space
minborg ce1d576
Remove white spaces
minborg f928081
Improve doc and fix typo
minborg badf281
Rename and move
minborg 3a3dc2e
Add benchmark
minborg a13b360
Remove methods with implicit zero offset
minborg 469cac0
Remove redundant @throws documentation
minborg f331d2e
Merge master and fix RAF issues
minborg 4bf1a50
Update RAF
minborg File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No objection to the readBuffer but it does make it wonder if we should re-visit bytearr and chararr as they are only need for reading modified UTF-8 strings and shouldn't need to be eagerly created.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://bugs.openjdk.org/browse/JDK-8300979