diff: more changes to --binary

The changes introduced to support the --binary option gave incorrect
results when comparing files with CRLF line endings *without* the
--binary option present.

The code needs to keep track of the position within the file and is
confused by text mode.

As an alternative solution, always use binary mode but skip the CR
of a CRLF pair when the --binary option isn't used.  This gives
results matching GNU diff when comparing files with matching line
endings, with or without --binary.  When line endings differ the
results aren't always the same.

Costs 32 bytes in the 32-bit build, saves 16 in 64-bit.

(GitHub issue #348)

1 files changed, 63 insertions, 0 deletions

diff --git a/testsuite/diff.tests b/testsuite/diff.tests
index 0ced0f248..ee0567a80 100755
--- a/testsuite/diff.tests
+++ b/testsuite/diff.tests
@@ -123,6 +123,69 @@ testing "diff always takes context from old file" \
         "abc\na  c\ndef\n" \
         "a c\n"
 
+optional PLATFORM_MINGW32 LONG_OPTS
+testing "diff LF line endings" \
+        'diff -u - input' \
+"\
+--- -
++++ input
+@@ -1,4 +1,4 @@
+ a
+ b
++c
+ d
+-e
+" \
+        "a\nb\nc\nd\n" \
+        "a\nb\nd\ne\n"
+
+testing "diff --binary LF line endings" \
+        'diff --binary -u - input' \
+"\
+--- -
++++ input
+@@ -1,4 +1,4 @@
+ a
+ b
++c
+ d
+-e
+" \
+        "a\nb\nc\nd\n" \
+        "a\nb\nd\ne\n"
+
+testing "diff CRLF line endings" \
+        'diff -u - input' \
+"\
+--- -
++++ input
+@@ -1,4 +1,4 @@
+ a


+ b


++c


+ d


+-e


+" \
+        "a\r\nb\r\nc\r\nd\r\n" \
+        "a\r\nb\r\nd\r\ne\r\n"
+
+testing "diff --binary CRLF line endings" \
+        'diff --binary -u - input' \
+"\
+--- -
++++ input
+@@ -1,4 +1,4 @@
+ a


+ b


++c


+ d


+-e


+" \
+        "a\r\nb\r\nc\r\nd\r\n" \
+        "a\r\nb\r\nd\r\ne\r\n"
+
+SKIP=
+
 # testing "test name" "commands" "expected result" "file input" "stdin"
 
 # clean up