Caching size in loops for json, remote and tools packages

Author: LuisOsv

java/src/dev/selenium/tools/javadoc/JavadocJarMaker.java

@@ -55,7 +55,8 @@ public static void main(String[] args) throws IOException {
     Path out = null;
     Set<Path> classpath = new HashSet<>();
 
-    for (int i = 0; i < args.length; i++) {
+    int argCount = args.length;
+    for (int i = 0; i < argCount; i++) {
       String flag = args[i];
       String next = args[++i];
 

java/src/dev/selenium/tools/modules/ModuleGenerator.java

@@ -108,7 +108,8 @@ public static void main(String[] args) throws IOException {
     Set<String> openTo = new TreeSet<>();
     boolean isOpen = false;
 
-    for (int i = 0; i < args.length; i++) {
+    int argCount = args.length;
+    for (int i = 0; i < argCount; i++) {
       String flag = args[i];
       String next = args[++i];
       switch (flag) {

java/src/org/openqa/selenium/json/JsonInput.java

@@ -507,7 +507,8 @@ private void expect(JsonType type) {
   private <X> X read(String toCompare, Function<String, X> mapper) {
     skipWhitespace(input);
 
-    for (int i = 0; i < toCompare.length(); i++) {
+    int toCompareLength = toCompare.length();
+    for (int i = 0; i < toCompareLength; i++) {
       char read = input.read();
       if (read != toCompare.charAt(i)) {
         throw new JsonException(

java/src/org/openqa/selenium/remote/WebElementToJsonConverter.java

@@ -84,8 +84,9 @@ public Object apply(Object arg) {
   }
 
   private static List<Object> arrayToList(Object array) {
-    List<Object> list = new ArrayList<>();
-    for (int i = 0; i < Array.getLength(array); i++) {
+    int arrayLength = Array.getLength(array);
+    List<Object> list = new ArrayList<>(arrayLength);
+    for (int i = 0; i < arrayLength; i++) {
       list.add(Array.get(array, i));
     }
     return list;

java/src/org/openqa/selenium/remote/codec/AbstractHttpCommandCodec.java

@@ -387,11 +387,12 @@ boolean isFor(HttpMethod method, List<String> parts) {
         return false;
       }
 
-      if (parts.size() != this.pathSegments.size()) {
+      int pathSegmentCount = parts.size();
+      if (pathSegmentCount != this.pathSegments.size()) {
         return false;
       }
 
-      for (int i = 0; i < parts.size(); ++i) {
+      for (int i = 0; i < pathSegmentCount; ++i) {
         String reqPart = parts.get(i);
         String specPart = pathSegments.get(i);
         if (!(specPart.startsWith(":") || specPart.equals(reqPart))) {
@@ -403,7 +404,8 @@ boolean isFor(HttpMethod method, List<String> parts) {
     }
 
     void parsePathParameters(List<String> parts, Map<String, Object> parameters) {
-      for (int i = 0; i < parts.size(); ++i) {
+      int pathSegmentCount = parts.size();
+      for (int i = 0; i < pathSegmentCount; ++i) {
         if (pathSegments.get(i).startsWith(":")) {
           parameters.put(pathSegments.get(i).substring(1), parts.get(i));
         }

java/src/org/openqa/selenium/remote/http/Route.java

@@ -254,10 +254,11 @@ public boolean matches(HttpRequest request) {
 
     private boolean hasPrefix(HttpRequest request) {
       String[] parts = request.getUri().split("/");
-      if (parts.length < prefixPaths.length) {
+      int prefixPathCount = prefixPaths.length;
+      if (parts.length < prefixPathCount) {
         return false;
       }
-      for (int i = 0; i < prefixPaths.length; i++) {
+      for (int i = 0; i < prefixPathCount; i++) {
         if (!prefixPaths[i].equals(parts[i])) {
           return false;
         }

java/src/org/openqa/selenium/remote/http/UrlTemplate.java

@@ -69,7 +69,8 @@ public UrlTemplate(String template) {
 
     List<String> allGroups = List.copyOf(groups);
     // do we hit a fast path?
-    switch (allGroups.size()) {
+    int groupCount = allGroups.size();
+    switch (groupCount) {
       case 0: // no groups, just .equals
         this.compiled =
             (matchAgainst) -> {
@@ -119,7 +120,7 @@ public UrlTemplate(String template) {
               }
 
               Map<String, String> params = new LinkedHashMap<>();
-              for (int i = 0; i < allGroups.size(); i++) {
+              for (int i = 0; i < groupCount; i++) {
                 params.put(allGroups.get(i), matcher.group(i + 1));
               }