2019-03-28 14:40 GMT+01:00, Zhong Li <zhong...@intel.com>: > Signed-off-by: Zhong Li <zhong...@intel.com> > Reviewed-by: Carl Eugen Hoyos <ceffm...@gmail.com> > --- > libavcodec/qsv_internal.h | 4 ++-- > libavfilter/qsvvpp.h | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/qsv_internal.h b/libavcodec/qsv_internal.h > index 394c558883..57d861d2be 100644 > --- a/libavcodec/qsv_internal.h > +++ b/libavcodec/qsv_internal.h > @@ -39,8 +39,8 @@ > MFX_VERSION_MAJOR == (MAJOR) && MFX_VERSION_MINOR >= (MINOR)) > > #define QSV_RUNTIME_VERSION_ATLEAST(MFX_VERSION, MAJOR, MINOR) \ > - (MFX_VERSION.Major > (MAJOR)) || \ > - (MFX_VERSION.Major == (MAJOR) && MFX_VERSION.Minor >= (MINOR)) > + ((MFX_VERSION.Major > (MAJOR)) || \ > + (MFX_VERSION.Major == (MAJOR) && MFX_VERSION.Minor >= (MINOR)))
I wonder if this would be more readable without the parenthesis around ">" and "==" but since you are the maintainer this patch should be ok. Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
