[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