On Wed, 27 Apr 2016 13:44:58 +0200
Hendrik Leppkes <[email protected]> wrote:

> On Wed, Apr 27, 2016 at 1:37 PM, Alexandra Hájková
> <[email protected]> wrote:
> > ---
> >  libavutil/attributes.h | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/libavutil/attributes.h b/libavutil/attributes.h
> > index d7f2bb5..1a8d738 100644
> > --- a/libavutil/attributes.h
> > +++ b/libavutil/attributes.h
> > @@ -123,4 +123,10 @@
> >  #    define av_noreturn
> >  #endif
> >
> > +#if AV_GCC_VERSION_AT_LEAST(4,3)
> > +#   define av_unlikely(x) __builtin_expect((x) != 0, 0)
> > +#else
> > +#   define av_unlikely(x) x
> > +#endif
> > +  
> 
> I dont like this functionality. Ever tried to read GNU sources where
> this is used commonly? It gets practically unreadable.

I believe the idea is to use it only in a few key places, where it
actually matters.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to