Skip to content

Commit

Permalink
Out of memory error when creating a chart #1727
Browse files Browse the repository at this point in the history
Sometimes NumberFormat is used to handle the decimal point
and in other cases there is a dependency on ULocale.

A small fix was made to use the default ULocale in the
Numberformat calculations instead of the default NumberFormat

The whole operation is not fully understood so more issues
could arise.

See the issue for a discussion.
  • Loading branch information
wimjongman committed Jun 13, 2024
1 parent 7eb0e6a commit 9643ee7
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ public static String getExpressionString(String colName) {
* @return the default number format
*/
public static NumberFormat getDefaultNumberFormatInstance() {
NumberFormat numberFormat = NumberFormat.getInstance();
NumberFormat numberFormat = NumberFormat.getInstance(ULocale.getDefault());
// fix icu limitation which only allow 3 fraction digits as maximum by
// default. ?100 is enough.
numberFormat.setMaximumFractionDigits(100);
Expand Down

0 comments on commit 9643ee7

Please sign in to comment.