[Mesa-dev] [PATCH] glx: Don't enclose includes inside `extern "C" { }`.
Ian Romanick
idr at freedesktop.org
Wed Apr 27 08:11:58 UTC 2016
On 04/26/2016 08:55 PM, Jose Fonseca wrote:
> Ran `make check` inside src/glx to verify everything compiles and links
> correctly.
>
> https://bugs.freedesktop.org/show_bug.cgi?id=95158
In the future, please use
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95158
It helps various scripts operate correctly.
> ---
> src/glx/dri2_priv.h | 8 ++++++++
> src/glx/glx_error.h | 8 ++++++++
> src/glx/glxclient.h | 9 +++++++++
> src/glx/glxextensions.h | 8 ++++++++
> src/glx/tests/clientinfo_unittest.cpp | 2 --
> src/glx/tests/create_context_unittest.cpp | 2 --
> src/glx/tests/fake_glx_screen.h | 2 --
> src/glx/tests/query_renderer_implementation_unittest.cpp | 2 --
> src/glx/tests/query_renderer_unittest.cpp | 2 --
> 9 files changed, 33 insertions(+), 10 deletions(-)
>
> diff --git a/src/glx/dri2_priv.h b/src/glx/dri2_priv.h
> index e81b1bf..8a64072 100644
> --- a/src/glx/dri2_priv.h
> +++ b/src/glx/dri2_priv.h
> @@ -30,6 +30,10 @@
> * Kristian Høgsberg (krh at redhat.com)
> */
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> struct dri2_screen {
> struct glx_screen base;
>
> @@ -74,3 +78,7 @@ _X_HIDDEN int
> dri2_interop_export_object(struct glx_context *ctx,
> const mesa_glinterop_export_in *in,
> mesa_glinterop_export_out *out);
> +
> +#ifdef __cplusplus
> +}
> +#endif
> diff --git a/src/glx/glx_error.h b/src/glx/glx_error.h
> index eaf6b81..5d39926 100644
> --- a/src/glx/glx_error.h
> +++ b/src/glx/glx_error.h
> @@ -31,9 +31,17 @@
> #include <X11/Xlib.h>
> #include <xcb/xcb.h>
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> void __glXSendError(Display * dpy, int_fast8_t errorCode,
> uint_fast32_t resourceID, uint_fast16_t minorCode,
> bool coreX11error);
>
> _X_HIDDEN void __glXSendErrorForXcb(Display * dpy,
> const xcb_generic_error_t *err);
> +
> +#ifdef __cplusplus
> +}
> +#endif
> diff --git a/src/glx/glxclient.h b/src/glx/glxclient.h
> index 7a24928..141e46a 100644
> --- a/src/glx/glxclient.h
> +++ b/src/glx/glxclient.h
> @@ -57,6 +57,11 @@
> #include "glxextensions.h"
>
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> +
> #define GLX_MAJOR_VERSION 1 /* current version numbers */
> #define GLX_MINOR_VERSION 4
>
> @@ -824,4 +829,8 @@ indirect_create_context_attribs(struct glx_screen *base,
> const uint32_t *attribs,
> unsigned *error);
>
> +#ifdef __cplusplus
> +}
> +#endif
> +
> #endif /* !__GLX_client_h__ */
> diff --git a/src/glx/glxextensions.h b/src/glx/glxextensions.h
> index 906b3fc..743ed97 100644
> --- a/src/glx/glxextensions.h
> +++ b/src/glx/glxextensions.h
> @@ -31,6 +31,10 @@
> #ifndef GLX_GLXEXTENSIONS_H
> #define GLX_GLXEXTENSIONS_H
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> enum
> {
> ARB_create_context_bit = 0,
> @@ -303,4 +307,8 @@ typedef void (*PFNGLXDISABLEEXTENSIONPROC) (const char *name);
> # endif /* __GNUC__ */
> #endif /* GLX_NO_STATIC_EXTENSION_FUNCTIONS */
>
> +#ifdef __cplusplus
> +}
> +#endif
> +
> #endif /* GLX_GLXEXTENSIONS_H */
> diff --git a/src/glx/tests/clientinfo_unittest.cpp b/src/glx/tests/clientinfo_unittest.cpp
> index b56c94b..320e8fc 100644
> --- a/src/glx/tests/clientinfo_unittest.cpp
> +++ b/src/glx/tests/clientinfo_unittest.cpp
> @@ -23,9 +23,7 @@
> #include <gtest/gtest.h>
> #include <string.h>
>
> -extern "C" {
> #include "glxclient.h"
> -}
>
> #include <xcb/glx.h>
>
> diff --git a/src/glx/tests/create_context_unittest.cpp b/src/glx/tests/create_context_unittest.cpp
> index 819b0b2..a259058 100644
> --- a/src/glx/tests/create_context_unittest.cpp
> +++ b/src/glx/tests/create_context_unittest.cpp
> @@ -23,10 +23,8 @@
> #include <gtest/gtest.h>
> #include <string.h>
>
> -extern "C" {
> #include "glxclient.h"
> #include "glx_error.h"
> -}
>
> #include <xcb/glx.h>
> #include "mock_xdisplay.h"
> diff --git a/src/glx/tests/fake_glx_screen.h b/src/glx/tests/fake_glx_screen.h
> index 0249e79..39b250f 100644
> --- a/src/glx/tests/fake_glx_screen.h
> +++ b/src/glx/tests/fake_glx_screen.h
> @@ -20,9 +20,7 @@
> * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> * DEALINGS IN THE SOFTWARE.
> */
> -extern "C" {
> #include "glxclient.h"
> -};
>
> class fake_glx_screen : public glx_screen {
> public:
> diff --git a/src/glx/tests/query_renderer_implementation_unittest.cpp b/src/glx/tests/query_renderer_implementation_unittest.cpp
> index 7f8f301..c3f5cc7 100644
> --- a/src/glx/tests/query_renderer_implementation_unittest.cpp
> +++ b/src/glx/tests/query_renderer_implementation_unittest.cpp
> @@ -26,13 +26,11 @@
>
> #if defined(GLX_DIRECT_RENDERING) && !defined(GLX_USE_APPLEGL)
>
> -extern "C" {
> #include "glxclient.h"
> #include "glx_error.h"
> #include "dri2.h"
> #include "dri_interface.h"
> #include "dri2_priv.h"
> -}
>
> struct attribute_test_vector {
> const char *glx_string;
> diff --git a/src/glx/tests/query_renderer_unittest.cpp b/src/glx/tests/query_renderer_unittest.cpp
> index 2f3c4ef..dc88cc9 100644
> --- a/src/glx/tests/query_renderer_unittest.cpp
> +++ b/src/glx/tests/query_renderer_unittest.cpp
> @@ -24,10 +24,8 @@
> #include <signal.h>
> #include <setjmp.h>
>
> -extern "C" {
> #include "glxclient.h"
> #include "glx_error.h"
> -}
>
> extern bool GetGLXScreenConfigs_called;
> extern struct glx_screen *psc;
>
More information about the mesa-dev
mailing list