From 6eaac951dcdf0cbe39352683ce183f5a4f036b5f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Kautler?= <Bjoern@Kautler.net>
Date: Tue, 5 Mar 2019 17:03:52 +0100
Subject: [PATCH] Issue #4073: Use lf_cr_crlf as default for NewlineAtEndOfFile
 check

---
 .../checkstyle/checks/NewlineAtEndOfFileCheck.java    |  2 +-
 .../checks/NewlineAtEndOfFileCheckTest.java           | 11 ++---------
 src/xdocs/config_misc.xml                             |  2 +-
 3 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheck.java
index 86179f11f1c5..c0ffac33e115 100644
--- a/src/main/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheck.java
+++ b/src/main/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheck.java
@@ -71,7 +71,7 @@ public class NewlineAtEndOfFileCheck
     public static final String MSG_KEY_NO_NEWLINE_EOF = "noNewlineAtEOF";
 
     /** The line separator to check against. */
-    private LineSeparatorOption lineSeparator = LineSeparatorOption.SYSTEM;
+    private LineSeparatorOption lineSeparator = LineSeparatorOption.LF_CR_CRLF;
 
     @Override
     protected void processFiltered(File file, FileText fileText) {
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheckTest.java b/src/test/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheckTest.java
index d1c902b4740d..3c0561a026b4 100644
--- a/src/test/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheckTest.java
+++ b/src/test/java/com/puppycrawl/tools/checkstyle/checks/NewlineAtEndOfFileCheckTest.java
@@ -21,7 +21,6 @@
 
 import static com.puppycrawl.tools.checkstyle.checks.NewlineAtEndOfFileCheck.MSG_KEY_NO_NEWLINE_EOF;
 import static com.puppycrawl.tools.checkstyle.checks.NewlineAtEndOfFileCheck.MSG_KEY_UNABLE_OPEN;
-import static java.util.Locale.ENGLISH;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.fail;
 
@@ -224,14 +223,8 @@ public int read(byte[] bytes) {
             fail("Exception is expected");
         }
         catch (IOException ex) {
-            if (System.getProperty("os.name").toLowerCase(ENGLISH).startsWith("windows")) {
-                assertEquals("Error message is unexpected",
-                        "Unable to read 2 bytes, got 0", ex.getMessage());
-            }
-            else {
-                assertEquals("Error message is unexpected",
-                        "Unable to read 1 bytes, got 0", ex.getMessage());
-            }
+            assertEquals("Error message is unexpected",
+                    "Unable to read 1 bytes, got 0", ex.getMessage());
         }
     }
 
diff --git a/src/xdocs/config_misc.xml b/src/xdocs/config_misc.xml
index f2072ae677b2..3b630a500237 100644
--- a/src/xdocs/config_misc.xml
+++ b/src/xdocs/config_misc.xml
@@ -1236,7 +1236,7 @@ void foo(String aFooString,
             <td>lineSeparator</td>
             <td>type of line separator</td>
             <td><a href="property_types.html#lineSeparator">Line Separator Policy</a></td>
-            <td>system</td>
+            <td>lf_cr_crlf</td>
             <td>3.1</td>
           </tr>
           <tr>