AjcTestCase: be less verbose when appending to buffer repeatedly

Also put 'else (if)' and 'finally' on new lines rather than after the
preceding closing curly braces. This helps IDEs when using code folding
(show/hide content of code blocks enclosed in curly braces).

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>

diff --git a/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java b/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java
index f812f8f8598c99657f55bb3cd7e4b0136ef2f38d..e50ea975751e6743f6ea05e3f965e2e3f8915bc3 100644 (file)
--- a/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java
+++ b/org.aspectj.ajdt.core/src/test/java/org/aspectj/tools/ajc/AjcTestCase.java
@@ -256,29 +256,16 @@ public abstract class AjcTestCase extends TestCase {
                public String toString() {
                        StringBuilder buff = new StringBuilder();
                        buff.append("message ");
-                       if (sourceFileName != null) {
-                               buff.append("in file ");
-                               buff.append(sourceFileName);
-                               buff.append(" ");
-                       }
-                       if (line != -1) {
-                               buff.append("on line ");
-                               buff.append(line);
-                               buff.append(" ");
-                       }
-                       if (text != null) {
-                               buff.append("containing text '");
-                               buff.append(text);
-                               buff.append("' ");
-                       }
+                       if (sourceFileName != null)
+                               buff.append("in file ").append(sourceFileName).append(" ");
+                       if (line != -1)
+                               buff.append("on line ").append(line).append(" ");
+                       if (text != null)
+                               buff.append("containing text '").append(text).append("' ");
                        if (seeAlsos != null) {
                                buff.append("\n\twith see alsos:");
-                               for (ISourceLocation seeAlso : seeAlsos) {
-                                       buff.append("\t\t");
-                                       buff.append(seeAlso.getSourceFile().getPath());
-                                       buff.append(":");
-                                       buff.append(seeAlso.getLine());
-                               }
+                               for (ISourceLocation seeAlso : seeAlsos)
+                                       buff.append("\t\t").append(seeAlso.getSourceFile().getPath()).append(":").append(seeAlso.getLine());
                        }
                        return buff.toString();
                }
@@ -485,10 +472,8 @@ public abstract class AjcTestCase extends TestCase {
                        addExtra(failureReport, "weaveInfo", extraWeaves);
                        failureReport.append("\nCommand: 'ajc");
                        String[] args = result.getArgs();
-                       for (String arg : args) {
-                               failureReport.append(" ");
-                               failureReport.append(arg);
-                       }
+                       for (String arg : args)
+                               failureReport.append(" ").append(arg);
                        String report = failureReport.toString();
                        System.err.println(failureReport);
                        fail(assertionFailedMessage + "'\n" + report);
@@ -603,8 +588,7 @@ public abstract class AjcTestCase extends TestCase {
                StringBuilder cp = new StringBuilder();
                if (classpath != null) {
                        // allow replacing this special variable, rather than copying all files to allow tests of jars that don't end in .jar
-                       cp.append(substituteSandbox(classpath));
-                       cp.append(pathSeparator);
+                       cp.append(substituteSandbox(classpath)).append(pathSeparator);
                }
                if (moduleName == null) {
                        // When running modules, we want more control so don't try to be helpful by adding all jars
@@ -612,10 +596,8 @@ public abstract class AjcTestCase extends TestCase {
                        getAnyJars(ajc.getSandboxDirectory(), cp);
                }
                StringBuilder mp = new StringBuilder();
-               if (modulepath != null) {
-                       mp.append(substituteSandbox(modulepath));
-                       mp.append(pathSeparator);
-               }
+               if (modulepath != null)
+                       mp.append(substituteSandbox(modulepath)).append(pathSeparator);
 
                URLClassLoader sandboxLoader;
                ClassLoader parentLoader = getClass().getClassLoader().getParent();
@@ -637,7 +619,8 @@ public abstract class AjcTestCase extends TestCase {
                        URL[] sandboxUrls = getURLs(cp.toString());
                        sandboxLoader = createWeavingClassLoader(sandboxUrls, aspectjLoader);
                        // sandboxLoader = createWeavingClassLoader(sandboxUrls,testLoader);
-               } else if(useFullLTW  && useLTW) {
+               }
+               else if(useFullLTW  && useLTW) {
                        if(vmargs == null){
                                vmargs ="";
                        }
@@ -669,7 +652,8 @@ public abstract class AjcTestCase extends TestCase {
                                e.printStackTrace();
                        }
                        return lastRunResult;
-               } else if (moduleName != null) {
+               }
+               else if (moduleName != null) {
                        // CODE FOR RUNNING MODULES
                        if(vmargs == null){
                                vmargs ="";
@@ -681,12 +665,11 @@ public abstract class AjcTestCase extends TestCase {
                                if (mp.indexOf("$runtime") != -1) {
                                        mp = mp.replace(mp.indexOf("$runtime"),"$runtime".length(),TestUtil.aspectjrtPath().toString());
                                }
-                               if (cp.indexOf("aspectjrt")==-1) {
+                               if (cp.indexOf("aspectjrt") == -1)
                                        cp.append(TestUtil.aspectjrtPath().getPath()).append(pathSeparator);
-                               }
-                               String command = LangUtil.getJavaExecutable().getAbsolutePath() + " " +vmargs+ (cp.length()==0?"":" -classpath " + cp) + " -p "+mp+" --module "+moduleName   ;
+                               String command = LangUtil.getJavaExecutable().getAbsolutePath() + " " + vmargs + (cp.length() == 0 ? "" : " -classpath " + cp) + " -p " + mp + " --module " + moduleName;
                                if (Ajc.verbose)
-                                                       System.out.println("\nCommand: '" + command + "'\n");
+                                       System.out.println("\nCommand: '" + command + "'\n");
                                // Command is executed using ProcessBuilder to allow setting CWD for ajc sandbox compliance
                                ProcessBuilder pb = new ProcessBuilder(tokenizeCommand(command));
                                pb.directory( new File(ajc.getSandboxDirectory().getAbsolutePath()));
@@ -700,7 +683,8 @@ public abstract class AjcTestCase extends TestCase {
                                e.printStackTrace();
                        }
                        return lastRunResult;
-               } else if (
+               }
+               else if (
                        vmargs != null && (
                                vmargs.contains("--enable-preview") ||
                                vmargs.contains("--add-modules") ||
@@ -719,7 +703,7 @@ public abstract class AjcTestCase extends TestCase {
                                }
                                String command = LangUtil.getJavaExecutable().getAbsolutePath() + " " +vmargs+ (cp.length()==0?"":" -classpath " + cp) + " " + className   ;
                                if (Ajc.verbose)
-                                                       System.out.println("\nCommand: '" + command + "'\n");
+                                       System.out.println("\nCommand: '" + command + "'\n");
                                // Command is executed using ProcessBuilder to allow setting CWD for ajc sandbox compliance
                                ProcessBuilder pb = new ProcessBuilder(tokenizeCommand(command));
                                pb.directory( new File(ajc.getSandboxDirectory().getAbsolutePath()));
@@ -733,7 +717,8 @@ public abstract class AjcTestCase extends TestCase {
                                e.printStackTrace();
                        }
                        return lastRunResult;
-               } else {
+               }
+               else {
                        cp.append(DEFAULT_CLASSPATH_ENTRIES);
                        URL[] urls = getURLs(cp.toString());
                        sandboxLoader = new URLClassLoader(urls, parentLoader);
@@ -741,18 +726,13 @@ public abstract class AjcTestCase extends TestCase {
                ByteArrayOutputStream baosOut = new ByteArrayOutputStream();
                ByteArrayOutputStream baosErr = new ByteArrayOutputStream();
 
-
                StringBuilder command = new StringBuilder();
-               command.append("java -classpath ");
-               command.append(cp.toString());
-               command.append(" ");
-               command.append(className);
-               for (String arg : args) {
-                       command.append(" ");
-                       command.append(arg);
-               }
+               command.append("java -classpath ").append(cp).append(" ").append(className);
+               for (String arg : args)
+                       command.append(" ").append(arg);
                if (Ajc.verbose)
-                               System.out.println("\nCommand: '" + command + "'\n");
+                       System.out.println("\nCommand: '" + command + "'\n");
+
                //              try {
                //                      // Enable the security manager
                //                      Policy.setPolicy(new MyPolicy());
@@ -1025,27 +1005,17 @@ public abstract class AjcTestCase extends TestCase {
 
        private void addMissing(StringBuilder buff, String type, List<AjcTestCase.Message> messages) {
                if (!messages.isEmpty()) {
-                       buff.append("Missing expected ");
-                       buff.append(type);
-                       buff.append(" messages:\n");
-                       for (Message message : messages) {
-                               buff.append("\t");
-                               buff.append(message.toString());
-                               buff.append("\n");
-                       }
+                       buff.append("Missing expected ").append(type).append(" messages:\n");
+                       for (Message message : messages)
+                               buff.append("\t").append(message.toString()).append("\n");
                }
        }
 
        private void addExtra(StringBuilder buff, String type, List messages) {
                if (!messages.isEmpty()) {
-                       buff.append("Unexpected ");
-                       buff.append(type);
-                       buff.append(" messages:\n");
-                       for (Object message : messages) {
-                               buff.append("\t");
-                               buff.append(message.toString());
-                               buff.append("\n");
-                       }
+                       buff.append("Unexpected ").append(type).append(" messages:\n");
+                       for (Object message : messages)
+                               buff.append("\t").append(message.toString()).append("\n");
                }
        }
 
@@ -1053,12 +1023,10 @@ public abstract class AjcTestCase extends TestCase {
        private void getAnyJars(File dir, StringBuilder buff) {
                File[] files = dir.listFiles();
                for (File file : files) {
-                       if (file.getName().endsWith(".jar")) {
-                               buff.append(pathSeparator);
-                               buff.append(file.getAbsolutePath());
-                       } else if (file.isDirectory()) {
+                       if (file.getName().endsWith(".jar"))
+                               buff.append(pathSeparator).append(file.getAbsolutePath());
+                       else if (file.isDirectory())
                                getAnyJars(file, buff);
-                       }
                }
        }