You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2014/06/19 22:02:49 UTC
svn commit: r1604026 -
/tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java
Author: kkolinko
Date: Thu Jun 19 20:02:48 2014
New Revision: 1604026
URL: http://svn.apache.org/r1604026
Log:
Use for-each iterations
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=1604026&r1=1604025&r2=1604026&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/WebAnnotationSet.java Thu Jun 19 20:02:48 2014
@@ -75,10 +75,9 @@ public class WebAnnotationSet {
* Process the annotations for the listeners.
*/
protected static void loadApplicationListenerAnnotations(Context context) {
- Class<?> classClass = null;
String[] applicationListeners = context.findApplicationListeners();
- for (int i = 0; i < applicationListeners.length; i++) {
- classClass = Introspection.loadClass(context, applicationListeners[i]);
+ for (String className : applicationListeners) {
+ Class<?> classClass = Introspection.loadClass(context, className);
if (classClass == null) {
continue;
}
@@ -94,11 +93,10 @@ public class WebAnnotationSet {
* Process the annotations for the filters.
*/
protected static void loadApplicationFilterAnnotations(Context context) {
- Class<?> classClass = null;
FilterDef[] filterDefs = context.findFilterDefs();
- for (int i = 0; i < filterDefs.length; i++) {
- classClass = Introspection.loadClass(context,
- (filterDefs[i]).getFilterClass());
+ for (FilterDef filterDef : filterDefs) {
+ Class<?> classClass = Introspection.loadClass(context,
+ filterDef.getFilterClass());
if (classClass == null) {
continue;
}
@@ -115,19 +113,16 @@ public class WebAnnotationSet {
*/
protected static void loadApplicationServletAnnotations(Context context) {
- Wrapper wrapper = null;
- Class<?> classClass = null;
-
Container[] children = context.findChildren();
- for (int i = 0; i < children.length; i++) {
- if (children[i] instanceof Wrapper) {
+ for (Container child : children) {
+ if (child instanceof Wrapper) {
- wrapper = (Wrapper) children[i];
+ Wrapper wrapper = (Wrapper) child;
if (wrapper.getServletClass() == null) {
continue;
}
- classClass = Introspection.loadClass(context,
+ Class<?> classClass = Introspection.loadClass(context,
wrapper.getServletClass());
if (classClass == null) {
continue;
@@ -148,7 +143,6 @@ public class WebAnnotationSet {
}
}
-
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org