[MPlayer-dev-eng] [PATCH]Include stdlib from sdl_common.c
Alexander Strasser
eclipse7 at gmx.net
Wed Jul 25 20:35:33 CEST 2012
Hi Carl Eugen,
Carl Eugen Hoyos wrote:
> Alexander Strasser <eclipse7 <at> gmx.net> writes:
>
> > I could not really test it as I guess things get pulled in from
> > somewhere else on my system. But maybe attached patch is what you
> > meant. I still need to double-check stdlib.h is not really needed
> > for somthing else in those files.
>
> On my system, stdlib.h is included by SDL.h but afaict there is no
> guarantee (and a user report that the include is missing on an
> ancient system).
I think my patch would still be correct. I must check if we can
guarantee that NULL is defined though, else maybe we could just
leave out the deletions of the "#include <stdlib.h>" lines.
Alexander
More information about the MPlayer-dev-eng
mailing list