[PATCH rendercheck] Report the success_mask to stdout

Jeremy Huddleston jeremyhu at apple.com
Mon May 9 15:37:30 PDT 2011


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 <jeremyhu at apple.com> wrote:
>> 
>> This was previously computed but never passed on to the caller.
>> 
>> Found-by: clang static analyzer
>> Signed-off-by: Jeremy Huddleston <jeremyhu at 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;
>> }
>> -- 



More information about the xorg-devel mailing list