[PATCH libinput 0/6] Return value cleanup
Yong Bakos
junk at humanoriented.com
Tue Jul 19 20:05:08 UTC 2016
Hi Peter,
> On Jul 18, 2016, at 5:49 PM, Peter Hutterer <peter.hutterer at who-t.net> wrote:
>
>
> Return values are a bit messy, we randomly use bool and int. Functions
> returning int sometimes return 0 or 1, sometimes 0 and -1, etc. This set
> cleans up a bit, mostly by switching a bunch of functions to just be void
> and the rest to actual bool true/false returns (with renames where needed
> for obviousness)
Other than the trailing return in 2/6, looks good to me. I would explicitly
add the includes for stdbool, but that's at your discretion.
> No functional changes, should be an easy review for a rainy day.
Or to procrastinate other work...
With my comments above in mind, this is
Reviewed-by: Yong Bakos <ybakos at humanoriented.com>
yong
> Cheers,
> Peter
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel
More information about the wayland-devel
mailing list