All in one method rather than for loop

Signed-off-by: Fred Bricon <fbricon@gmail.com>
eclipse · May 26, 2021 · 7ce8901 · 7ce8901
1 parent 88797bc
commit 7ce8901
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/...x/src/main/java/org/eclipse/lemminx/extensions/xsd/participants/XSDRenameParticipant.java b/...x/src/main/java/org/eclipse/lemminx/extensions/xsd/participants/XSDRenameParticipant.java
@@ -43,9 +43,7 @@ public void doRename(IRenameRequest request, List<TextEdit> locations) {
 		if (!DOMUtils.isXSD(xmlDocument)) {
 			return;
 		}
-		for (TextEdit textEdit: getRenameTextEdits(request)) {
-			locations.add(textEdit);
-		}
+		locations.addAll(getRenameTextEdits(request));
 
 	}
 

diff --git a/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/XMLRename.java b/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/XMLRename.java
@@ -84,9 +84,7 @@ public WorkspaceEdit doRename(DOMDocument xmlDocument, Position position, String
 			}
 		}
 
-		for (TextEdit textEdit : getRenameTextEdits(xmlDocument, node, position, newText)) {
-			textEdits.add(textEdit);
-		}
+		textEdits.addAll(getRenameTextEdits(xmlDocument, node, position, newText));
 
 		return createWorkspaceEdit(xmlDocument.getDocumentURI(), textEdits);
 	}