changeset 10141:7d6a854a5fe5

cleanup, use vf->dmpi rather than vf->priv->dmpi for consistency
author rfelker
date Tue, 20 May 2003 18:36:55 +0000
parents 30cad6ad9dbc
children 1031f03eec7e
files libmpcodecs/vf_field.c libmpcodecs/vf_flip.c libmpcodecs/vf_noise.c libmpcodecs/vf_pp.c libmpcodecs/vf_unsharp.c
diffstat 5 files changed, 63 insertions(+), 73 deletions(-) [+]
line wrap: on
line diff
--- a/libmpcodecs/vf_field.c	Tue May 20 17:42:33 2003 +0000
+++ b/libmpcodecs/vf_field.c	Tue May 20 18:36:55 2003 +0000
@@ -12,7 +12,6 @@
 
 struct vf_priv_s {
     int field;
-    mp_image_t *dmpi;
 };
 
 //===========================================================================//
@@ -29,24 +28,24 @@
 	return vf_next_put_image(vf,(mp_image_t*)mpi->priv);
     }
 
-    vf->priv->dmpi=vf_get_image(vf->next,mpi->imgfmt,
+    vf->dmpi=vf_get_image(vf->next,mpi->imgfmt,
 	MP_IMGTYPE_EXPORT, MP_IMGFLAG_ACCEPT_STRIDE,
 	mpi->width, mpi->height/2);
     
     // set up mpi as a double-stride image of dmpi:
-    vf->priv->dmpi->planes[0]=mpi->planes[0]+mpi->stride[0]*vf->priv->field;
-    vf->priv->dmpi->stride[0]=2*mpi->stride[0];
-    if(vf->priv->dmpi->flags&MP_IMGFLAG_PLANAR){
-        vf->priv->dmpi->planes[1]=mpi->planes[1]+
+    vf->dmpi->planes[0]=mpi->planes[0]+mpi->stride[0]*vf->priv->field;
+    vf->dmpi->stride[0]=2*mpi->stride[0];
+    if(vf->dmpi->flags&MP_IMGFLAG_PLANAR){
+        vf->dmpi->planes[1]=mpi->planes[1]+
 	    mpi->stride[1]*vf->priv->field;
-	vf->priv->dmpi->stride[1]=2*mpi->stride[1];
-        vf->priv->dmpi->planes[2]=mpi->planes[2]+
+	vf->dmpi->stride[1]=2*mpi->stride[1];
+        vf->dmpi->planes[2]=mpi->planes[2]+
 	    mpi->stride[2]*vf->priv->field;
-	vf->priv->dmpi->stride[2]=2*mpi->stride[2];
+	vf->dmpi->stride[2]=2*mpi->stride[2];
     } else
-	vf->priv->dmpi->planes[1]=mpi->planes[1]; // passthru bgr8 palette!!!
+	vf->dmpi->planes[1]=mpi->planes[1]; // passthru bgr8 palette!!!
     
-    return vf_next_put_image(vf,vf->priv->dmpi);
+    return vf_next_put_image(vf,vf->dmpi);
 }
 
 //===========================================================================//
--- a/libmpcodecs/vf_flip.c	Tue May 20 17:42:33 2003 +0000
+++ b/libmpcodecs/vf_flip.c	Tue May 20 18:36:55 2003 +0000
@@ -8,11 +8,6 @@
 #include "mp_image.h"
 #include "vf.h"
 
-#include "../libvo/fastmemcpy.h"
-
-struct vf_priv_s {
-    mp_image_t *dmpi;
-};
 
 //===========================================================================//
 
@@ -26,22 +21,22 @@
 static void get_image(struct vf_instance_s* vf, mp_image_t *mpi){
     if(mpi->flags&MP_IMGFLAG_ACCEPT_STRIDE){
 	// try full DR !
-	vf->priv->dmpi=vf_get_image(vf->next,mpi->imgfmt,
+	vf->dmpi=vf_get_image(vf->next,mpi->imgfmt,
 	    mpi->type, mpi->flags, mpi->width, mpi->height);
 	// set up mpi as a upside-down image of dmpi:
-	mpi->planes[0]=vf->priv->dmpi->planes[0]+
-		    vf->priv->dmpi->stride[0]*(vf->priv->dmpi->height-1);
-	mpi->stride[0]=-vf->priv->dmpi->stride[0];
+	mpi->planes[0]=vf->dmpi->planes[0]+
+		    vf->dmpi->stride[0]*(vf->dmpi->height-1);
+	mpi->stride[0]=-vf->dmpi->stride[0];
 	if(mpi->flags&MP_IMGFLAG_PLANAR){
-	    mpi->planes[1]=vf->priv->dmpi->planes[1]+
-		    vf->priv->dmpi->stride[1]*((vf->priv->dmpi->height>>mpi->chroma_y_shift)-1);
-	    mpi->stride[1]=-vf->priv->dmpi->stride[1];
-	    mpi->planes[2]=vf->priv->dmpi->planes[2]+
-		    vf->priv->dmpi->stride[2]*((vf->priv->dmpi->height>>mpi->chroma_y_shift)-1);
-	    mpi->stride[2]=-vf->priv->dmpi->stride[2];
+	    mpi->planes[1]=vf->dmpi->planes[1]+
+		    vf->dmpi->stride[1]*((vf->dmpi->height>>mpi->chroma_y_shift)-1);
+	    mpi->stride[1]=-vf->dmpi->stride[1];
+	    mpi->planes[2]=vf->dmpi->planes[2]+
+		    vf->dmpi->stride[2]*((vf->dmpi->height>>mpi->chroma_y_shift)-1);
+	    mpi->stride[2]=-vf->dmpi->stride[2];
 	}
 	mpi->flags|=MP_IMGFLAG_DIRECT;
-	mpi->priv=(void*)vf->priv->dmpi;
+	mpi->priv=(void*)vf->dmpi;
     }
 }
 
@@ -53,25 +48,25 @@
 	return vf_next_put_image(vf,(mp_image_t*)mpi->priv);
     }
 
-    vf->priv->dmpi=vf_get_image(vf->next,mpi->imgfmt,
+    vf->dmpi=vf_get_image(vf->next,mpi->imgfmt,
 	MP_IMGTYPE_EXPORT, MP_IMGFLAG_ACCEPT_STRIDE,
 	mpi->width, mpi->height);
     
     // set up mpi as a upside-down image of dmpi:
-    vf->priv->dmpi->planes[0]=mpi->planes[0]+
+    vf->dmpi->planes[0]=mpi->planes[0]+
 		    mpi->stride[0]*(mpi->height-1);
-    vf->priv->dmpi->stride[0]=-mpi->stride[0];
-    if(vf->priv->dmpi->flags&MP_IMGFLAG_PLANAR){
-        vf->priv->dmpi->planes[1]=mpi->planes[1]+
+    vf->dmpi->stride[0]=-mpi->stride[0];
+    if(vf->dmpi->flags&MP_IMGFLAG_PLANAR){
+        vf->dmpi->planes[1]=mpi->planes[1]+
 	    mpi->stride[1]*((mpi->height>>mpi->chroma_y_shift)-1);
-	vf->priv->dmpi->stride[1]=-mpi->stride[1];
-	vf->priv->dmpi->planes[2]=mpi->planes[2]+
+	vf->dmpi->stride[1]=-mpi->stride[1];
+	vf->dmpi->planes[2]=mpi->planes[2]+
 	    mpi->stride[2]*((mpi->height>>mpi->chroma_y_shift)-1);
-	vf->priv->dmpi->stride[2]=-mpi->stride[2];
+	vf->dmpi->stride[2]=-mpi->stride[2];
     } else
-	vf->priv->dmpi->planes[1]=mpi->planes[1]; // passthru bgr8 palette!!!
+	vf->dmpi->planes[1]=mpi->planes[1]; // passthru bgr8 palette!!!
     
-    return vf_next_put_image(vf,vf->priv->dmpi);
+    return vf_next_put_image(vf,vf->dmpi);
 }
 
 //===========================================================================//
@@ -81,7 +76,6 @@
     vf->get_image=get_image;
     vf->put_image=put_image;
     vf->default_reqs=VFCAP_ACCEPT_STRIDE;
-    vf->priv=malloc(sizeof(struct vf_priv_s));
     return 1;
 }
 
--- a/libmpcodecs/vf_noise.c	Tue May 20 17:42:33 2003 +0000
+++ b/libmpcodecs/vf_noise.c	Tue May 20 18:36:55 2003 +0000
@@ -62,7 +62,6 @@
 struct vf_priv_s {
 	FilterParam lumaParam;
 	FilterParam chromaParam;
-	mp_image_t *dmpi;
 	unsigned int outfmt;
 };
 
@@ -320,16 +319,16 @@
     if(mpi->flags&MP_IMGFLAG_PRESERVE) return; // don't change
     if(mpi->imgfmt!=vf->priv->outfmt) return; // colorspace differ
     // ok, we can do pp in-place (or pp disabled):
-    vf->priv->dmpi=vf_get_image(vf->next,mpi->imgfmt,
+    vf->dmpi=vf_get_image(vf->next,mpi->imgfmt,
         mpi->type, mpi->flags, mpi->w, mpi->h);
-    mpi->planes[0]=vf->priv->dmpi->planes[0];
-    mpi->stride[0]=vf->priv->dmpi->stride[0];
-    mpi->width=vf->priv->dmpi->width;
+    mpi->planes[0]=vf->dmpi->planes[0];
+    mpi->stride[0]=vf->dmpi->stride[0];
+    mpi->width=vf->dmpi->width;
     if(mpi->flags&MP_IMGFLAG_PLANAR){
-        mpi->planes[1]=vf->priv->dmpi->planes[1];
-        mpi->planes[2]=vf->priv->dmpi->planes[2];
-	mpi->stride[1]=vf->priv->dmpi->stride[1];
-	mpi->stride[2]=vf->priv->dmpi->stride[2];
+        mpi->planes[1]=vf->dmpi->planes[1];
+        mpi->planes[2]=vf->dmpi->planes[2];
+	mpi->stride[1]=vf->dmpi->stride[1];
+	mpi->stride[2]=vf->dmpi->stride[2];
     }
     mpi->flags|=MP_IMGFLAG_DIRECT;
 }
@@ -339,13 +338,13 @@
 
 	if(!(mpi->flags&MP_IMGFLAG_DIRECT)){
 		// no DR, so get a new image! hope we'll get DR buffer:
-		vf->priv->dmpi=vf_get_image(vf->next,vf->priv->outfmt,
+		vf->dmpi=vf_get_image(vf->next,vf->priv->outfmt,
 		MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE,
 		mpi->w,mpi->h);
 //printf("nodr\n");
 	}
 //else printf("dr\n");
-	dmpi= vf->priv->dmpi;
+	dmpi= vf->dmpi;
 
 	noise(dmpi->planes[0], mpi->planes[0], dmpi->stride[0], mpi->stride[0], mpi->w, mpi->h, &vf->priv->lumaParam);
 	noise(dmpi->planes[1], mpi->planes[1], dmpi->stride[1], mpi->stride[1], mpi->w/2, mpi->h/2, &vf->priv->chromaParam);
--- a/libmpcodecs/vf_pp.c	Tue May 20 17:42:33 2003 +0000
+++ b/libmpcodecs/vf_pp.c	Tue May 20 18:36:55 2003 +0000
@@ -30,7 +30,6 @@
     int pp;
     pp_mode_t *ppMode[PP_QUALITY_MAX+1];
     void *context;
-    mp_image_t *dmpi;
     unsigned int outfmt;
 };
 
@@ -97,16 +96,16 @@
     if(!(mpi->flags&MP_IMGFLAG_ACCEPT_STRIDE) && mpi->imgfmt!=vf->priv->outfmt)
 	return; // colorspace differ
     // ok, we can do pp in-place (or pp disabled):
-    vf->priv->dmpi=vf_get_image(vf->next,mpi->imgfmt,
+    vf->dmpi=vf_get_image(vf->next,mpi->imgfmt,
         mpi->type, mpi->flags, mpi->w, mpi->h);
-    mpi->planes[0]=vf->priv->dmpi->planes[0];
-    mpi->stride[0]=vf->priv->dmpi->stride[0];
-    mpi->width=vf->priv->dmpi->width;
+    mpi->planes[0]=vf->dmpi->planes[0];
+    mpi->stride[0]=vf->dmpi->stride[0];
+    mpi->width=vf->dmpi->width;
     if(mpi->flags&MP_IMGFLAG_PLANAR){
-        mpi->planes[1]=vf->priv->dmpi->planes[1];
-        mpi->planes[2]=vf->priv->dmpi->planes[2];
-	mpi->stride[1]=vf->priv->dmpi->stride[1];
-	mpi->stride[2]=vf->priv->dmpi->stride[2];
+        mpi->planes[1]=vf->dmpi->planes[1];
+        mpi->planes[2]=vf->dmpi->planes[2];
+	mpi->stride[1]=vf->dmpi->stride[1];
+	mpi->stride[2]=vf->dmpi->stride[2];
     }
     mpi->flags|=MP_IMGFLAG_DIRECT;
 }
@@ -114,18 +113,18 @@
 static int put_image(struct vf_instance_s* vf, mp_image_t *mpi){
     if(!(mpi->flags&MP_IMGFLAG_DIRECT)){
 	// no DR, so get a new image! hope we'll get DR buffer:
-	vf->priv->dmpi=vf_get_image(vf->next,mpi->imgfmt,
+	vf->dmpi=vf_get_image(vf->next,mpi->imgfmt,
 	    MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE|MP_IMGFLAG_PREFER_ALIGNED_STRIDE,
 //	    MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE,
 //	    mpi->w,mpi->h);
 	    (mpi->w+7)&(~7),(mpi->h+7)&(~7));
-	vf->priv->dmpi->w=mpi->w; vf->priv->dmpi->h=mpi->h; // display w;h
+	vf->dmpi->w=mpi->w; vf->dmpi->h=mpi->h; // display w;h
     }
     
     if(vf->priv->pp || !(mpi->flags&MP_IMGFLAG_DIRECT)){
 	// do the postprocessing! (or copy if no DR)
 	pp_postprocess(mpi->planes           ,mpi->stride,
-		    vf->priv->dmpi->planes,vf->priv->dmpi->stride,
+		    vf->dmpi->planes,vf->dmpi->stride,
 		    (mpi->w+7)&(~7),mpi->h,
 		    mpi->qscale, mpi->qstride,
 		    vf->priv->ppMode[ vf->priv->pp ], vf->priv->context,
@@ -135,7 +134,7 @@
 		    mpi->pict_type);
 #endif
     }
-    return vf_next_put_image(vf,vf->priv->dmpi);
+    return vf_next_put_image(vf,vf->dmpi);
 }
 
 //===========================================================================//
--- a/libmpcodecs/vf_unsharp.c	Tue May 20 17:42:33 2003 +0000
+++ b/libmpcodecs/vf_unsharp.c	Tue May 20 18:36:55 2003 +0000
@@ -61,7 +61,6 @@
 struct vf_priv_s {
     FilterParam lumaParam;
     FilterParam chromaParam;
-    mp_image_t *dmpi;
     unsigned int outfmt;
 };
 
@@ -175,15 +174,15 @@
     if( mpi->imgfmt!=vf->priv->outfmt )
 	return; // colorspace differ
 
-    vf->priv->dmpi = vf_get_image( vf->next, mpi->imgfmt, mpi->type, mpi->flags, mpi->w, mpi->h );
-    mpi->planes[0] = vf->priv->dmpi->planes[0];
-    mpi->stride[0] = vf->priv->dmpi->stride[0];
-    mpi->width = vf->priv->dmpi->width;
+    vf->dmpi = vf_get_image( vf->next, mpi->imgfmt, mpi->type, mpi->flags, mpi->w, mpi->h );
+    mpi->planes[0] = vf->dmpi->planes[0];
+    mpi->stride[0] = vf->dmpi->stride[0];
+    mpi->width = vf->dmpi->width;
     if( mpi->flags & MP_IMGFLAG_PLANAR ) {
-        mpi->planes[1] = vf->priv->dmpi->planes[1];
-        mpi->planes[2] = vf->priv->dmpi->planes[2];
-	mpi->stride[1] = vf->priv->dmpi->stride[1];
-	mpi->stride[2] = vf->priv->dmpi->stride[2];
+        mpi->planes[1] = vf->dmpi->planes[1];
+        mpi->planes[2] = vf->dmpi->planes[2];
+	mpi->stride[1] = vf->dmpi->stride[1];
+	mpi->stride[2] = vf->dmpi->stride[2];
     }
     mpi->flags |= MP_IMGFLAG_DIRECT;
 }
@@ -193,8 +192,8 @@
 
     if( !(mpi->flags & MP_IMGFLAG_DIRECT) )
 	// no DR, so get a new image! hope we'll get DR buffer:
-	vf->priv->dmpi = vf_get_image( vf->next,vf->priv->outfmt, MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE, mpi->w, mpi->h);
-    dmpi= vf->priv->dmpi;
+	vf->dmpi = vf_get_image( vf->next,vf->priv->outfmt, MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE, mpi->w, mpi->h);
+    dmpi= vf->dmpi;
     
     unsharp( dmpi->planes[0], mpi->planes[0], dmpi->stride[0], mpi->stride[0], mpi->w,   mpi->h,   &vf->priv->lumaParam );
     unsharp( dmpi->planes[1], mpi->planes[1], dmpi->stride[1], mpi->stride[1], mpi->w/2, mpi->h/2, &vf->priv->chromaParam );