[FFmpeg-cvslog] r18898 - in trunk: libavcodec/svq1dec.c libavcodec/utils.c libavcodec/vp3.c libavcodec/zmbvenc.c libavfilter/defaults.c libavutil/common.h libpostproc/postprocess.c

conrad subversion
Fri May 22 23:32:13 CEST 2009


Author: conrad
Date: Fri May 22 23:32:13 2009
New Revision: 18898

Log:
Move ALIGN macro to libavutil/common.h and use it in various places

Modified:
   trunk/libavcodec/svq1dec.c
   trunk/libavcodec/utils.c
   trunk/libavcodec/vp3.c
   trunk/libavcodec/zmbvenc.c
   trunk/libavfilter/defaults.c
   trunk/libavutil/common.h
   trunk/libpostproc/postprocess.c

Modified: trunk/libavcodec/svq1dec.c
==============================================================================
--- trunk/libavcodec/svq1dec.c	Fri May 22 21:11:56 2009	(r18897)
+++ trunk/libavcodec/svq1dec.c	Fri May 22 23:32:13 2009	(r18898)
@@ -696,13 +696,13 @@ static int svq1_decode_frame(AVCodecCont
   for (i=0; i < 3; i++) {
     int linesize;
     if (i == 0) {
-      width  = (s->width+15)&~15;
-      height = (s->height+15)&~15;
+      width  = FFALIGN(s->width, 16);
+      height = FFALIGN(s->height, 16);
       linesize= s->linesize;
     } else {
       if(s->flags&CODEC_FLAG_GRAY) break;
-      width  = (s->width/4+15)&~15;
-      height = (s->height/4+15)&~15;
+      width  = FFALIGN(s->width/4, 16);
+      height = FFALIGN(s->height/4, 16);
       linesize= s->uvlinesize;
     }
 

Modified: trunk/libavcodec/utils.c
==============================================================================
--- trunk/libavcodec/utils.c	Fri May 22 21:11:56 2009	(r18897)
+++ trunk/libavcodec/utils.c	Fri May 22 23:32:13 2009	(r18898)
@@ -134,8 +134,6 @@ typedef struct InternalBuffer{
 
 #define INTERNAL_BUFFER_SIZE 32
 
-#define ALIGN(x, a) (((x)+(a)-1)&~((a)-1))
-
 void avcodec_align_dimensions(AVCodecContext *s, int *width, int *height){
     int w_align= 1;
     int h_align= 1;
@@ -193,8 +191,8 @@ void avcodec_align_dimensions(AVCodecCon
         break;
     }
 
-    *width = ALIGN(*width , w_align);
-    *height= ALIGN(*height, h_align);
+    *width = FFALIGN(*width , w_align);
+    *height= FFALIGN(*height, h_align);
     if(s->codec_id == CODEC_ID_H264)
         *height+=2; // some of the optimized chroma MC reads one line too much
 }
@@ -317,7 +315,7 @@ int avcodec_default_get_buffer(AVCodecCo
             if((s->flags&CODEC_FLAG_EMU_EDGE) || !size[2])
                 buf->data[i] = buf->base[i];
             else
-                buf->data[i] = buf->base[i] + ALIGN((buf->linesize[i]*EDGE_WIDTH>>v_shift) + (EDGE_WIDTH>>h_shift), stride_align[i]);
+                buf->data[i] = buf->base[i] + FFALIGN((buf->linesize[i]*EDGE_WIDTH>>v_shift) + (EDGE_WIDTH>>h_shift), stride_align[i]);
         }
         if(size[1] && !size[2])
             ff_set_systematic_pal((uint32_t*)buf->data[1], s->pix_fmt);

Modified: trunk/libavcodec/vp3.c
==============================================================================
--- trunk/libavcodec/vp3.c	Fri May 22 21:11:56 2009	(r18897)
+++ trunk/libavcodec/vp3.c	Fri May 22 23:32:13 2009	(r18898)
@@ -1636,8 +1636,8 @@ static av_cold int vp3_decode_init(AVCod
         s->version = 1;
 
     s->avctx = avctx;
-    s->width = (avctx->width + 15) & 0xFFFFFFF0;
-    s->height = (avctx->height + 15) & 0xFFFFFFF0;
+    s->width = FFALIGN(avctx->width, 16);
+    s->height = FFALIGN(avctx->height, 16);
     avctx->pix_fmt = PIX_FMT_YUV420P;
     avctx->chroma_sample_location = AVCHROMA_LOC_CENTER;
     if(avctx->idct_algo==FF_IDCT_AUTO)

Modified: trunk/libavcodec/zmbvenc.c
==============================================================================
--- trunk/libavcodec/zmbvenc.c	Fri May 22 21:11:56 2009	(r18897)
+++ trunk/libavcodec/zmbvenc.c	Fri May 22 23:32:13 2009	(r18898)
@@ -284,7 +284,7 @@ static av_cold int encode_init(AVCodecCo
         av_log(avctx, AV_LOG_ERROR, "Can't allocate compression buffer.\n");
         return -1;
     }
-    c->pstride = (avctx->width + 15) & ~15;
+    c->pstride = FFALIGN(avctx->width, 16);
     if ((c->prev = av_malloc(c->pstride * avctx->height)) == NULL) {
         av_log(avctx, AV_LOG_ERROR, "Can't allocate picture.\n");
         return -1;

Modified: trunk/libavfilter/defaults.c
==============================================================================
--- trunk/libavfilter/defaults.c	Fri May 22 21:11:56 2009	(r18897)
+++ trunk/libavfilter/defaults.c	Fri May 22 23:32:13 2009	(r18898)
@@ -29,10 +29,6 @@ void avfilter_default_free_video_buffer(
     av_free(pic);
 }
 
-#define ALIGN(a) do{ \
-                     (a) = ((a) + 15) & (~15); \
-                 } while(0);
-
 /* TODO: set the buffer's priv member to a context structure for the whole
  * filter chain.  This will allow for a buffer pool instead of the constant
  * alloc & free cycle currently implemented. */
@@ -56,7 +52,7 @@ AVFilterPicRef *avfilter_default_get_vid
     ff_fill_linesize((AVPicture *)pic, pic->format, ref->w);
 
     for (i=0; i<4;i++)
-        ALIGN(pic->linesize[i]);
+        pic->linesize[i] = FFALIGN(pic->linesize[i], 16);
 
     tempsize = ff_fill_pointer((AVPicture *)pic, NULL, pic->format, ref->h);
     buf = av_malloc(tempsize);

Modified: trunk/libavutil/common.h
==============================================================================
--- trunk/libavutil/common.h	Fri May 22 21:11:56 2009	(r18897)
+++ trunk/libavutil/common.h	Fri May 22 23:32:13 2009	(r18898)
@@ -127,6 +127,7 @@
 
 #define FFSWAP(type,a,b) do{type SWAP_tmp= b; b= a; a= SWAP_tmp;}while(0)
 #define FF_ARRAY_ELEMS(a) (sizeof(a) / sizeof((a)[0]))
+#define FFALIGN(x, a) (((x)+(a)-1)&~((a)-1))
 
 /* misc math functions */
 extern const uint8_t ff_log2_tab[256];

Modified: trunk/libpostproc/postprocess.c
==============================================================================
--- trunk/libpostproc/postprocess.c	Fri May 22 21:11:56 2009	(r18897)
+++ trunk/libpostproc/postprocess.c	Fri May 22 23:32:13 2009	(r18898)
@@ -951,7 +951,7 @@ static const AVClass av_codec_context_cl
 
 pp_context *pp_get_context(int width, int height, int cpuCaps){
     PPContext *c= av_malloc(sizeof(PPContext));
-    int stride= (width+15)&(~15);    //assumed / will realloc if needed
+    int stride= FFALIGN(width, 16);  //assumed / will realloc if needed
     int qpStride= (width+15)/16 + 2; //assumed / will realloc if needed
 
     memset(c, 0, sizeof(PPContext));



More information about the ffmpeg-cvslog mailing list