Merge commit 'e3e317e0c015b164b6c2eb8913e393216d78de23'
authorMichael Niedermayer <michaelni@gmx.at>
Sun, 9 Nov 2014 23:57:41 +0000 (00:57 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Sun, 9 Nov 2014 23:57:41 +0000 (00:57 +0100)
* commit 'e3e317e0c015b164b6c2eb8913e393216d78de23':
  lavc: Compact the side-data passthrough

Conflicts:
libavcodec/utils.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/utils.c

@@@ -732,70 -529,93 +732,60 @@@ FF_ENABLE_DEPRECATION_WARNING
      }
  }
  
 -#if FF_API_GET_BUFFER
 -FF_DISABLE_DEPRECATION_WARNINGS
 -int avcodec_default_get_buffer(AVCodecContext *avctx, AVFrame *frame)
 -{
 -    return avcodec_default_get_buffer2(avctx, frame, 0);
 -}
 -
 -typedef struct CompatReleaseBufPriv {
 -    AVCodecContext avctx;
 -    AVFrame frame;
 -} CompatReleaseBufPriv;
 -
 -static void compat_free_buffer(void *opaque, uint8_t *data)
 -{
 -    CompatReleaseBufPriv *priv = opaque;
 -    if (priv->avctx.release_buffer)
 -        priv->avctx.release_buffer(&priv->avctx, &priv->frame);
 -    av_freep(&priv);
 -}
 -
 -static void compat_release_buffer(void *opaque, uint8_t *data)
 -{
 -    AVBufferRef *buf = opaque;
 -    av_buffer_unref(&buf);
 -}
 -FF_ENABLE_DEPRECATION_WARNINGS
 -#endif
 -
 -int ff_decode_frame_props(AVCodecContext *avctx, AVFrame *frame)
 +int ff_init_buffer_info(AVCodecContext *avctx, AVFrame *frame)
  {
      AVPacket *pkt = avctx->internal->pkt;
+     int i;
+     struct {
+         enum AVPacketSideDataType packet;
+         enum AVFrameSideDataType frame;
+     } sd[] = {
+         { AV_PKT_DATA_REPLAYGAIN ,   AV_FRAME_DATA_REPLAYGAIN },
+         { AV_PKT_DATA_DISPLAYMATRIX, AV_FRAME_DATA_DISPLAYMATRIX },
+         { AV_PKT_DATA_STEREO3D,      AV_FRAME_DATA_STEREO3D },
+     };
  
 -    frame->color_primaries = avctx->color_primaries;
 -    frame->color_trc       = avctx->color_trc;
 -    frame->colorspace      = avctx->colorspace;
 -    frame->color_range     = avctx->color_range;
 -    frame->chroma_location = avctx->chroma_sample_location;
 -
 -    frame->reordered_opaque = avctx->reordered_opaque;
 -    if (!pkt) {
 -        frame->pkt_pts = AV_NOPTS_VALUE;
 -        return 0;
 -    }
 -
 -    frame->pkt_pts = pkt->pts;
 -
 -    for (i = 0; i < FF_ARRAY_ELEMS(sd); i++) {
 -        int size;
 -        uint8_t *packet_sd = av_packet_get_side_data(pkt, sd[i].packet, &size);
 -        if (packet_sd) {
 -            AVFrameSideData *frame_sd = av_frame_new_side_data(frame,
 -                                                               sd[i].frame,
 -                                                               size);
 -            if (!frame_sd)
 -                return AVERROR(ENOMEM);
 -
 -            memcpy(frame_sd->data, packet_sd, size);
 +    if (pkt) {
-         uint8_t *packet_sd;
-         AVFrameSideData *frame_sd;
-         int size;
 +        frame->pkt_pts = pkt->pts;
 +        av_frame_set_pkt_pos     (frame, pkt->pos);
 +        av_frame_set_pkt_duration(frame, pkt->duration);
 +        av_frame_set_pkt_size    (frame, pkt->size);
 +
-         /* copy the replaygain data to the output frame */
-         packet_sd = av_packet_get_side_data(pkt, AV_PKT_DATA_REPLAYGAIN, &size);
-         if (packet_sd) {
-             frame_sd = av_frame_new_side_data(frame, AV_FRAME_DATA_REPLAYGAIN, size);
-             if (!frame_sd)
-                 return AVERROR(ENOMEM);
-             memcpy(frame_sd->data, packet_sd, size);
-         }
-         /* copy the displaymatrix to the output frame */
-         packet_sd = av_packet_get_side_data(pkt, AV_PKT_DATA_DISPLAYMATRIX, &size);
-         if (packet_sd) {
-             frame_sd = av_frame_new_side_data(frame, AV_FRAME_DATA_DISPLAYMATRIX, size);
-             if (!frame_sd)
-                 return AVERROR(ENOMEM);
-             memcpy(frame_sd->data, packet_sd, size);
-         }
-         /* copy the stereo3d format to the output frame */
-         packet_sd = av_packet_get_side_data(pkt, AV_PKT_DATA_STEREO3D, &size);
-         if (packet_sd) {
-             frame_sd = av_frame_new_side_data(frame, AV_FRAME_DATA_STEREO3D, size);
-             if (!frame_sd)
-                 return AVERROR(ENOMEM);
-             memcpy(frame_sd->data, packet_sd, size);
++        for (i = 0; i < FF_ARRAY_ELEMS(sd); i++) {
++            int size;
++            uint8_t *packet_sd = av_packet_get_side_data(pkt, sd[i].packet, &size);
++            if (packet_sd) {
++                AVFrameSideData *frame_sd = av_frame_new_side_data(frame,
++                                                                   sd[i].frame,
++                                                                   size);
++                if (!frame_sd)
++                    return AVERROR(ENOMEM);
++
++                memcpy(frame_sd->data, packet_sd, size);
++            }
          }
 +    } else {
 +        frame->pkt_pts = AV_NOPTS_VALUE;
 +        av_frame_set_pkt_pos     (frame, -1);
 +        av_frame_set_pkt_duration(frame, 0);
 +        av_frame_set_pkt_size    (frame, -1);
      }
 +    frame->reordered_opaque = avctx->reordered_opaque;
  
 -    return 0;
 -}
 -
 -int ff_get_buffer(AVCodecContext *avctx, AVFrame *frame, int flags)
 -{
 -    const AVHWAccel *hwaccel = avctx->hwaccel;
 -    int override_dimensions = 1;
 -    int ret;
 -
 -    switch (avctx->codec_type) {
 +    if (frame->color_primaries == AVCOL_PRI_UNSPECIFIED)
 +        frame->color_primaries = avctx->color_primaries;
 +    if (frame->color_trc == AVCOL_TRC_UNSPECIFIED)
 +        frame->color_trc = avctx->color_trc;
 +    if (av_frame_get_colorspace(frame) == AVCOL_SPC_UNSPECIFIED)
 +        av_frame_set_colorspace(frame, avctx->colorspace);
 +    if (av_frame_get_color_range(frame) == AVCOL_RANGE_UNSPECIFIED)
 +        av_frame_set_color_range(frame, avctx->color_range);
 +    if (frame->chroma_location == AVCHROMA_LOC_UNSPECIFIED)
 +        frame->chroma_location = avctx->chroma_sample_location;
 +
 +    switch (avctx->codec->type) {
      case AVMEDIA_TYPE_VIDEO:
 -        if (frame->width <= 0 || frame->height <= 0) {
 -            frame->width  = FFMAX(avctx->width, avctx->coded_width);
 -            frame->height = FFMAX(avctx->height, avctx->coded_height);
 -            override_dimensions = 0;
 -        }
 -        if (frame->format < 0)
 -            frame->format              = avctx->pix_fmt;
 +        frame->format              = avctx->pix_fmt;
          if (!frame->sample_aspect_ratio.num)
              frame->sample_aspect_ratio = avctx->sample_aspect_ratio;