[Mesa-dev] [PATCH] include: Add helper header to help trap includes inside extern C.
kallisti5
kallisti5 at unixzen.com
Wed May 13 10:48:01 PDT 2015
On 2015-05-13 10:42, Jason Ekstrand wrote:
> On May 13, 2015 8:01 AM, "kallisti5" <kallisti5 at unixzen.com> wrote:
> src/mesa/drivers/haiku/swrast/SoftwareRast.cpp:28:
> > include/no_extern_c.h:47:1: error: template with C linkage
> >
> > template<class T> class _IncludeInsideExternCNotPortable;
> >
> > Any alternative fix ideas besides me adding a if
> !defined(__HAIKU__) to include/no_extern_c.h?
>
> Please do not do that! The whole point of this patch is to catch
> places where headers are included inside an extern "C" block. This
> patch makes them a compile error instead of being silently wrong.
>
> The correct solution is to fix the haiku platform integration so it
> doesn't do this.
>
> --Jason
Yeah, I had a chat in IRC and determined that I need to add extern "C"
a few places within Mesa instead of doing a bulk extern "C" to the
mesa C headers.
One example is glapi_priv lacks any extern "C" (ifdef C++ wrapped)
I'll begin submitting patches to the ML over the next few days.
Thanks!
-- Alex
More information about the mesa-dev
mailing list