27 Apr
2009
27 Apr
'09
8:14 p.m.
Hi Jeff, I just wanted to clarify one of Detlef's comments:
- ok( len == sizeof( expect4 ), "Got size %d\n", len); + ok( len == sizeof( expect4 ), "Expected size to be %d, got %d\n", sizeof( expect4 ), len);
Do not use sizeof() in a test.
Using sizeof is okay, but printing the result of it is not, due to compile warnings this will introduce on some platforms. Basically, omit this change from your patch. Thanks, --Juan