JAL-4130 Replace Collection#forEach call with for loop
authorMateusz Warowny <mmzwarowny@dundee.ac.uk>
Wed, 22 Feb 2023 13:28:59 +0000 (14:28 +0100)
committerMateusz Warowny <mmzwarowny@dundee.ac.uk>
Wed, 22 Feb 2023 13:28:59 +0000 (14:28 +0100)
src/jalview/ws2/gui/WebServicesMenuManager.java

index 592a370..d6cf5b7 100644 (file)
@@ -153,18 +153,21 @@ public class WebServicesMenuManager
                 k -> new ArrayList<>())
             .add(action);
       }
-      actionsByCategory.forEach((k, v) -> {
+      for (var entry : actionsByCategory.entrySet())
+      {
+        var category = entry.getKey();
+        var actions = entry.getValue();
         // create submenu named {subcategory} with {service} or use root menu
-        var atMenu = k.isEmpty() ? menu : new JMenu(String.format("%s with %s", k, service.getName()));
+        var atMenu = category.isEmpty() ? menu : new JMenu(String.format("%s with %s", category, service.getName()));
         if (atMenu != menu)
           menu.add(atMenu); // add only if submenu
         // sort actions by name pulling nulls to the front
-        v.sort(Comparator.comparing(
+        actions.sort(Comparator.comparing(
             ActionI::getName, Comparator.nullsFirst(Comparator.naturalOrder())));
-        for (int i = 0; i < v.size(); i++) {
-          addEntriesForAction(v.get(i), atMenu, atMenu == menu);
+        for (int i = 0; i < actions.size(); i++) {
+          addEntriesForAction(actions.get(i), atMenu, atMenu == menu);
         }
-      });
+      }
     }
   }