[SCM] libav/experimental: Rename av_fill_image_linesizes() internal variables max_plane_step and max_plane_step_comp by removing the "plane_" word, and add a comment for explaining what they represent.

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 17:13:02 UTC 2013


The following commit has been merged in the experimental branch:
commit 7860b436514cf04eccf7f85e11a11323b17d88e7
Author: Stefano Sabatini <stefano.sabatini-lala at poste.it>
Date:   Fri Jul 30 08:42:12 2010 +0000

    Rename av_fill_image_linesizes() internal variables max_plane_step and
    max_plane_step_comp by removing the "plane_" word, and add a comment
    for explaining what they represent.
    
    Increase readability.
    
    Originally committed as revision 24588 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavcore/imgutils.c b/libavcore/imgutils.c
index e7e016c..28e063e 100644
--- a/libavcore/imgutils.c
+++ b/libavcore/imgutils.c
@@ -28,8 +28,8 @@ int av_fill_image_linesizes(int linesize[4], enum PixelFormat pix_fmt, int width
 {
     int i;
     const AVPixFmtDescriptor *desc = &av_pix_fmt_descriptors[pix_fmt];
-    int max_plane_step     [4];
-    int max_plane_step_comp[4];
+    int max_step     [4];       /* max pixel step for each plane */
+    int max_step_comp[4];       /* the component for each plane which has the max pixel step */
 
     memset(linesize, 0, 4*sizeof(linesize[0]));
 
@@ -41,19 +41,19 @@ int av_fill_image_linesizes(int linesize[4], enum PixelFormat pix_fmt, int width
         return 0;
     }
 
-    memset(max_plane_step     , 0, sizeof(max_plane_step     ));
-    memset(max_plane_step_comp, 0, sizeof(max_plane_step_comp));
+    memset(max_step     , 0, sizeof(max_step     ));
+    memset(max_step_comp, 0, sizeof(max_step_comp));
     for (i = 0; i < 4; i++) {
         const AVComponentDescriptor *comp = &(desc->comp[i]);
-        if ((comp->step_minus1+1) > max_plane_step[comp->plane]) {
-            max_plane_step     [comp->plane] = comp->step_minus1+1;
-            max_plane_step_comp[comp->plane] = i;
+        if ((comp->step_minus1+1) > max_step[comp->plane]) {
+            max_step     [comp->plane] = comp->step_minus1+1;
+            max_step_comp[comp->plane] = i;
         }
     }
 
     for (i = 0; i < 4; i++) {
-        int s = (max_plane_step_comp[i] == 1 || max_plane_step_comp[i] == 2) ? desc->log2_chroma_w : 0;
-        linesize[i] = max_plane_step[i] * (((width + (1 << s) - 1)) >> s);
+        int s = (max_step_comp[i] == 1 || max_step_comp[i] == 2) ? desc->log2_chroma_w : 0;
+        linesize[i] = max_step[i] * (((width + (1 << s) - 1)) >> s);
     }
 
     return 0;

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list