tests: remove FIXME comments from tricks to pacify syntax checks
authorStefano Lattarini <stefano.lattarini@gmail.com>
Fri, 23 Dec 2011 20:01:59 +0000 (21:01 +0100)
committerStefano Lattarini <stefano.lattarini@gmail.com>
Fri, 23 Dec 2011 20:06:43 +0000 (21:06 +0100)
commit2a0534dad9161af0e9dc7fdf5cfc6dba8414c293
tree53924b4047b91e0db9cba8bc464510bfaf3fbfe3
parente3186cb4953d35da93e4648355f60b2efb5ff0d6
tests: remove FIXME comments from tricks to pacify syntax checks

Originally, I myself had insisted on adding those "FIXME" comments
near every unusual construct or use of "creative quoting" whose
only purpose was to avoid false positive from out syntax checks.
I did that with the expectation that we could have easily and soon
added a whitelisting capability to our maintainer-specific syntax
checks.  This hasn't been the case unfortunately, and now those
"FIXME" in the comments are just distracting noise, making it more
difficult to grep the test cases for stuff that could really use a
fix or an improvement.  So just get rid of them.

* tests/instdir-ltlib.test: Remove that extra "FIXME".
* tests/instdir-prog.test: Likewise.
* tests/parallel-tests3.test: Likewise.
* tests/parallel-tests-dry-run.test: Likewise.
* tests/parallel-tests-console-output.test: Likewise.
* tests/parallel-tests-no-color-in-log.test: Likewise.
* tests/tap-realtime.test: Likewise.
* tests/test-trs-recover.test: Likewise.
* tests/tap-deps.test: Likewise.
ChangeLog
tests/instdir-ltlib.test
tests/instdir-prog.test
tests/parallel-tests-console-output.test
tests/parallel-tests-dry-run.test
tests/parallel-tests-no-color-in-log.test
tests/parallel-tests3.test
tests/tap-deps.test
tests/tap-realtime.test
tests/test-trs-recover.test