[Mesa-dev] [PATCH mesa] egl/android: add missing include

Eric Engestrom eric.engestrom at imgtec.com
Thu Aug 24 15:08:25 UTC 2017


On Thursday, 2017-08-24 10:02:29 -0500, Rob Herring wrote:
> On Thu, Aug 24, 2017 at 9:26 AM, Rob Herring <robh at kernel.org> wrote:
> > On Thu, Aug 24, 2017 at 9:22 AM, Eric Engestrom
> > <eric.engestrom at imgtec.com> wrote:
> >> Cc: Rob Herring <robh at kernel.org>
> >> Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
> >> ---
> >> This needs to land before [1], otherwise the latter will break android.
> >>
> >> [1] https://lists.freedesktop.org/archives/mesa-dev/2017-August/167428.html
> >>
> >>  src/egl/drivers/dri2/platform_android.c | 1 +
> >>  1 file changed, 1 insertion(+)
> >
> > For both patches:
> >
> > Reviewed-by: Rob Herring <robh at kernel.org>
> 
> Actually, on further examination I think this isn't needed.

I pushed it a minute before your email: 688d866eca

(I'm waiting on Emil (and maybe others?) to chime in on the khronos
header change before I push it.)

> egl_dri2.h includes system/window.h which in turn includes
> native_window.h. I'll test out the EGL change without this.

Personally, I like the include-what-you-use philosophy [1], so I'd
rather have this include here anyway.

[1] https://github.com/include-what-you-use/include-what-you-use/blob/master/docs/WhyIWYU.md


More information about the mesa-dev mailing list