[U-Boot] [PATCH] Introduce a global bool type
Måns Rullgård
mans at mansr.com
Mon Jan 21 23:36:42 CET 2013
Scott Wood <scottwood at freescale.com> writes:
> On 01/19/2013 03:30:30 AM, Albert ARIBAUD wrote:
>> /* what I favor */
>> clk_is_enabled = ((reg_val >> 9) & 1) ? true: false;
>> ip_is_enabled = clk_is_enabled && pwd_is_enabled;
>> if (clk_is_enabled) { ...
>>
>> rather than assigning them 'zero/nonzero', or using bitwise ops on
>> booleans, or testing against boolean constants (although I concede
>> that the first line below wins over its counterpart above as far as
>> concision is concerned).
>
> Conciseness can be improved with "!!((reg_val >> 9) & 1)".
x & 1 is already either zero or one. Any further operations are nothing
but obfuscation.
--
Måns Rullgård
mans at mansr.com
More information about the U-Boot
mailing list