try to remove goto

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1893596 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java b/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java
index 1119c6c211579a0c71cb58715f6a56a548a7ce06..44f1c6c68d880fdc10615c4f8a255e8fa8ba86b0 100644 (file)
--- a/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java
+++ b/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java
@@ -40,8 +40,6 @@ public abstract class HeaderFooter implements org.apache.poi.ss.usermodel.Header
         String _center = "";
         String _right = "";
 
-// FIXME: replace outer goto. just eww.
-outer:
         while (text.length() > 1) {
             if (text.charAt(0) != '&') {
                 // Mimics the behaviour of Excel, which would put it in the center.
@@ -59,7 +57,7 @@ outer:
                 }
                 _left = text.substring(2, pos);
                 text = text.substring(pos);
-                break;
+                return new String[] { _left, _center, _right, };
             case 'C':
                 if (text.contains("&L")) {
                     pos = Math.min(pos, text.indexOf("&L"));
@@ -69,7 +67,7 @@ outer:
                 }
                 _center = text.substring(2, pos);
                 text = text.substring(pos);
-                break;
+                return new String[] { _left, _center, _right, };
             case 'R':
                 if (text.contains("&C")) {
                     pos = Math.min(pos, text.indexOf("&C"));
@@ -79,14 +77,14 @@ outer:
                 }
                 _right = text.substring(2, pos);
                 text = text.substring(pos);
-                break;
+                return new String[] { _left, _center, _right, };
             default:
                 // Mimics the behaviour of Excel, which would put it in the center.
                 _center = text;
-                break outer;
+                break;
             }
         }
-        return new String[] { _left, _center, _right, };
+        return null;
     }
 
     /**