[FFmpeg-devel] [PATCH] Remove AVFilterBuffer.w,h fields.

[Michael Niedermayer]

On Sat, Jul 17, 2010 at 12:53:24PM +0200, Stefano Sabatini wrote:
> On date Friday 2010-07-16 01:00:16 -0700, S.N. Hemanth Meenakshisundaram encoded:
> > On 07/16/2010 12:58 AM, S.N. Hemanth Meenakshisundaram wrote:
> >> On 07/15/2010 04:52 AM, S.N. Hemanth Meenakshisundaram wrote:
> >>> On 07/14/2010 07:51 AM, Michael Niedermayer wrote:
> >>>> [...]
> >>>> to elaborate on this, we need patches that apply to svn.
> >>>> you can send a patch series so that patch n depends on patches 0..n-1
> >>>> to be applied before it.
> >>>> but if patch x (x<n) is changed due to discussions all later patches
> >>>> must be rebased on the new code. We dont apply bad patches and then
> >>>> apply fixes on top.
> >>>>
> >>>>    
> >>
> >> [...]
> >>
> >> Am sending the series of patches again with the changes pointed out  
> >> earlier. [...]
> >>
> >> This is for removing redundant fields from common buffer struct.
> >>
> >> Regards,
> >> Hemanth
> >>
> >
> 
> > diff --git a/libavfilter/avfilter.h b/libavfilter/avfilter.h
> > index 96df532..c8db36f 100644
> > --- a/libavfilter/avfilter.h
> > +++ b/libavfilter/avfilter.h
> > @@ -85,7 +85,6 @@ typedef struct AVFilterBuffer
> >       */
> >      void (*free)(struct AVFilterBuffer *pic);
> >  
> > -    int w, h;                  ///< width and height of the allocated buffer
> >  } AVFilterBuffer;
> >  
> >  /**
> > diff --git a/libavfilter/defaults.c b/libavfilter/defaults.c
> > index cef45b2..c0fe725 100644
> > --- a/libavfilter/defaults.c
> > +++ b/libavfilter/defaults.c
> > @@ -40,8 +40,8 @@ AVFilterPicRef *avfilter_default_get_video_buffer(AVFilterLink *link, int perms,
> >      char *buf;
> >  
> >      ref->pic   = pic;
> > -    ref->w     = pic->w = w;
> > -    ref->h     = pic->h = h;
> > +    ref->w     = w;
> > +    ref->h     = h;
> >  
> >      /* make sure the buffer gets read permission or it's useless for output */
> >      ref->perms = perms | AV_PERM_READ;
> 
> This looks fine to me, assuming that we never used w/h and that that
> info is video-related, so shouldn't stay in AVFilterBuffer which is
> generic.
> 
> OK to apply?

if people want the w/h fields removed then i have no objections

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100717/eb0a347f/attachment.pgp>