Re: [Mesa-dev] [PATCH] Fix invalid extern "C" around header inclusion.

2015-03-06 Thread Jose Fonseca
On 06/03/15 00:11, Mark Janes wrote: Matt Turner writes: On Thu, Mar 5, 2015 at 4:54 AM, Jose Fonseca wrote: Thanks for doing this. It looks great. Reviewed-by: Jose Fonseca Feel free to push my patch or I'll push it after you push this one. Thanks José. I've just pushed it.

Re: [Mesa-dev] [PATCH] Fix invalid extern "C" around header inclusion.

2015-03-05 Thread Mark Janes
Matt Turner writes: > On Thu, Mar 5, 2015 at 4:54 AM, Jose Fonseca wrote: >> Thanks for doing this. It looks great. >> >> Reviewed-by: Jose Fonseca >> >> >> Feel free to push my patch or I'll push it after you push this one. > > Thanks José. I've just pushed it.

Re: [Mesa-dev] [PATCH] Fix invalid extern "C" around header inclusion.

2015-03-05 Thread Matt Turner
On Thu, Mar 5, 2015 at 4:54 AM, Jose Fonseca wrote: > Thanks for doing this. It looks great. > > Reviewed-by: Jose Fonseca > > > Feel free to push my patch or I'll push it after you push this one. Thanks José. I've just pushed it. ___ mesa-dev mailing

Re: [Mesa-dev] [PATCH] Fix invalid extern "C" around header inclusion.

2015-03-05 Thread Jose Fonseca
Thanks for doing this. It looks great. Reviewed-by: Jose Fonseca Feel free to push my patch or I'll push it after you push this one. Jose On 05/03/15 00:37, Mark Janes wrote: System headers may contain C++ declarations, which cannot be given C linkage. For this reason, include statements

[Mesa-dev] [PATCH] Fix invalid extern "C" around header inclusion.

2015-03-04 Thread Mark Janes
System headers may contain C++ declarations, which cannot be given C linkage. For this reason, include statements should never occur inside extern "C". This patch moves the C linkage statements to enclose only the declarations within a single header. --- src/gallium/auxiliary/util/u_math.h