Re: [head tinderbox] failure on sparc64/sparc64

From: Bjoern A. Zeeb <bzeeb-lists_at_lists.zabbadoz.net>
Date: Tue, 9 Jun 2009 07:42:24 +0000 (UTC)
On Tue, 9 Jun 2009, Xin LI wrote:

> Danny Braniss wrote:
>>> Xin, good day.
>>>
>>> Mon, Jun 08, 2009 at 04:49:36PM -0700, Xin LI wrote:
>>>> The attached patch should fix this, any objections?
>>> Yes, you missed negation operator in the copyin check.  The issue
>>> was already fixed by hrs_at_ two hours ago:
>>>   http://svn.freebsd.org/viewvc/base?view=revision&revision=193796
>> sorry to barge in, but:
>>            if ((options | GIF_FULLOPTS) == GIF_FULLOPTS)
>> is not clear,
>> 	   if ((options & ~GIF_FULLOPTS) == 0)
>> seems to be less offuscated or I'm missing something?
>
> Yes this looks like the usually used idiom (perhaps more efficient
> anyway)...  I just kept the style consistent with the old code.
> Hiroki-san, could you have a look at this and consider if we should use
> this idiom?

Also see the mail I had sent in reply to the commit message yesterday.

/bz

-- 
Bjoern A. Zeeb                      The greatest risk is not taking one.
Received on Tue Jun 09 2009 - 05:45:07 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:49 UTC