[vlc-devel] [PATCH 21/23] HACK: avoid linker warning about error and warning attributes

Rémi Denis-Courmont remi at remlab.net
Sat Oct 3 20:31:53 CEST 2009


Le samedi 3 octobre 2009 21:13:26 Rafaël Carré, vous avez écrit :
> Le 03/10/2009 19:52, Rémi Denis-Courmont a écrit :
> > The Fremantle toolchain does not seem to support those extensions.
> > ---
> >  include/vlc_mtime.h |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/include/vlc_mtime.h b/include/vlc_mtime.h
> > index 49f53ca..f2188dd 100644
> > --- a/include/vlc_mtime.h
> > +++ b/include/vlc_mtime.h
> > @@ -68,7 +68,7 @@ VLC_EXPORT( void,    mwait,    ( mtime_t date ) );
> >  VLC_EXPORT( void,    msleep,   ( mtime_t delay ) );
> >  VLC_EXPORT( char *,  secstotimestr, ( char *psz_buffer, int secs ) );
> >
> > -#if defined (__GNUC__) && defined (__linux__)
> > +#if 0 //defined (__GNUC__) && defined (__linux__)
> >  # define VLC_HARD_MIN_SLEEP 10000 /* Linux has 100, 250, 300 or 1000Hz
> > */ # define VLC_SOFT_MIN_SLEEP 9000000
> 
> I have seen other versions of gcc which don't support them too (although
> compilation didn't fail), we should find the gcc version which started
> supporting them

Maemo 5 uses CodeSoucery gcc 4.2.1 2007q3 for amrv7-a. I am fairly sure that 
Debian gcc 4.2 for x86 supported those parameters. I suspect it is linker 
problem. And we don't know the linker at compile-time. So hmm, I don't know 
what to do, other than add ad if __386__ || __x86_64__ :/

-- 
Rémi Denis-Courmont
http://www.remlab.net/



More information about the vlc-devel mailing list