Re: [RFA/commit] include alloca.h if available.

Joel Brobecker
2010-09-01T01:59:59Z

> > On LynxOS, alloca is defined in allocal.h. This fixes one warning that
> > occurs when building GDBserver for LynxOS.
>
> No objections here. One note.

Cool - checked in.

> Every include of malloc.h in gdbserver's common code:
>
> $ grep malloc\\.h *.c
> gdbreplay.c:#include <malloc.h>
> mem-break.c:#include <malloc.h>
> server.c:#include <malloc.h>
> tracepoint.c:#include <malloc.h>
> utils.c:#include <malloc.h>
> win32-low.c:#include <malloc.h>
>
> is there for alloca on mingw32. It might have made
> sense to keep those together. I don't really care that
> much either way, just pointing it out. :-)

If it is just for malloc, it sounds like the include is no longer
necessary? Unless of course malloc.h is the only location where
alloca is defined on mingw32. Either way, since I'm working in
that area of the code, might as well look at it - I think I'm
setup to build the gdbserver on MinGW.

--
Joel