Browse Source

Stop ignoring whitespace while testing.

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.
tags/mesa-7.9-rc1
Carl Worth 15 years ago
parent
commit
660bda057a
1 changed files with 1 additions and 1 deletions
  1. 1
    1
      tests/glcpp-test

+ 1
- 1
tests/glcpp-test View File

../glcpp < $test > $test.out ../glcpp < $test > $test.out
gcc -E $test -o $test.gcc gcc -E $test -o $test.gcc
grep -v '^#' < $test.gcc > $test.expected grep -v '^#' < $test.gcc > $test.expected
diff -B -w -u $test.expected $test.out
diff -u $test.expected $test.out
done done

Loading…
Cancel
Save