On 2/10/2016 9:30 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > --- > libavfilter/vf_nnedi.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/libavfilter/vf_nnedi.c b/libavfilter/vf_nnedi.c > index 9a98ec2..d265fea 100644 > --- a/libavfilter/vf_nnedi.c > +++ b/libavfilter/vf_nnedi.c > @@ -362,7 +362,7 @@ static int32_t process_line0(const uint8_t *tempu, int > width, uint8_t *dstp8, co > tmp /= 32; > dstp[x] = FFMAX(FFMIN(tmp, maximum), minimum); > } else { > - memset(dstp + x, 255, sizeof(uint8_t)); > + dstp[x] = 255; > count++; > } > } > @@ -612,14 +612,10 @@ static void evalfunc_1(NNEDIContext *s, FrameData > *frame_data) > > for (y = ystart; y < ystop; y += 2) { > for (x = 32; x < width - 32; x++) { > - uint32_t pixel = 0; > - uint32_t all_ones = 0; > + uint32_t pixel = dstp[x]; > float mstd[4]; > > - memcpy(&pixel, dstp + x, sizeof(uint8_t)); > - memset(&all_ones, 255, sizeof(uint8_t)); > - > - if (pixel != all_ones)
lol > + if (pixel != 255) Just make it "if (dstp[x] != 255)". No point using a local variable. > continue; > > s->extract((const uint8_t *)(srcpp + x), src_stride, xdia, > ydia, mstd, input); > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel