# HG changeset patch # User stefano # Date 1282323158 0 # Node ID ebe3b7ce4cca350c74a15f67b73e7c4f0e5761aa # Parent 046c1e5166050c43a0e2c7a6045ca36e41233580 Rename av_fill_image_max_pixstep() to av_fill_image_max_pixsteps(). The plural form is preferred as it is more consistent with the other functions: av_fill_image_linesizes() av_fill_image_pointers() and looks semantically more correct as it fills an array of elements. diff -r 046c1e516605 -r ebe3b7ce4cca avcore.h --- a/avcore.h Fri Aug 20 16:34:41 2010 +0000 +++ b/avcore.h Fri Aug 20 16:52:38 2010 +0000 @@ -27,7 +27,7 @@ #include #define LIBAVCORE_VERSION_MAJOR 0 -#define LIBAVCORE_VERSION_MINOR 5 +#define LIBAVCORE_VERSION_MINOR 6 #define LIBAVCORE_VERSION_MICRO 0 #define LIBAVCORE_VERSION_INT AV_VERSION_INT(LIBAVCORE_VERSION_MAJOR, \ diff -r 046c1e516605 -r ebe3b7ce4cca imgutils.c --- a/imgutils.c Fri Aug 20 16:34:41 2010 +0000 +++ b/imgutils.c Fri Aug 20 16:52:38 2010 +0000 @@ -34,7 +34,7 @@ if (desc->flags & PIX_FMT_BITSTREAM) return (width * (desc->comp[0].step_minus1+1) + 7) >> 3; - av_fill_image_max_pixstep(max_step, max_step_comp, desc); + av_fill_image_max_pixsteps(max_step, max_step_comp, desc); s = (max_step_comp[plane] == 1 || max_step_comp[plane] == 2) ? desc->log2_chroma_w : 0; return max_step[plane] * (((width + (1 << s) - 1)) >> s); } @@ -56,7 +56,7 @@ return 0; } - av_fill_image_max_pixstep(max_step, max_step_comp, desc); + av_fill_image_max_pixsteps(max_step, max_step_comp, desc); for (i = 0; i < 4; i++) { int s = (max_step_comp[i] == 1 || max_step_comp[i] == 2) ? desc->log2_chroma_w : 0; linesizes[i] = max_step[i] * (((width + (1 << s) - 1)) >> s); diff -r 046c1e516605 -r ebe3b7ce4cca imgutils.h --- a/imgutils.h Fri Aug 20 16:34:41 2010 +0000 +++ b/imgutils.h Fri Aug 20 16:52:38 2010 +0000 @@ -36,27 +36,27 @@ * byte of the successive group in the same plane for the same * component. * - * @param max_pixstep an array which is filled with the max pixel step + * @param max_pixsteps an array which is filled with the max pixel step * for each plane. Since a plane may contain different pixel - * components, the computed max_pixstep[plane] is relative to the + * components, the computed max_pixsteps[plane] is relative to the * component in the plane with the max pixel step. - * @param max_pixstep_comp an array which is filled with the component + * @param max_pixstep_comps an array which is filled with the component * for each plane which has the max pixel step. May be NULL. */ -static inline void av_fill_image_max_pixstep(int max_pixstep[4], int max_pixstep_comp[4], - const AVPixFmtDescriptor *pixdesc) +static inline void av_fill_image_max_pixsteps(int max_pixsteps[4], int max_pixstep_comps[4], + const AVPixFmtDescriptor *pixdesc) { int i; - memset(max_pixstep, 0, 4*sizeof(max_pixstep[0])); - if (max_pixstep_comp) - memset(max_pixstep_comp, 0, 4*sizeof(max_pixstep_comp[0])); + memset(max_pixsteps, 0, 4*sizeof(max_pixsteps[0])); + if (max_pixstep_comps) + memset(max_pixstep_comps, 0, 4*sizeof(max_pixstep_comps[0])); for (i = 0; i < 4; i++) { const AVComponentDescriptor *comp = &(pixdesc->comp[i]); - if ((comp->step_minus1+1) > max_pixstep[comp->plane]) { - max_pixstep[comp->plane] = comp->step_minus1+1; - if (max_pixstep_comp) - max_pixstep_comp[comp->plane] = i; + if ((comp->step_minus1+1) > max_pixsteps[comp->plane]) { + max_pixsteps[comp->plane] = comp->step_minus1+1; + if (max_pixstep_comps) + max_pixstep_comps[comp->plane] = i; } } }