[FFmpeg-devel] [PATCH] split out ff_hwaccel_pixfmt_list_420[] over individual codecs.

Michael Niedermayer michaelni at gmx.at
Mon Feb 18 21:43:25 CET 2013


On Mon, Feb 18, 2013 at 09:24:29PM +0100, Michael Niedermayer wrote:
> On Mon, Feb 18, 2013 at 08:15:52AM -0800, Ronald S. Bultje wrote:
> > From: "Ronald S. Bultje" <rsbultje at gmail.com>
> > 
> > Not all hwaccels implement all codecs, so using one single list for
> > multiple such codecs means some codecs will be represented in the list,
> > even though they don't actually handle that codec. Copying specific
> > lists in each codec fixes that.
> > ---
> >  libavcodec/h263.h          |  2 ++
> >  libavcodec/h263dec.c       | 15 +++++++++++++--
> >  libavcodec/h264.c          | 19 ++++++++++++++++++-
> >  libavcodec/mpeg4videodec.c |  2 +-
> >  libavcodec/mpegvideo.c     | 17 -----------------
> >  libavcodec/mpegvideo.h     |  1 -
> >  libavcodec/vc1dec.c        | 18 ++++++++++++++++--
> >  7 files changed, 50 insertions(+), 24 deletions(-)
> > 
> > diff --git a/libavcodec/h263.h b/libavcodec/h263.h
> > index 28d59ec..ccbccbc 100644
> > --- a/libavcodec/h263.h
> > +++ b/libavcodec/h263.h
> > @@ -207,4 +207,6 @@ static inline void memsetw(short *tab, int val, int n)
> >          tab[i] = val;
> >  }
> >  
> > +extern const enum AVPixelFormat h263_hwaccel_pixfmt_list_420[];
> > +
> >  #endif /* AVCODEC_H263_H */
> > diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c
> > index 6b24880..dd56e11 100644
> > --- a/libavcodec/h263dec.c
> > +++ b/libavcodec/h263dec.c
> > @@ -763,6 +763,17 @@ av_log(avctx, AV_LOG_DEBUG, "%"PRId64"\n", rdtsc()-time);
> >      return (ret && (avctx->err_recognition & AV_EF_EXPLODE))?ret:get_consumed_bytes(s, buf_size);
> >  }
> >  
> > +const enum AVPixelFormat h263_hwaccel_pixfmt_list_420[] = {
> 
> missing ff_ prefix

locally fixed, will push after some basic tests

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130218/5e8f4c34/attachment.asc>


More information about the ffmpeg-devel mailing list