You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ki...@apache.org on 2016/11/01 10:52:20 UTC

svn commit: r1767474 - /poi/trunk/src/java/org/apache/poi/ss/formula/functions/Fixed.java

Author: kiwiwings
Date: Tue Nov  1 10:52:20 2016
New Revision: 1767474

URL: http://svn.apache.org/viewvc?rev=1767474&view=rev
Log:
Handle null returns of coerceValueToBoolean

Modified:
    poi/trunk/src/java/org/apache/poi/ss/formula/functions/Fixed.java

Modified: poi/trunk/src/java/org/apache/poi/ss/formula/functions/Fixed.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/ss/formula/functions/Fixed.java?rev=1767474&r1=1767473&r2=1767474&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/ss/formula/functions/Fixed.java (original)
+++ poi/trunk/src/java/org/apache/poi/ss/formula/functions/Fixed.java Tue Nov  1 10:52:20 2016
@@ -92,7 +92,7 @@ public final class Fixed implements Func
             // Format number conditionally using a thousands separator.
             NumberFormat nf = NumberFormat.getNumberInstance(Locale.US);
             DecimalFormat formatter = (DecimalFormat)nf;
-            formatter.setGroupingUsed(! skipThousandsSeparator);
+            formatter.setGroupingUsed(!(skipThousandsSeparator != null && skipThousandsSeparator));
             formatter.setMinimumFractionDigits(places >= 0 ? places : 0);
             formatter.setMaximumFractionDigits(places >= 0 ? places : 0);
             String numberString = formatter.format(number.doubleValue());



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org