Well it's better than nothing... but ok, I'll beautify it.

On May 9, 2011, at 13:47, Eric Anholt wrote:

> On Sat, 7 May 2011 18:20:17 -0700, Jeremy Huddleston <jerem...@apple.com> 
> wrote:
>> 
>> This was previously computed but never passed on to the caller.
>> 
>> Found-by: clang static analyzer
>> Signed-off-by: Jeremy Huddleston <jerem...@apple.com>
> 
> Doing something useful with those success flags would be nice, but I
> don't think a hex dump of it is that.
> 
>> ---
>> tests.c |    2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>> 
>> diff --git a/tests.c b/tests.c
>> index f722bc6..1b47eee 100644
>> --- a/tests.c
>> +++ b/tests.c
>> @@ -752,7 +752,7 @@ do {                                                     
>>         \
>>      free(test_mask);
>>      free(test_dst);
>> 
>> -    printf("%d tests passed of %d total\n", tests_passed, tests_total);
>> +    printf("%d tests passed of %d total (successful groups: 0x%x)\n", 
>> tests_passed, tests_total, success_mask);
>> 
>>      return tests_passed == tests_total;
>> }
>> -- 

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to