Re: Fail to build SDL-2.0.5, after enabling -std=c99 ...

Daniel Gibson <[email protected]> Tue, 7 Mar 2017 02:09:59 +0100
Newsgroups gmane.comp.lib.sdl
Message-ID <[email protected]>
Hi,

building with -std=3Dc99 (instead of gnu99) doesn't only change the syntax =

that's accepted by the compiler, but also what parts of headers are used.
All (or most?) things in standard headers that are not part of the C99 =

standard, like "GNU extensions" is not enabled (this is the "#define =

_GNU_SOURCE" stuff mentioned in manpages).
IMHO it's not very useful to use -std=3Dc99, especially for code like =

libSDL that needs to do operating-system specific things. Just use =

-std=3Dgnu99 (or even gnu89)..


Cheers,
Daniel

On 07.03.2017 02:01, JIA Pei wrote:
>
> Hi, Ryan:
>
>
> My current solution is to*change -std=3Dc99 to -std=3Dgnu99.*
>
> I'm testing your solution, which seems to be correct !!! However, I'm
> NOW facing some new issues:
>
> SDL2/SDL2-2.0.5/src/render/software/SDL_rotate.c:136:29:error: =91M_PI=92
> undeclared (first use in this function)
>         radangle =3D angle * (M_PI / -180.0); /* reverse the angle because
> our rotations are clockwise */
>
> SDL2/SDL2-2.0.5/src/events/SDL_quit.c:55:22:error: storage size of
> =91action=92 isn=92t known
>     struct sigaction action;
>
>
> I can easily solve the first problem  *'M_PI' hasn't been declared* by
> defining *M_PI *as*:*
>
>     *#ifndef M_PI*
>     *#define M_PI    3.14159265358979323846264338327950288   /* pi */*
>     *#endif*
>
>
>
> However, how to solve the second issue ????
> SDL2/SDL2-2.0.5/src/events/SDL_quit.c:55:22: error: storage size of
> =91action=92 isn=92t known
>     struct sigaction action;
>
>
> Cheers
>
>
> On Mon, Mar 6, 2017 at 2:00 PM, Ryan C. Gordon <[email protected]
> <mailto:[email protected]>> wrote:
>
>     On 3/3/17 5:13 PM, Sik the hedgehog wrote:
>
>         Try -std=3Dgnu99 (if I recall correctly)
>
>         Looks like the problem is using asm instead of a reserved keyword
>         (whoops). -std=3Dc99 will disable any extensions, -std=3Dgnu99 wi=
ll keep
>         them.
>
>
>     So the actual fix should be to change that to __asm__, right?
>
>     --ryan.
>
>
>
>
>     _______________________________________________
>     SDL mailing list
>     [email protected] <mailto:[email protected]>
>     http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org
>     <http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org>
>
>
>
>
> --
>
> Pei JIA, Ph.D.
>
> Email: [email protected] <mailto:[email protected]>
> cell in Canada:    +1 778-863-5816
> cell in China: +86 186-8244-3503
>
> Welcome to Vision Open
> http://www.visionopen.com
>
>
> _______________________________________________
> SDL mailing list
> [email protected]
> http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org
>