On 8/26/2016 3:19 PM, Michael Niedermayer wrote: > IIRC, The spaces are not standard before the #
We use them sometimes when nesting several preprocessor checks, but in this case yes, it should have no spaces. > > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > --- > libavfilter/motion_estimation.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavfilter/motion_estimation.c b/libavfilter/motion_estimation.c > index fa6f49d..0f9ba21 100644 > --- a/libavfilter/motion_estimation.c > +++ b/libavfilter/motion_estimation.c > @@ -262,10 +262,10 @@ uint64_t ff_me_search_ds(AVMotionEstContext *me_ctx, > int x_mb, int y_mb, int *mv > x = mv[0]; > y = mv[1]; > > - #if 1 > +#if 1 > for (i = 0; i < 8; i++) > COST_P_MV(x + dia2[i][0], y + dia2[i][1]); > - #else > +#else > /* this version skips previously examined 3 or 5 locations based on > prev origin */ > if (dir_x <= 0) > COST_P_MV(x - 2, y); > @@ -286,7 +286,7 @@ uint64_t ff_me_search_ds(AVMotionEstContext *me_ctx, int > x_mb, int y_mb, int *mv > > dir_x = mv[0] - x; > dir_y = mv[1] - y; > - #endif > +#endif > > } while (x != mv[0] || y != mv[1]); > > LGTM, assuming the disabled code has a reason to be there. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel