You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by jl...@apache.org on 2013/10/13 16:19:43 UTC

svn commit: r1531695 - /ant/easyant/core/trunk/src/main/java/org/apache/easyant/core/EasyAntMain.java

Author: jlboudart
Date: Sun Oct 13 14:19:43 2013
New Revision: 1531695

URL: http://svn.apache.org/r1531695
Log:
Since java 1.5 String.format can be used to lpad/rightpad a given string

Modified:
    ant/easyant/core/trunk/src/main/java/org/apache/easyant/core/EasyAntMain.java

Modified: ant/easyant/core/trunk/src/main/java/org/apache/easyant/core/EasyAntMain.java
URL: http://svn.apache.org/viewvc/ant/easyant/core/trunk/src/main/java/org/apache/easyant/core/EasyAntMain.java?rev=1531695&r1=1531694&r2=1531695&view=diff
==============================================================================
--- ant/easyant/core/trunk/src/main/java/org/apache/easyant/core/EasyAntMain.java (original)
+++ ant/easyant/core/trunk/src/main/java/org/apache/easyant/core/EasyAntMain.java Sun Oct 13 14:19:43 2013
@@ -510,14 +510,10 @@ public class EasyAntMain implements AntM
      */
     private static void printTargets(Project project, List<String> names, List<String> descriptions, String heading,
             int maxlen) {
-        if (names.size() > 0) {
+        if (!names.isEmpty()) {
             // now, start printing the targets and their descriptions
             String lSep = System.getProperty("line.separator");
-            // got a bit annoyed that I couldn't find a pad function
-            String spaces = "    ";
-            while (spaces.length() <= maxlen) {
-                spaces += spaces;
-            }
+            String spaces = String.format("%" + maxlen + "s", ' ');
             StringBuffer msg = new StringBuffer();
             msg.append(lSep).append(heading).append(lSep).append(lSep);
             for (int i = 0; i < names.size(); i++) {