From: Greg Woolsey <gwoolsey@apache.org>
Date: Tue, 14 Feb 2017 16:49:24 +0000 (+0000)
Subject: Ugh, that's what I get for copying someone else's code without fully reviewing it... 
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=c88649ccd27425710966d98734d5cd7c1d9ed67d;p=poi.git

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
---

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 a09ae77b7b..8b3570ac9f 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; 
     }