Avoid trailing space in processing instructions

Signed-off-by: Anthony Vanelverdinghe <[email protected]>
angelozerr · Nov 16, 2020 · 3bf5229 · 3bf5229
1 parent 51b6d6b
commit 3bf5229
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/utils/XMLBuilder.java b/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/utils/XMLBuilder.java
@@ -340,7 +340,6 @@ public XMLBuilder startPrologOrPI(String tagName) {
 	public XMLBuilder addContentPI(String content) {
 		appendSpace();
 		append(content);
-		appendSpace();
 		return this;
 	}
 

diff --git a/org.eclipse.lemminx/src/test/java/org/eclipse/lemminx/services/XMLFormatterTest.java b/org.eclipse.lemminx/src/test/java/org/eclipse/lemminx/services/XMLFormatterTest.java
@@ -366,7 +366,7 @@ public void testProlog4WithUnknownVariable() throws BadLocationException {
 	public void testPI() throws BadLocationException {
 		String content = "<a><?m2e asd as das das ?></a>";
 		String expected = "<a>" + lineSeparator() + //
-				"  <?m2e asd as das das ?>" + lineSeparator() + //
+				"  <?m2e asd as das das?>" + lineSeparator() + //
 				"</a>";
 		assertFormat(content, expected);
 	}
@@ -404,7 +404,7 @@ public void testDefinedPIWithJustAttributeNames() throws BadLocationException {
 	public void testPIWithVariables() throws BadLocationException {
 		String content = "<a><?xml-styleZZ   href=\"my-style.css\"     type=   \"text/css\"?></a>";
 		String expected = "<a>" + lineSeparator() + //
-				"  <?xml-styleZZ href=\"my-style.css\"     type=   \"text/css\" ?>" + lineSeparator() + //
+				"  <?xml-styleZZ href=\"my-style.css\"     type=   \"text/css\"?>" + lineSeparator() + //
 				"</a>";
 		assertFormat(content, expected);
 	}