# HG changeset patch # User aleonard # Date 1553542400 0 # Mon Mar 25 19:33:20 2019 +0000 # Node ID e7d5fac46d42ddd4d9322ed5220401aee2c1a33c # Parent d25b24c70126cc55db57579f950d1f3e12b64b77 8221430: StringBuffer(CharSequence) constructor truncates when -XX:-CompactStrings specified diff --git a/src/java.base/share/classes/java/lang/AbstractStringBuilder.java b/src/java.base/share/classes/java/lang/AbstractStringBuilder.java --- a/src/java.base/share/classes/java/lang/AbstractStringBuilder.java +++ b/src/java.base/share/classes/java/lang/AbstractStringBuilder.java @@ -1741,4 +1741,21 @@ "start " + start + ", end " + end + ", length " + len); } } + + /** + * Determine the "coder" of the given CharSequence + */ + static byte getCharSequenceCoder(CharSequence seq) { + byte coder; + if (seq instanceof String) { + coder = ((String)seq).coder(); + } else if (seq instanceof AbstractStringBuilder) { + coder = ((AbstractStringBuilder)seq).getCoder(); + } else if (COMPACT_STRINGS) { + coder = LATIN1; + } else { + coder = UTF16; + } + return coder; + } } diff --git a/src/java.base/share/classes/java/lang/StringBuffer.java b/src/java.base/share/classes/java/lang/StringBuffer.java --- a/src/java.base/share/classes/java/lang/StringBuffer.java +++ b/src/java.base/share/classes/java/lang/StringBuffer.java @@ -162,7 +162,7 @@ * @since 1.5 */ public StringBuffer(CharSequence seq) { - super(String.LATIN1, seq.length(), 16); + super(getCharSequenceCoder(seq), seq.length(), 16); append(seq); } diff --git a/src/java.base/share/classes/java/lang/StringBuilder.java b/src/java.base/share/classes/java/lang/StringBuilder.java --- a/src/java.base/share/classes/java/lang/StringBuilder.java +++ b/src/java.base/share/classes/java/lang/StringBuilder.java @@ -134,7 +134,7 @@ * @param seq the sequence to copy. */ public StringBuilder(CharSequence seq) { - super(String.LATIN1, seq.length(), 16); + super(getCharSequenceCoder(seq), seq.length(), 16); append(seq); } diff --git a/test/jdk/java/lang/StringBuffer/CompactStringBuffer.java b/test/jdk/java/lang/StringBuffer/CompactStringBuffer.java --- a/test/jdk/java/lang/StringBuffer/CompactStringBuffer.java +++ b/test/jdk/java/lang/StringBuffer/CompactStringBuffer.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2015, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2015, 2019, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -440,6 +440,12 @@ "abcdefgh1.23456"); check(new StringBuffer().append(bmp).append(1.23456).toString(), "\u4e00\u4e01\u4e02\u4e03\u4e04\u4e05\u4e06\u4e07\u4e081.23456"); + + //////////////////////////////////////////////////////////////////// + check(new StringBuffer((CharSequence)new StringBuffer(ascii)).toString(), + ascii); + check(new StringBuffer((CharSequence)new StringBuffer(asciiMixed)).toString(), + asciiMixed); } private void checkGetChars(StringBuffer sb, int srcBegin, int srcEnd,