[FFmpeg-cvslog] r24827 - in trunk: libavcore/avcore.h libavcore/imgutils.c libavcore/imgutils.h libavfilter/vf_crop.c libavfilter/vf_hflip.c

Stefano Sabatini stefano.sabatini-lala
Thu Aug 19 10:30:14 CEST 2010


On date Wednesday 2010-08-18 23:02:38 +0200, stefano wrote:
> Author: stefano
> Date: Wed Aug 18 23:02:38 2010
> New Revision: 24827
> 
> Log:
> Implement inline function av_fill_image_max_pixstep() and use it for
> factorizing code.

If possible I'd like to change the name of the function, from
av_fill_image_max_pixstep() to av_get_image_max_pixstep().

"fill" sounds like a setter so it was a bad choice, "get" is also
shorter, and is also consistent with av_get_image_linesize().

It is OK just do it, or do I have to add a backward-compatibility
layer? Since there is no risk of breaking apps then I'd just bump
micro and consider it like an API bugfix.

Regards.



More information about the ffmpeg-cvslog mailing list