Performance: Cache method calls in loops and optimize string concaten…

bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/MenuManagerShowProcessor.java

@@ -123,8 +123,8 @@ public void menuAboutToHide(IMenuManager manager) {
 	 * {@link MDynamicMenuContribution} application model elements
 	 */
 	private void processDynamicElements(MMenu menuModel, MenuManager menuManager) {
-		MMenuElement[] menuElements = menuModel.getChildren().toArray(
-				new MMenuElement[menuModel.getChildren().size()]);
+		List<MMenuElement> children = menuModel.getChildren();
+		MMenuElement[] menuElements = children.toArray(new MMenuElement[children.size()]);
 		for (MMenuElement currentMenuElement : menuElements) {
 
 			if (currentMenuElement instanceof MDynamicMenuContribution dmc) {
@@ -154,9 +154,9 @@ private void processDynamicElements(MMenu menuModel, MenuManager menuManager) {
 				if (mel.size() > 0) {
 
 					int position = 0;
-					while (position < menuModel.getChildren().size()) {
-						if (currentMenuElement == menuModel.getChildren().get(
-								position)) {
+					children = menuModel.getChildren();
+					while (position < children.size()) {
+						if (currentMenuElement == children.get(position)) {
 							position++;
 							break;
 						}

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/WorkbenchPage.java

@@ -3423,16 +3423,18 @@ public void resetPerspective() {
 		// Hide placeholders for parts that exist in the 'global' areas
 		modelService.hideLocalPlaceholders(window, dummyPerspective);
 
-		int dCount = dummyPerspective.getChildren().size();
-		while (!dummyPerspective.getChildren().isEmpty()) {
-			MPartSashContainerElement dChild = dummyPerspective.getChildren().remove(0);
-			persp.getChildren().add(dChild);
+		List<MPartSashContainerElement> dummyChildren = dummyPerspective.getChildren();
+		List<MPartSashContainerElement> perspChildren = persp.getChildren();
+		int dCount = dummyChildren.size();
+		while (!dummyChildren.isEmpty()) {
+			MPartSashContainerElement dChild = dummyChildren.remove(0);
+			perspChildren.add(dChild);
 		}
 
-		while (persp.getChildren().size() > dCount) {
-			MUIElement child = persp.getChildren().get(0);
+		while (perspChildren.size() > dCount) {
+			MUIElement child = perspChildren.get(0);
 			child.setToBeRendered(false);
-			persp.getChildren().remove(0);
+			perspChildren.remove(0);
 		}
 
 		List<MWindow> existingDetachedWindows = new ArrayList<>();

bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/registry/WizardsRegistryReader.java

@@ -76,14 +76,15 @@ private static class CategoryNode {
 
 		CategoryNode(Category cat) {
 			category = cat;
-			path = ""; //$NON-NLS-1$
 			String[] categoryPath = category.getParentPath();
+			StringBuilder pathBuilder = new StringBuilder();
 			if (categoryPath != null) {
 				for (String parentPath : categoryPath) {
-					path += parentPath + '/';
+					pathBuilder.append(parentPath).append('/');
 				}
 			}
-			path += cat.getId();
+			pathBuilder.append(cat.getId());
+			path = pathBuilder.toString();
 		}
 
 		String getPath() {