Bug 98691

Summary: egl.h:55: error: redefinition of typedef ‘EGLDisplay’
Product: Mesa Reporter: Vinson Lee <vlee>
Component: Mesa coreAssignee: mesa-dev
Status: RESOLVED FIXED QA Contact: mesa-dev
Severity: normal    
Priority: medium CC: emil.l.velikov, jano.vilicic
Version: unspecified   
Hardware: x86-64 (AMD64)   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Vinson Lee 2016-11-11 22:49:59 UTC
mesa: 0b98e885e707a679fffb096aa802cfe4a80ce334 (master 13.1.0-devel)

  CC     main/eglapi.lo
In file included from main/egltypedefs.h:34,
                 from main/eglglobals.h:37,
                 from main/eglapi.c:94:
../../include/EGL/egl.h:55: error: redefinition of typedef ‘EGLDisplay’
../../include/GL/mesa_glinterop.h:65: note: previous declaration of ‘EGLDisplay’ was here
Comment 1 Emil Velikov 2016-11-14 09:58:41 UTC
I feared this might happen :-\

Vinson, can you please try a fix similar to c85b34ffd04f9a7a16fe30173474e857d0f42d5f
Comment 2 Vinson Lee 2017-08-30 04:57:36 UTC
commit 4e97084591c206338af4425c33edb833de348816
Author: Matt Turner <mattst88@gmail.com>
Date:   Thu Jul 6 18:40:53 2017 -0700

    egl: Fix inclusion of egl.h+mesa_glinterop.h
    
    Previously clang would warn about redefinition of typedef EGLDisplay. Avoid
    this by adding preprocessor guards to mesa_glinterop.h and including it
    after EGL.h is indirectly included.
    
    Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.