[PATCH weston] cosmetic: replace boolean function return values with bool

Pekka Paalanen ppaalanen at gmail.com
Tue Nov 4 02:48:35 PST 2014


On Mon, 13 Oct 2014 11:41:32 -0700
Bryce Harrington <bryce at osg.samsung.com> wrote:

> On Fri, Oct 03, 2014 at 01:13:42PM -0500, Derek Foreman wrote:
> > For functions that test if something is true/valid and return a 1
> > or 0, it makes sense to switch to bool.
> 
> I like it.  Probably many other places this shows up.
> 
> Only question I have is if bool can be used for WL_EXPORT functions?
> bool isn't mentioned in the wire format docs[0], so I'm guessing that
> might not be an allowed type for those routines?

This is a Weston patch and I don't see anything suspicious in using
stdbool.h, so it's ok here.

I kind of wonder what including stdbool.h does to C++ sources using
compositor.h, but if that breaks, we'll hear about it.

Looks good to me, so pushed!


Thanks,
pq

> 
> Assuming that's fine,
> 
> Reviewed-by: Bryce Harrington <b.harrington at samsung.com>
> 
> 0: http://wayland.freedesktop.org/docs/html/sect-Protocol-Wire-Format.html
> 
> > ---
> >  src/cms-colord.c    | 10 +++++-----
> >  src/compositor.c    | 20 ++++++++++----------
> >  src/compositor.h    |  5 +++--
> >  src/input.c         |  8 ++++----
> >  src/weston-launch.c | 11 ++++++-----
> >  5 files changed, 28 insertions(+), 26 deletions(-)


More information about the wayland-devel mailing list