[PATCH vkd3d v3 01/12] vkd3d-shader/hlsl: Add support for sm4 instruction modifiers.

Giovanni Mascellani gmascellani at codeweavers.com
Wed Jan 19 06:48:24 CST 2022


Hi,

Il 19/01/22 13:02, Matteo Bruni ha scritto:
>> Ech, this is hard to read, how about a simpler:
>>
>>       if (instr->modifier_count > 0)
>>           token |= VKD3D_SM4_INSTRUCTION_MODIFIER;
> Or even
> 
> if (instr->modifier_count)
>      token |= VKD3D_SM4_INSTRUCTION_MODIFIER;

Not a big deal, but between these two I like Zeb's version more, in that 
it makes more explicit that we're dealing with a number and not with a 
boolean condition.

Giovanni.



More information about the wine-devel mailing list