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 2013/07/15 23:52:08 UTC

svn commit: r1503493 - /tomcat/trunk/java/org/apache/el/stream/Stream.java

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-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org