You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2014/09/08 00:32:45 UTC

svn commit: r1623238 - /tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java

Author: markt
Date: Sun Sep  7 22:32:45 2014
New Revision: 1623238

URL: http://svn.apache.org/r1623238
Log:
Checkstyle codeblocks

Modified:
    tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java

Modified: tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java?rev=1623238&r1=1623237&r2=1623238&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java Sun Sep  7 22:32:45 2014
@@ -154,21 +154,17 @@ public class WebAnnotationSet {
         /* Process Resource annotation.
          * Ref JSR 250
          */
-        {
-            Resource annotation = classClass.getAnnotation(Resource.class);
-            if (annotation != null) {
-                addResource(context, annotation);
-            }
+        Resource resourceAnnotation = classClass.getAnnotation(Resource.class);
+        if (resourceAnnotation != null) {
+            addResource(context, resourceAnnotation);
         }
         /* Process Resources annotation.
          * Ref JSR 250
          */
-        {
-            Resources annotation = classClass.getAnnotation(Resources.class);
-            if (annotation != null && annotation.value() != null) {
-                for (Resource resource : annotation.value()) {
-                    addResource(context, resource);
-                }
+        Resources resourcesAnnotation = classClass.getAnnotation(Resources.class);
+        if (resourcesAnnotation != null && resourcesAnnotation.value() != null) {
+            for (Resource resource : resourcesAnnotation.value()) {
+                addResource(context, resource);
             }
         }
         /* Process EJB annotation.
@@ -244,13 +240,11 @@ public class WebAnnotationSet {
          * Ref JSR 250, equivalent to the security-role element in
          * the deployment descriptor
          */
-        {
-            DeclareRoles annotation = classClass
-                    .getAnnotation(DeclareRoles.class);
-            if (annotation != null && annotation.value() != null) {
-                for (String role : annotation.value()) {
-                    context.addSecurityRole(role);
-                }
+        DeclareRoles declareRolesAnnotation = classClass
+                .getAnnotation(DeclareRoles.class);
+        if (declareRolesAnnotation != null && declareRolesAnnotation.value() != null) {
+            for (String role : declareRolesAnnotation.value()) {
+                context.addSecurityRole(role);
             }
         }
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org