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

2019-03-28 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Carl Eugen Hoyos > Sent: Thursday, March 28, 2019 8:15 PM > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH] qsv: fix the dangerous macro > definitions > &

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

2019-03-28 Thread Carl Eugen Hoyos
2019-03-28 10:09 GMT+01:00, Li, Zhong : > But I see another case now: > MFX_VERSION_MAJOR > (MAJOR) || MFX_VERSION_MAJOR == (MAJOR) && > MFX_VERSION_MINOR >= (MINOR) > Actually it is equal to: > (MFX_VERSION_MAJOR > (MAJOR) || MFX_VERSION_MAJOR == (MAJOR)) && > MFX_VERSION_MINOR >= (MINOR) No?

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

2019-03-28 Thread Li, Zhong
> 2019-03-27 11:24 GMT+01:00, Zhong Li : > > Signed-off-by: Zhong Li > > --- > > libavcodec/qsv_internal.h | 8 > > libavfilter/qsvvpp.h | 8 > > 2 files changed, 8 insertions(+), 8 deletions(-) > > > > diff --git a/libavcodec/qsv_internal.h b/libavcodec/qsv_internal.h > >

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

2019-03-28 Thread Carl Eugen Hoyos
2019-03-27 11:24 GMT+01:00, Zhong Li : > Signed-off-by: Zhong Li > --- > libavcodec/qsv_internal.h | 8 > libavfilter/qsvvpp.h | 8 > 2 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/qsv_internal.h b/libavcodec/qsv_internal.h > index

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

2019-03-28 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Li, Zhong > Sent: Thursday, March 28, 2019 7:12 PM > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH] qsv: fix the dangerous macro > definitions >

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

2019-03-28 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Carl Eugen Hoyos > Sent: Thursday, March 28, 2019 6:55 PM > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH] qsv: fix the dangerous macro > definitions > &

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

2019-03-28 Thread Carl Eugen Hoyos
2019-03-28 10:09 GMT+01:00, Li, Zhong : >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >> Of Mark Thompson >> Sent: Thursday, March 28, 2019 6:23 AM >> To: ffmpeg-devel@ffmpeg.org >> Subject: Re: [FFmpeg-devel] [PATCH] qsv: fix the

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

2019-03-28 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Mark Thompson > Sent: Thursday, March 28, 2019 6:23 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] qsv: fix the dangerous macro > definitions > > On 27/03/2019 10:24, Zh

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

2019-03-27 Thread Mark Thompson
On 27/03/2019 10:24, Zhong Li wrote: > Signed-off-by: Zhong Li > --- > libavcodec/qsv_internal.h | 8 > libavfilter/qsvvpp.h | 8 > 2 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/qsv_internal.h b/libavcodec/qsv_internal.h > index

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

2019-03-27 Thread Zhong Li
Signed-off-by: Zhong Li --- libavcodec/qsv_internal.h | 8 libavfilter/qsvvpp.h | 8 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/libavcodec/qsv_internal.h b/libavcodec/qsv_internal.h index 394c558883..86a5dbad98 100644 --- a/libavcodec/qsv_internal.h +++