From c88649ccd27425710966d98734d5cd7c1d9ed67d Mon Sep 17 00:00:00 2001 From: Greg Woolsey Date: Tue, 14 Feb 2017 16:49:24 +0000 Subject: [PATCH] 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(). git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1782989 13f79535-47bb-0310-9956-ffa450edef68 --- .../java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java index a09ae77b7..8b3570ac9 100644 --- a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java +++ b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFFontFormatting.java @@ -79,7 +79,7 @@ public class XSSFFontFormatting implements FontFormatting { */ @Override public boolean isStruckout() { - for (CTBooleanProperty bProp : _font.getStrikeArray()) if (bProp.getVal()) return true; + for (CTBooleanProperty bProp : _font.getStrikeArray()) return bProp.getVal(); return false; }