[MNG-6816] Prefer System.lineSeparator() for Writing System-Dependent Line Separator Strings in Java

maven-artifact/src/main/java/org/apache/maven/artifact/resolver/AbstractArtifactResolutionException.java

@@ -52,7 +52,7 @@ public class AbstractArtifactResolutionException
 
     private final String path;
 
-    static final String LS = System.getProperty( "line.separator" );
+    static final String LS = System.lineSeparator();
 
     @SuppressWarnings( "checkstyle:parameternumber" )
     protected AbstractArtifactResolutionException( String message,

maven-compat/src/main/java/org/apache/maven/project/validation/ModelValidationResult.java

@@ -30,7 +30,7 @@ public class ModelValidationResult
 {
 
     /** */
-    private static final String NEWLINE = System.getProperty( "line.separator" );
+    private static final String NEWLINE = System.lineSeparator();
 
     /** */
     private List<String> messages;

maven-compat/src/test/java/org/apache/maven/artifact/resolver/ArtifactResolutionExceptionTest.java

@@ -32,7 +32,7 @@
 public class ArtifactResolutionExceptionTest
     extends TestCase
 {
-    private static final String LS = System.getProperty( "line.separator" );
+    private static final String LS = System.lineSeparator();
 
     public void testMissingArtifactMessageFormat()
     {

maven-core/src/test/java/org/apache/maven/plugin/PluginParameterExpressionEvaluatorTest.java

@@ -53,7 +53,7 @@
 public class PluginParameterExpressionEvaluatorTest
     extends AbstractCoreMavenComponentTestCase
 {
-    private static final String FS = System.getProperty( "file.separator" );
+    private static final String FS = File.separator;
 
     private RepositorySystem factory;
 

maven-core/src/test/java/org/apache/maven/toolchain/building/DefaultToolchainsBuilderTest.java

@@ -47,7 +47,7 @@
 
 public class DefaultToolchainsBuilderTest
 {
-    private static final String LS = System.getProperty( "line.separator" );
+    private static final String LS = System.lineSeparator();
 
     @Spy
     private DefaultToolchainsReader toolchainsReader;

maven-core/src/test/java/org/apache/maven/toolchain/building/ToolchainsBuildingExceptionTest.java

@@ -30,7 +30,7 @@
 
 public class ToolchainsBuildingExceptionTest
 {
-    private static final String LS = System.getProperty( "line.separator" );
+    private static final String LS = System.lineSeparator();
 
     @Test
     public void testNoProblems()

maven-embedder/src/main/java/org/apache/maven/cli/CLIReportingUtils.java

@@ -55,7 +55,7 @@ public final class CLIReportingUtils
 
     public static String showVersion()
     {
-        final String ls = System.getProperty( "line.separator" );
+        final String ls = System.lineSeparator();
         Properties properties = getBuildProperties();
         StringBuilder version = new StringBuilder( 256 );
         version.append( buffer().strong( createMavenVersionString( properties ) ) ).append( ls );

maven-embedder/src/main/java/org/apache/maven/cli/logging/impl/UnsupportedSlf4jBindingConfiguration.java

@@ -54,7 +54,7 @@ public void activate()
         logger.warn( "The SLF4J binding actually used is not supported by Maven: {}", slf4jBinding );
         logger.warn( "Maven supported bindings are:" );
 
-        String ls = System.getProperty( "line.separator" );
+        String ls = System.lineSeparator();
 
         for ( Map.Entry<URL, Set<Object>> entry : supported.entrySet() )
         {

maven-embedder/src/test/java/org/apache/maven/cli/CLIManagerDocumentationTest.java

@@ -39,7 +39,7 @@
 public class CLIManagerDocumentationTest
     extends TestCase
 {
-    private final static String LS = System.getProperty( "line.separator" );
+    private final static String LS = System.lineSeparator();
 
     private static class OptionComparator
         implements Comparator<Option>