[MPlayer-dev-eng] [PATCH]Include stdlib from sdl_common.c

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Jul 25 21:12:24 CEST 2012


On Wed, Jul 25, 2012 at 12:39:43AM +0200, Alexander Strasser wrote:
> Hi all,
> 
> Reimar Döffinger wrote:
> > On 24 Jul 2012, at 09:35, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> > > sdl_common.c uses setenv() which needs stdlib.h.
> > 
> > I believe that include belongs into osdep/setenv.h (not sure whether unconditionally or under if is better).
> 
>   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. 

If it works for fixing the issue and you double-check that the stdlib
removal is correct (or just don't remove them) that should be fine.


More information about the MPlayer-dev-eng mailing list