On Fri, 8 Sep 2023 23:02:02 GMT, Justin Lu <j...@openjdk.org> wrote:

> Please review this change which adjusts the pattern syntax specification for 
> the two classes to represent the actual behavior. That is, U+FFFE and U+FFFF 
> are allowed in the suffix/prefix. (Additionally; 'Unicode' is dropped from 
> the definitions, as a Java character is composed of Unicode code points).
> 
> See code below, no exception is thrown.
> 
> 
> String uFFFE = "\uFFFE";
> String uFFFF = "\uFFFF";
> var a = new DecimalFormat("prefixStart"+uFFFE+"0.00"+uFFFF+"SuffixEnd");
> a.format(1); // returns "prefixStart�1.00�SuffixEnd"
> var b = new CompactNumberFormat(a.toPattern(), a.getDecimalFormatSymbols(), 
> new String[] {""});
> b.format(1); // returns "prefixStart�1�SuffixEnd"

This pull request has now been integrated.

Changeset: dde11551
Author:    Justin Lu <j...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/dde11551e26dedd28168d2d4528e9dd66ed82999
Stats:     8 lines in 2 files changed: 2 ins; 0 del; 6 mod

8315946: DecimalFormat and CompactNumberFormat do allow U+FFFE and U+FFFF in 
the pattern

Reviewed-by: naoto

-------------

PR: https://git.openjdk.org/jdk/pull/15648

Reply via email to