[PATCH weston 0/4] Extract common macros to single header.

Bryce Harrington bryce at osg.samsung.com
Mon Jun 15 13:48:27 PDT 2015


On Mon, Jun 15, 2015 at 10:57:31AM +1000, Peter Hutterer wrote:
> On Fri, Jun 12, 2015 at 09:25:01PM -0700, Jon A. Cruz wrote:
> > As a follow-up to review feedback from Peter Hutterer this
> > set moves a few duplicated macros into a common header,
> > including ARRAY_LENGTH.
> > 
> > Jon A. Cruz (4):
> >   Remove redundant #include path component.
> >   Moved helper macro to a discrete include file.
> >   Moved the MIN() macro to the helper include.
> >   Unified multiple definitions of container_of() macro.
> 
> Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net> for the series

LGTM too.

Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>

But can you update it to current head?  Looks like my boilerplate
fiddlery broke your patches.  :-/
 
> Cheers,
>    Peter
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list