<fixcrlf> failed on Mac files.

Reported by:	Matt McHenry <mmchenry@carnegielearning.com>


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@270662 13f79535-47bb-0310-9956-ffa450edef68

WHATSNEW

@@ -60,6 +60,8 @@ Fixed bugs:
 * Some junit formatters incorrectly assumed that all testcases would
   inherit from junit.framework.TestCase.
 
+* <fixcrlf> dropped the first characters from Mac files.
+
 Other changes:
 --------------
 * <move> attempts to rename the directory, if everything inside it is

src/etc/testcases/taskdefs/fixcrlf/build.xml

@@ -97,6 +97,13 @@
              />
   </target>
 
+  <target name="testMacLines" depends="init">
+    <fixcrlf srcdir="input" destdir="result"
+             includes="Mac2Unix"
+             eol="lf"
+             />
+  </target>
+
   <target name="testEncoding" depends="init">
     <fixcrlf srcdir="input" destdir="result"
              includes="input.crlf.utf16"

src/etc/testcases/taskdefs/fixcrlf/expected/Mac2Unix

@@ -0,0 +1,2 @@
+line1
+line2

src/etc/testcases/taskdefs/fixcrlf/input/Mac2Unix

@@ -0,0 +1 @@
+line1
line2

src/main/org/apache/tools/ant/taskdefs/FixCRLF.java

@@ -57,6 +57,7 @@ package org.apache.tools.ant.taskdefs;
 import org.apache.tools.ant.BuildException;
 import org.apache.tools.ant.DirectoryScanner;
 import org.apache.tools.ant.Project;
+import org.apache.tools.ant.taskdefs.condition.Os;
 import org.apache.tools.ant.types.EnumeratedAttribute;
 import org.apache.tools.ant.util.FileUtils;
 
@@ -170,20 +171,18 @@ public class FixCRLF extends MatchingTask {
      */
     public FixCRLF () {
         tabs = ASIS;
-        if (System.getProperty("path.separator").equals(":")) {
+        if (Os.isFamily("mac")) {
             ctrlz = REMOVE;
-            if (System.getProperty("os.name").indexOf("Mac") > -1) {
-                eol = CR;
-                eolstr = "\r";
-            } else {
-                eol = LF;
-                eolstr = "\n";
-            }
-        }
-        else {
+            eol = CR;
+            eolstr = "\r";
+        } else if (Os.isFamily("dos")) {
             ctrlz = ASIS;
             eol = CRLF;
             eolstr = "\r\n";
+        } else {
+            ctrlz = REMOVE;
+            eol = LF;
+            eolstr = "\n";
         }
     }
 
@@ -853,6 +852,7 @@ public class FixCRLF extends MatchingTask {
                     // Regard \r\r not followed by \n as two lines
                     ++eolcount;
                     eolStr.append('\r');
+                    reader.mark(2);
                     switch ((char)(ch = reader.read())) {
                     case '\r':
                         if ((char)(ch = reader.read()) == '\n') {
@@ -864,6 +864,9 @@ public class FixCRLF extends MatchingTask {
                         ++eolcount;
                         eolStr.append('\n');
                         break;
+                    default:
+                        reader.reset();
+                        break;
                     } // end of switch ((char)(ch = reader.read()))
                     break;
                     

src/testcases/org/apache/tools/ant/taskdefs/FixCrLfTest.java

@@ -1,7 +1,7 @@
 /*
  * The Apache Software License, Version 1.1
  *
- * Copyright (c) 2001 The Apache Software Foundation.  All rights
+ * Copyright (c) 2001-2002 The Apache Software Foundation.  All rights
  * reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -132,6 +132,12 @@ public class FixCrLfTest extends BuildFileTest {
                            new File("src/etc/testcases/taskdefs/fixcrlf/result/Junk9.java"));
     }
     
+    public void testMacLines() throws IOException { 
+        executeTarget("testMacLines");
+        assertEqualContent(new File("src/etc/testcases/taskdefs/fixcrlf/expected/Mac2Unix"),
+                           new File("src/etc/testcases/taskdefs/fixcrlf/result/Mac2Unix"));
+    }
+    
     public void testNoOverwrite() throws IOException {
         executeTarget("test1");
         File result =