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

[Rafaël Carré]

On Sat, 3 Oct 2009 21:31:53 +0300
"Rémi Denis-Courmont" <remi at remlab.net> wrote:

> 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__ :/

You can test if using those attributes compile/link in configure.ac

-- 
Rafaël Carré
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20091004/92771187/attachment.sig>