[PATCH] dinput: Avoid negating FAILED()

Michael Stefaniuc mstefani at winehq.org
Tue Oct 19 16:32:48 CDT 2021


On 10/19/21 23:06, Rémi Bernon wrote:
> Signed-off-by: Rémi Bernon <rbernon at codeweavers.com>
> ---
> 
> I don't really mind, but it's a bit longer to write and I probably got used to 
> using FAILED more or less consistently as some kind of error handling keyword. 
> Is it actually making a difference?
It is a true no-op so no difference there.
Just a style issue issue, supposedly easier to read. And "think positive" and such.
Not a big issue but Wine is fairly consistent there with just a handful of 
stragglers added each year, if at all.


bye
	michael



More information about the wine-devel mailing list