> On Wed, 2008-02-06 at 21:26 +0200, Tsantilas Christos wrote:
>> Do we need something like that? Any comments/suggestions? Any testers?
>
> I believe we do and I appreciate you working on it! Please try to fix
> the remaining problems Amos pointed out.
>
> Also, can you apply it to the entire Squid tree, remove all whitespace,
> and calculate md5, comparing that with the virgin whitespace-less tree?
> The MD5s should be the same, right?
Oooh. Nice test.
>
> This is not a perfect check because spaces within strings/etc are
> stripped and not checked, but it is a pretty good one.
It will also miss the "#if 0 {" block problem.
Though my unit-test for that will catch it any any other whitsspace-exact
case we care to write the test for.
We may need a compile test for any other similar ones that have not had
test-cases written.
Just have to figure out the best way to have the test file built into the
codebase for automatic testing.
Amos
Received on Thu Feb 07 2008 - 18:31:18 MST
This archive was generated by hypermail pre-2.1.9 : Sat Mar 01 2008 - 12:00:09 MST