[FFmpeg-devel] [PATCH] avfilter/vf_bilateral: remove useless memcpy

Tao Zhang nowerzt at gmail.com
Wed Oct 30 11:05:38 EET 2019


Paul B Mahol <onemda at gmail.com> 于2019年10月30日周三 下午4:35写道:
>
> Why you think it is useless?
>
> Have you checked checksums matches before and after?
I compared md5sum were the same. Please point it out if I understand wrong.
>
> On 10/30/19, leozhang <leozhang at qiyi.com> wrote:
> > Signed-off-by: leozhang <leozhang at qiyi.com>
> > ---
> >  libavfilter/vf_bilateral.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/libavfilter/vf_bilateral.c b/libavfilter/vf_bilateral.c
> > index 3c9d800..ba3c6e1 100644
> > --- a/libavfilter/vf_bilateral.c
> > +++ b/libavfilter/vf_bilateral.c
> > @@ -277,8 +277,8 @@ static void bilateral_##name(BilateralContext *s, const
> > uint8_t *ssrc, uint8_t *
> >              factor_++;
> >               \
> >          }
> >               \
> >
> >               \
> > -        memcpy(ypy, ycy, sizeof(float) * width);
> >               \
> > -        memcpy(ypf, ycf, sizeof(float) * width);
> >               \
> > +        ypy = ycy;                                          \
> > +        ypf = ycf;                                          \
> >      }
> >               \
> >
> >               \
> >      for (int i = 0; i < height; i++)
> >               \
> > --
> > 1.8.3.1
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> > To unsubscribe, visit link above, or email
> > ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list