From ed089bc2b829bf363bf0e719cd612e6f4e849c38 Mon Sep 17 00:00:00 2001
From: Arjen Poutsma <apoutsma@vmware.com>
Date: Fri, 27 Mar 2009 16:44:19 +0000
Subject: [PATCH] Made looping over handlerExceptionResolvers more explicit.

git-svn-id: https://src.springframework.org/svn/spring-framework/trunk@850 50f2f4bb-b051-0410-bef5-90022cba6387
---
 .../org/springframework/web/servlet/DispatcherServlet.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/DispatcherServlet.java b/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/DispatcherServlet.java
index 40d3077cd66..f245b41b4bf 100644
--- a/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/DispatcherServlet.java
+++ b/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/DispatcherServlet.java
@@ -1055,8 +1055,11 @@ public class DispatcherServlet extends FrameworkServlet {
 
 		// Check registerer HandlerExceptionResolvers...
 		ModelAndView exMv = null;
-		for (Iterator<HandlerExceptionResolver> it = this.handlerExceptionResolvers.iterator(); exMv == null && it.hasNext();) {
-			exMv = it.next().resolveException(request, response, handler, ex);
+		for (HandlerExceptionResolver handlerExceptionResolver : this.handlerExceptionResolvers) {
+			exMv = handlerExceptionResolver.resolveException(request, response, handler, ex);
+			if (exMv != null) {
+				break;
+			}
 		}
 		if (exMv != null) {
 			if (exMv.isEmpty()) {