You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by gw...@apache.org on 2017/02/14 16:49:24 UTC
svn commit: r1782989 -
/poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java
Author: gwoolsey
Date: Tue Feb 14 16:49:24 2017
New Revision: 1782989
URL: http://svn.apache.org/viewvc?rev=1782989&view=rev
Log:
Ugh, that's what I get for copying someone else's code without fully reviewing it. Return the boolean property, don't wrap it in an extra if().
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java?rev=1782989&r1=1782988&r2=1782989&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java Tue Feb 14 16:49:24 2017
@@ -79,7 +79,7 @@ public class XSSFFontFormatting implemen
*/
@Override
public boolean isStruckout() {
- for (CTBooleanProperty bProp : _font.getStrikeArray()) if (bProp.getVal()) return true;
+ for (CTBooleanProperty bProp : _font.getStrikeArray()) return bProp.getVal();
return false;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org