[FFmpeg-devel] [PATCHv3] Deprecate AFD field and add AFD as side-data

[Michael Niedermayer]

On Sun, Aug 03, 2014 at 10:46:58PM +0200, Michael Niedermayer wrote:
> On Sun, Aug 03, 2014 at 07:24:56PM +0100, Kieran Kunhya wrote:
> > ---
> >  doc/APIchanges            |    4 ++++
> >  libavcodec/avcodec.h      |    5 ++++-
> >  libavcodec/mpeg12dec.c    |   20 +++++++++++++++++++-
> >  libavcodec/version.h      |    5 ++++-
> >  libavfilter/vf_showinfo.c |    3 +++
> >  libavutil/frame.h         |   16 ++++++++++++++++
> >  libavutil/version.h       |    2 +-
> >  7 files changed, 51 insertions(+), 4 deletions(-)
> [...]
> > @@ -1631,6 +1633,18 @@ static int mpeg_field_start(MpegEncContext *s, const uint8_t *buf, int buf_size)
> >              *stereo = s1->stereo3d;
> >              s1->has_stereo3d = 0;
> >          }
> > +
> > +        if (s1->has_afd) {
> > +            AVFrameSideData *sd = av_frame_new_side_data(
> > +                s->current_picture_ptr->f, AV_FRAME_DATA_AFD,
> > +                1);
> > +            if (!sd)
> > +                return AVERROR(ENOMEM);
> > +
> 
> > +            sd->data = s1->afd;
> 
> this should be
> *(uint8_t*)sd->data = s1->afd;

or simpler
*sd->data = s1->afd;

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140803/0e00b0f0/attachment.asc>