Stop ignoring whitespace while testing.
authorCarl Worth <cworth@cworth.org>
Thu, 20 May 2010 21:00:28 +0000 (14:00 -0700)
committerCarl Worth <cworth@cworth.org>
Thu, 20 May 2010 21:01:59 +0000 (14:01 -0700)
Sometime back the output of glcpp started differing from the output of
"gcc -E" in the amount of whitespace in emitted. At the time, I
switched the test suite to use "diff -w" to ignore this. This was a
mistake since it ignores whitespace entirely. (I meant to use "diff
-b" which ignores only changes in the amount of whitespace.)

So bugs have since been introduced that the test suite doesn't
notice. For example, glcpp is producing "twotokens" where it should be
producing "two tokens".

Let's stop ignoring whitespace in the test suite, which currently
introduces lots of failures---some real and some spurious.

tests/glcpp-test

index 673a4f4..25685ee 100755 (executable)
@@ -5,5 +5,5 @@ for test in *.c; do
     ../glcpp < $test > $test.out
     gcc -E $test -o $test.gcc
     grep -v '^#' < $test.gcc > $test.expected
-    diff -B -w -u $test.expected $test.out
+    diff -u $test.expected $test.out
 done