[FFmpeg-devel] [PATCH v2] qsv: fix the dangerous macro definitions

[Li, Zhong]

> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf
> Of Carl Eugen Hoyos
> Sent: Thursday, March 28, 2019 8:03 PM
> To: FFmpeg development discussions and patches
> <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH v2] qsv: fix the dangerous macro
> definitions
> 
> 2019-03-28 10:16 GMT+01:00, Zhong Li <zhong.li at intel.com>:
> > Signed-off-by: Zhong Li <zhong.li at intel.com>
> > ---
> >  libavcodec/qsv_internal.h | 6 +++---
> >  libavfilter/qsvvpp.h      | 6 +++---
> >  2 files changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/libavcodec/qsv_internal.h b/libavcodec/qsv_internal.h
> > index 394c558883..f9bfc07c36 100644
> > --- a/libavcodec/qsv_internal.h
> > +++ b/libavcodec/qsv_internal.h
> > @@ -36,11 +36,11 @@
> >
> >  #define QSV_VERSION_ATLEAST(MAJOR, MINOR)   \
> >      (MFX_VERSION_MAJOR > (MAJOR) ||         \
> > -     MFX_VERSION_MAJOR == (MAJOR) && MFX_VERSION_MINOR >=
> (MINOR))
> > +    (MFX_VERSION_MAJOR == (MAJOR) && MFX_VERSION_MINOR >=
> (MINOR)))
> 
> Why is this needed?

See my comment in http://ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241792.html