Do not indent new lines (#127)

Author: gastaldi

src/main/java/io/fabric8/maven/LineBreakProcessor.java

@@ -13,17 +13,46 @@
  * Inserts a line break on every major section
  */
 class LineBreakProcessor extends AbstractXMLOutputProcessor {
+
+    private String buffer;
+
     @Override
     protected void printElement(Writer out, FormatStack fstack, NamespaceStack nstack, Element element) throws IOException {
+        Element nextSiblingElement = findNextSiblingElement(element);
+        boolean shouldBreakLine = false;
+        if (nextSiblingElement != null) {
+            shouldBreakLine = shouldBreakLine(nextSiblingElement);
+        }
+        super.printElement(out, fstack, nstack, element);
+        buffer = (shouldBreakLine) ? fstack.getLineSeparator() : null;
+    }
+
+    private Element findNextSiblingElement(Element element) {
+        Element parent = element.getParentElement();
+        if (parent != null) {
+            List<Element> children = parent.getChildren();
+            int idx = children.indexOf(element);
+            if (idx < children.size() - 1) {
+                return children.get(idx + 1);
+            }
+        }
+        return null;
+    }
+
+    @Override
+    protected void textRaw(Writer out, String str) throws IOException {
+        if (buffer != null) {
+            out.write(buffer);
+            buffer = null;
+        }
+        super.textRaw(out, str);
+    }
+
+    private static boolean shouldBreakLine(Element element) {
         Element rootElement = element.getDocument().getRootElement();
         // Test if it's a major section
         int idxElement = rootElement.indexOf(element);
         List<Element> children = element.getChildren();
-        boolean addBreak = (idxElement > -1 && !children.isEmpty());
-        if (addBreak) {
-            textRaw(out, fstack.getLineSeparator());
-            textRaw(out, fstack.getLevelIndent());
-        }
-        super.printElement(out, fstack, nstack, element);
+        return (idxElement > -1 && !children.isEmpty());
     }
 }

src/test/approvals/io/fabric8/maven/MavenTest.should_write_existing_model_formatted.approved.txt

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
-    
+
     <parent>
         <groupId>io.quarkiverse.githubapp</groupId>
         <artifactId>quarkus-github-app-parent</artifactId>
@@ -11,7 +11,7 @@
     <artifactId>quarkus-github-app</artifactId>
     <name>Quarkus - GitHub App - Runtime</name>
     <description>Automate GitHub tasks with a GitHub App</description>
-    
+
     <dependencies>
         <dependency>
             <groupId>io.quarkiverse.githubapp</groupId>
@@ -59,7 +59,7 @@
             <artifactId>graal-sdk</artifactId>
         </dependency>
     </dependencies>
-    
+
     <build>
         <plugins>
             <plugin>

src/test/approvals/io/fabric8/maven/XMLFormatTest.should_ident_and_insert_line_break_on_major_sections.approved.txt

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
-    
+
     <parent>
         <groupId>io.quarkiverse.githubapp</groupId>
         <artifactId>quarkus-github-app-parent</artifactId>
@@ -11,7 +11,7 @@
     <artifactId>quarkus-github-app</artifactId>
     <name>Quarkus - GitHub App - Runtime</name>
     <description>Automate GitHub tasks with a GitHub App</description>
-    
+
     <dependencies>
         <dependency>
             <groupId>io.quarkiverse.githubapp</groupId>
@@ -59,7 +59,7 @@
             <artifactId>graal-sdk</artifactId>
         </dependency>
     </dependencies>
-    
+
     <build>
         <plugins>
             <plugin>