You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2015/08/08 20:41:30 UTC

svn commit: r1694837 - in /commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon: DaemonPermission.java support/DaemonConfiguration.java support/DaemonLoader.java support/DaemonWrapper.java

Author: sebb
Date: Sat Aug  8 18:41:30 2015
New Revision: 1694837

URL: http://svn.apache.org/r1694837
Log:
Always use blocks for conditionals

Modified:
    commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/DaemonPermission.java
    commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonConfiguration.java
    commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonLoader.java
    commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonWrapper.java

Modified: commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/DaemonPermission.java
URL: http://svn.apache.org/viewvc/commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/DaemonPermission.java?rev=1694837&r1=1694836&r2=1694837&view=diff
==============================================================================
--- commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/DaemonPermission.java (original)
+++ commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/DaemonPermission.java Sat Aug  8 18:41:30 2015
@@ -190,8 +190,9 @@ public final class DaemonPermission exte
         super(target);
 
         // Check if the permission target name was specified
-        if (target == null)
+        if (target == null) {
             throw new IllegalArgumentException("Null permission name");
+        }
 
         // Check if this is a "control" permission and set up accordingly.
         if (CONTROL.equalsIgnoreCase(target)) {
@@ -266,16 +267,19 @@ public final class DaemonPermission exte
      */
     public boolean equals(Object object)
     {
-        if (object == this)
+        if (object == this) {
             return true;
+        }
 
-        if (!(object instanceof DaemonPermission))
+        if (!(object instanceof DaemonPermission)) {
             return false;
+        }
 
         DaemonPermission that = (DaemonPermission) object;
 
-        if (this.type != that.type)
+        if (this.type != that.type) {
             return false;
+        }
         return this.mask == that.mask;
     }
 
@@ -289,16 +293,19 @@ public final class DaemonPermission exte
      */
     public boolean implies(Permission permission)
     {
-        if (permission == this)
+        if (permission == this) {
             return true;
+        }
 
-        if (!(permission instanceof DaemonPermission))
+        if (!(permission instanceof DaemonPermission)) {
             return false;
+        }
 
         DaemonPermission that = (DaemonPermission) permission;
 
-        if (this.type != that.type)
+        if (this.type != that.type) {
             return false;
+        }
         return (this.mask & that.mask) == that.mask;
     }
 
@@ -323,8 +330,9 @@ public final class DaemonPermission exte
      */
     private void setupDescription()
     {
-        if (this.desc != null)
+        if (this.desc != null) {
             return;
+        }
 
         StringBuffer buf = new StringBuffer();
         buf.append(this.getClass().getName());
@@ -350,8 +358,9 @@ public final class DaemonPermission exte
     private int createControlMask(String actions)
         throws IllegalArgumentException
     {
-        if (actions == null)
+        if (actions == null) {
             return 0;
+        }
 
         int mask = 0;
         StringTokenizer tok = new StringTokenizer(actions, ",", false);
@@ -395,26 +404,32 @@ public final class DaemonPermission exte
         }
 
         if ((mask & MASK_CONTROL_STOP) == MASK_CONTROL_STOP) {
-            if (sep)
+            if (sep) {
                 buf.append(",");
-            else
+            }
+            else {
                 sep = true;
+            }
             buf.append(CONTROL_STOP);
         }
 
         if ((mask & MASK_CONTROL_SHUTDOWN) == MASK_CONTROL_SHUTDOWN) {
-            if (sep)
+            if (sep) {
                 buf.append(",");
-            else
+            }
+            else {
                 sep = true;
+            }
             buf.append(CONTROL_SHUTDOWN);
         }
 
         if ((mask & MASK_CONTROL_RELOAD) == MASK_CONTROL_RELOAD) {
-            if (sep)
+            if (sep) {
                 buf.append(",");
-            else
+            }
+            else {
                 sep = true;
+            }
             buf.append(CONTROL_RELOAD);
         }
 

Modified: commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonConfiguration.java
URL: http://svn.apache.org/viewvc/commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonConfiguration.java?rev=1694837&r1=1694836&r2=1694837&view=diff
==============================================================================
--- commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonConfiguration.java (original)
+++ commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonConfiguration.java Sat Aug  8 18:41:30 2015
@@ -85,8 +85,9 @@ public final class DaemonConfiguration
         boolean ok = false;
         FileInputStream file = null;
         try {
-            if (fileName == null)
+            if (fileName == null) {
                 fileName = DEFAULT_CONFIG;
+            }
             file = new FileInputStream(fileName);
             configurationProperties.clear();
             configurationProperties.load(file);
@@ -99,8 +100,9 @@ public final class DaemonConfiguration
             // Error reading properties file
         } finally {
             try {
-                if (file != null)
+                if (file != null) {
                     file.close();
+                }
             } catch (IOException ex) {
             }
         }
@@ -164,10 +166,10 @@ public final class DaemonConfiguration
     public String getProperty(String name)
         throws ParseException
     {
-        if (name == null)
+        if (name == null) {
             return null;
-        else
-            return expandProperty(configurationProperties.getProperty(PREFIX + name));
+        }
+        return expandProperty(configurationProperties.getProperty(PREFIX + name));
     }
 
     /**

Modified: commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonLoader.java
URL: http://svn.apache.org/viewvc/commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonLoader.java?rev=1694837&r1=1694836&r2=1694837&view=diff
==============================================================================
--- commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonLoader.java (original)
+++ commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonLoader.java Sat Aug  8 18:41:30 2015
@@ -65,8 +65,9 @@ public final class DaemonLoader
     {
         try {
             /* Check the class name */
-            if (cn == null)
+            if (cn == null) {
                 throw new NullPointerException("Null class name specified");
+            }
 
             /* Get the ClassLoader loading this class */
             ClassLoader cl = DaemonLoader.class.getClassLoader();
@@ -79,8 +80,9 @@ public final class DaemonLoader
             Class c = cl.loadClass(cn);
 
             /* This should _never_ happen, but doublechecking doesn't harm */
-            if (c == null)
+            if (c == null) {
                 throw new ClassNotFoundException(cn);
+            }
 
             /* Create a new instance of the daemon */
             c.newInstance();
@@ -122,12 +124,14 @@ public final class DaemonLoader
 
             /* Check if the underlying libray supplied a valid list of
                arguments */
-            if (args == null)
+            if (args == null) {
                 args = new String[0];
+            }
 
             /* Check the class name */
-            if (className == null)
+            if (className == null) {
                 throw new NullPointerException("Null class name specified");
+            }
 
             /* Get the ClassLoader loading this class */
             ClassLoader cl = DaemonLoader.class.getClassLoader();
@@ -147,11 +151,13 @@ public final class DaemonLoader
                 System.arraycopy(args, 0, a, 2, args.length);
                 args = a;
             }
-            else
+            else {
                 c = cl.loadClass(className);
+            }
             /* This should _never_ happen, but doublechecking doesn't harm */
-            if (c == null)
+            if (c == null) {
                 throw new ClassNotFoundException(className);
+            }
             /* Check interfaces */
             boolean isdaemon = false;
 
@@ -243,8 +249,9 @@ public final class DaemonLoader
             start.invoke(daemon, arg);
 
             /* Set the availability flag in the controller */
-            if (controller != null)
+            if (controller != null) {
                 controller.setAvailable(true);
+            }
 
         } catch (Throwable t) {
             /* In case we encounter ANY error, we dump the stack trace and
@@ -260,8 +267,9 @@ public final class DaemonLoader
     {
         try {
             /* Set the availability flag in the controller */
-            if (controller != null)
+            if (controller != null) {
                 controller.setAvailable(false);
+            }
 
             /* Attempt to stop the daemon */
             Object arg[] = null;
@@ -381,10 +389,12 @@ public final class DaemonLoader
                 this.setAvailable(false);
                 String msg = message;
                 if (exception != null) {
-                    if (msg != null)
+                    if (msg != null) {
                         msg = msg + ": " + exception.toString();
-                    else
+                    }
+                    else {
                         msg = exception.toString();
+                    }
                 }
                 DaemonLoader.failed(msg);
             }
@@ -422,4 +432,3 @@ public final class DaemonLoader
 
     }
 }
-

Modified: commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonWrapper.java
URL: http://svn.apache.org/viewvc/commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonWrapper.java?rev=1694837&r1=1694836&r2=1694837&view=diff
==============================================================================
--- commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonWrapper.java (original)
+++ commons/proper/daemon/trunk/src/main/java/org/apache/commons/daemon/support/DaemonWrapper.java Sat Aug  8 18:41:30 2015
@@ -101,33 +101,39 @@ public class DaemonWrapper implements Da
                     break;
                 }
                 else if (args[i].equals("-daemon-properties")) {
-                    if (++i == args.length)
+                    if (++i == args.length) {
                         throw new IllegalArgumentException(args[i - 1]);
+                    }
                     configFileName = args[i];
                 }
                 else if (args[i].equals("-start")) {
-                    if (++i == args.length)
+                    if (++i == args.length) {
                         throw new IllegalArgumentException(args[i - 1]);
+                    }
                     startup.setClassName(args[i]);
                 }
                 else if (args[i].equals("-start-method")) {
-                    if (++i == args.length)
+                    if (++i == args.length) {
                         throw new IllegalArgumentException(args[i - 1]);
+                    }
                     startup.setMethodName(args[i]);
                 }
                 else if (args[i].equals("-stop")) {
-                    if (++i == args.length)
+                    if (++i == args.length) {
                         throw new IllegalArgumentException(args[i - 1]);
+                    }
                     shutdown.setClassName(args[i]);
                 }
                 else if (args[i].equals("-stop-method")) {
-                    if (++i == args.length)
+                    if (++i == args.length) {
                         throw new IllegalArgumentException(args[i - 1]);
+                    }
                     shutdown.setMethodName(args[i]);
                 }
                 else if (args[i].equals("-stop-argument")) {
-                    if (++i == args.length)
+                    if (++i == args.length) {
                         throw new IllegalArgumentException(args[i - 1]);
+                    }
                     String[] aa = new String[1];
                     aa[0] = args[i];
                     shutdown.addArguments(aa);
@@ -198,20 +204,23 @@ public class DaemonWrapper implements Da
 
         protected void setClassName(String name)
         {
-            if (this.name == null)
+            if (this.name == null) {
                 this.name = name;
+            }
         }
         protected void setMethodName(String name)
         {
-            if (this.call == null)
+            if (this.call == null) {
                 this.call = name;
+            }
         }
         protected void addArguments(String[] args)
         {
             if (args != null) {
                 ArrayList aa = new ArrayList();
-                if (this.args != null)
+                if (this.args != null) {
                     aa.addAll(Arrays.asList(this.args));
+                }
                 aa.addAll(Arrays.asList(args));
                 this.args = (String[])aa.toArray(new String[aa.size()]);
             }
@@ -248,21 +257,25 @@ public class DaemonWrapper implements Da
                 call = "exit";
                 return;
             }
-            if (args == null)
+            if (args == null) {
                 args = new String[0];
-            if (call == null)
+            }
+            if (call == null) {
                 call = "main";
+            }
 
             // Get the ClassLoader loading this class
             ClassLoader cl = DaemonWrapper.class.getClassLoader();
-            if (cl == null)
+            if (cl == null) {
                 throw new NullPointerException("Cannot retrieve ClassLoader instance");
+            }
             Class[] ca = new Class[1];
             ca[0]      = args.getClass();
             // Find the required class
             main = cl.loadClass(name);
-            if (main == null)
+            if (main == null) {
                 throw new ClassNotFoundException(name);
+            }
             // Find the required method.
             // NoSuchMethodException will be thrown if matching method
             // is not found.