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
Add option to decompress() for variable length #15618
Conversation
This was tested internally, I don't see any point it running testing again. @keithc-ca any concerns before this is merged? |
7a61aa8
to
a3dab8e
Compare
Updated PR to pass a length variable to |
a3dab8e
to
4f6f3f3
Compare
becf4fd
to
1c26f5e
Compare
@KeithCa updated the PR to address the review comments, can you take another look please? Internal testing done here. |
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.
I'll review StringBuilder
once I'm satisfied with the changes to StringBuffer
(I expect them to be similar).
cad3fc1
to
7e97b12
Compare
Updated PR to address comments. Also fixed some comments mixing up StringBuilder/StringBuffer and the trailing whitespace. |
e859bc7
to
dd65a80
Compare
dd65a80
to
f5c8513
Compare
f5c8513
to
06c644a
Compare
In StringBuilder and StringBuffer, decompress() usually only works up to the current data length. However, methods like move() can place characters past the current length, so add a parameter to decompress() in order to specify the number of characters that are decompressed. Fixes issues with insert() method when compact strings is enabled and the caller has compressed storage and the inserted object is not compressed. Also corrects some spelling of variable names and fixes some comments mixing up StringBuilder and StringBuffer. Signed-off-by: Mike Zhang <mike.h.zhang@ibm.com>
Signed-off-by: Mike Zhang <mike.h.zhang@ibm.com>
06c644a
to
4113c5a
Compare
jenkins test sanity zlinux jdk8 |
In StringBuilder and StringBuffer, decompress() usually only works up
to the current data length. However, methods like move() can place
characters past the current length, so add a parameter to decompress()
in order to specify the number of characters that are decompressed.
Fixes issues with insert() method when compact strings is enabled and
the caller has compressed storage and the inserted object is not
compressed.
fyi: @pshipton