Re: [FFmpeg-devel] [PATCH 1/5] avcodec/vaapi_encode: Simplify code with av_clip_int8()

2019-08-30 Thread Michael Niedermayer
On Thu, Aug 29, 2019 at 12:18:03AM +, Li, Zhong wrote:
> > From: ffmpeg-devel  On Behalf Of Michael
> > Niedermayer
> > Sent: Thursday, August 29, 2019 1:27 AM
> > To: FFmpeg development discussions and patches 
> > Subject: [FFmpeg-devel] [PATCH 1/5] avcodec/vaapi_encode: Simplify code with
> > av_clip_int8()
> > 
> > Only build tested
> > 
> > Signed-off-by: Michael Niedermayer 
> > ---
> >  libavcodec/vaapi_encode.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c index
> > 2fb43cf1a4..3be9159d37 100644
> > --- a/libavcodec/vaapi_encode.c
> > +++ b/libavcodec/vaapi_encode.c
> > @@ -480,7 +480,7 @@ static int vaapi_encode_issue(AVCodecContext *avctx,
> >  .width  = roi->right  - roi->left,
> >  .height = roi->bottom - roi->top,
> >  },
> > -.roi_value = av_clip_c(v, INT8_MIN, INT8_MAX),
> > +.roi_value = av_clip_int8(v),
> >  };
> >  }
> > 
> > --
> > 2.23.0
> 
> LGTM

will apply

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus


signature.asc
Description: PGP signature
___
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".

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/vaapi_encode: Simplify code with av_clip_int8()

2019-08-28 Thread Li, Zhong
> From: ffmpeg-devel  On Behalf Of Michael
> Niedermayer
> Sent: Thursday, August 29, 2019 1:27 AM
> To: FFmpeg development discussions and patches 
> Subject: [FFmpeg-devel] [PATCH 1/5] avcodec/vaapi_encode: Simplify code with
> av_clip_int8()
> 
> Only build tested
> 
> Signed-off-by: Michael Niedermayer 
> ---
>  libavcodec/vaapi_encode.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c index
> 2fb43cf1a4..3be9159d37 100644
> --- a/libavcodec/vaapi_encode.c
> +++ b/libavcodec/vaapi_encode.c
> @@ -480,7 +480,7 @@ static int vaapi_encode_issue(AVCodecContext *avctx,
>  .width  = roi->right  - roi->left,
>  .height = roi->bottom - roi->top,
>  },
> -.roi_value = av_clip_c(v, INT8_MIN, INT8_MAX),
> +.roi_value = av_clip_int8(v),
>  };
>  }
> 
> --
> 2.23.0

LGTM
___
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".