[PATCH] dix: add valuator_mask_fetch_double()

Daniel Stone daniel at fooishbar.org
Fri Sep 30 15:55:35 PDT 2011


Hi,

On 30 September 2011 17:18, Chase Douglas <chase.douglas at canonical.com> wrote:
> On 09/29/2011 09:29 PM, Peter Hutterer wrote:
>> Using this call simplifies callers that don't know if the mask bit is set.
>>
>> Before:
>>   if (valuator_mask_isset(mask, valnum))
>>     value = valuator_mask_get_double(mask, valnum));
>>   else
>>     value = someothervalue;
>>
>> Now:
>>  if (!valuator_mask_fetch_double(mask, valnum, &value))
>>     value = someothervalue;
>>
>> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
>
> I think this will help clean up code all over :).
>
> Reviewed-by: Chase Douglas <chase.douglas at canonical.com>

Yeah, seriously.  Is there any good reason to not make a double which
returns either 0.0 (if not set) or the mask value (if set) though, on
the grounds that anyone who wants to know will be using
isset/get_double?

Cheers,
Daniel


More information about the xorg-devel mailing list