Code Review for open

Prepared by:igerasim on Thu Mar 28 00:00:09 PDT 2019
Workspace:/home/igerasim/JDK/jdk-dev/open
Compare against: ssh://hg.openjdk.java.net/jdk/jdk
Compare against version:54317
Summary of changes: 72 lines changed: 53 ins; 4 del; 15 mod; 3729 unchg
Changeset: open.changeset
Bug id: JDK-8221430 : StringBuffer(CharSequence) constructor truncates when -XX:-CompactStrings specified
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/lang/AbstractStringBuilder.java

rev 54318 : 8221430: StringBuffer(CharSequence) constructor truncates when -XX:-CompactStrings specified
Reviewed-by: igerasim, rriggs
Contributed-by: Andrew Leonard <andrew_m_leonard@uk.ibm.com>
27 lines changed: 19 ins; 0 del; 8 mod; 1736 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/lang/StringBuffer.java

rev 54318 : 8221430: StringBuffer(CharSequence) constructor truncates when -XX:-CompactStrings specified
Reviewed-by: igerasim, rriggs
Contributed-by: Andrew Leonard <andrew_m_leonard@uk.ibm.com>
4 lines changed: 0 ins; 2 del; 2 mod; 765 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/java.base/share/classes/java/lang/StringBuilder.java

rev 54318 : 8221430: StringBuffer(CharSequence) constructor truncates when -XX:-CompactStrings specified
Reviewed-by: igerasim, rriggs
Contributed-by: Andrew Leonard <andrew_m_leonard@uk.ibm.com>
4 lines changed: 0 ins; 2 del; 2 mod; 484 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/StringBuffer/CompactStringBuffer.java

rev 54318 : 8221430: StringBuffer(CharSequence) constructor truncates when -XX:-CompactStrings specified
Reviewed-by: igerasim, rriggs
Contributed-by: Andrew Leonard <andrew_m_leonard@uk.ibm.com>
8 lines changed: 6 ins; 0 del; 2 mod; 486 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/micro/org/openjdk/bench/java/lang/StringBuilders.java

rev 54318 : 8221430: StringBuffer(CharSequence) constructor truncates when -XX:-CompactStrings specified
Reviewed-by: igerasim, rriggs
Contributed-by: Andrew Leonard <andrew_m_leonard@uk.ibm.com>
29 lines changed: 28 ins; 0 del; 1 mod; 258 unchg

This code review page was prepared using /home/igerasim/bin/webrev (vers 25.16-hg+openjdk.java.net).