Fix checkstyle warning wrt inner assignments, added unit test.

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/trunk@1439998 13f79535-47bb-0310-9956-ffa450edef68

Author: netomi

src/main/java/org/apache/commons/cli/HelpFormatter.java

@@ -981,29 +981,32 @@ private Appendable renderWrappedTextBlock(StringBuffer sb, int width, int nextLi
      */
     protected int findWrapPos(String text, int width, int startPos)
     {
-        int pos;
-
         // the line ends before the max wrap pos or a new line char found
-        if (((pos = text.indexOf('\n', startPos)) != -1 && pos <= width)
-                || ((pos = text.indexOf('\t', startPos)) != -1 && pos <= width))
+        int pos = text.indexOf('\n', startPos);
+        if (pos != -1 && pos <= width)
         {
             return pos + 1;
         }
-        else if (startPos + width >= text.length())
+
+        pos = text.indexOf('\t', startPos);
+        if (pos != -1 && pos <= width)
         {
-            return -1;
+            return pos + 1;
         }
 
+        if (startPos + width >= text.length())
+        {
+            return -1;
+        }
 
         // look for the last whitespace character before startPos+width
-        pos = startPos + width;
-
-        char c;
-
-        while ((pos >= startPos) && ((c = text.charAt(pos)) != ' ')
-                && (c != '\n') && (c != '\r'))
+        for (pos = startPos + width; pos >= startPos; --pos)
         {
-            --pos;
+            final char c = text.charAt(pos);
+            if (c == ' ' || c == '\n' || c == '\r')
+            {
+                break;
+            }
         }
 
         // if we found it - just return

src/test/java/org/apache/commons/cli/HelpFormatterTest.java

@@ -54,6 +54,12 @@ public void testFindWrapPos() throws Exception
         text = "aaaaaa aaaaaa";
         assertEquals("wrap position 4", 6, hf.findWrapPos(text, 6, 0));
         assertEquals("wrap position 4", -1, hf.findWrapPos(text, 6, 7));
+        
+        text = "aaaaaa\n aaaaaa";
+        assertEquals("wrap position 5", 7, hf.findWrapPos(text, 6, 0));
+        
+        text = "aaaaaa\t aaaaaa";
+        assertEquals("wrap position 6", 7, hf.findWrapPos(text, 6, 0));
     }
 
     public void testRenderWrappedTextWordCut()