Author: kkolinko Date: Mon Jul 15 21:52:07 2013 New Revision: 1503493 URL: http://svn.apache.org/r1503493 Log: Followup to r1503469 Correct whitespaces. An integer comparison should be a bit cheaper that hasNext() call.
Modified: tomcat/trunk/java/org/apache/el/stream/Stream.java Modified: tomcat/trunk/java/org/apache/el/stream/Stream.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/el/stream/Stream.java?rev=1503493&r1=1503492&r2=1503493&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/el/stream/Stream.java (original) +++ tomcat/trunk/java/org/apache/el/stream/Stream.java Mon Jul 15 21:52:07 2013 @@ -218,7 +218,7 @@ public class Stream { return substream(start, Integer.valueOf(Integer.MAX_VALUE)); } - public Stream substream(final Number start, final Number end) { + public Stream substream(final Number start, final Number end) { Iterator<Object> downStream = new OpIterator() { @@ -232,8 +232,8 @@ public class Stream { iterator.next(); itemCount++; } - if (iterator.hasNext() && itemCount < endPos) { - itemCount ++; + if (itemCount < endPos && iterator.hasNext()) { + itemCount++; next = iterator.next(); foundNext = true; } --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org