Merge commit '472f9ed312ec784f7c72876b77163f4741880d05'
authorMichael Niedermayer <michaelni@gmx.at>
Sat, 9 Aug 2014 23:21:30 +0000 (01:21 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Sat, 9 Aug 2014 23:29:24 +0000 (01:29 +0200)
* commit '472f9ed312ec784f7c72876b77163f4741880d05':
  Remove obsolete FF_API_AVFRAME_COLORSPACE cruft.

Conflicts:
libavcodec/utils.c
libavutil/frame.h
libavutil/version.h

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

@@@ -740,62 -541,92 +740,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;
 -    uint8_t *packet_sd;
 -    int size;
 -    AVFrameSideData *frame_sd;
 -
 -    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;
 +    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 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);
 +        /* 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 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);
 +            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;
  
- #if FF_API_AVFRAME_COLORSPACE
 -    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;
- #endif
 +
 +    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;
  
@@@ -481,16 -385,13 +479,14 @@@ int av_frame_copy_props(AVFrame *dst, c
      dst->coded_picture_number   = src->coded_picture_number;
      dst->display_picture_number = src->display_picture_number;
      dst->flags                  = src->flags;
 -#if FF_API_AVFRAME_COLORSPACE
 +    dst->decode_error_flags     = src->decode_error_flags;
- #if FF_API_AVFRAME_COLORSPACE
      dst->color_primaries        = src->color_primaries;
      dst->color_trc              = src->color_trc;
      dst->colorspace             = src->colorspace;
      dst->color_range            = src->color_range;
      dst->chroma_location        = src->chroma_location;
--#endif
 +
 +    av_dict_copy(&dst->metadata, src->metadata, 0);
  
      memcpy(dst->error, src->error, sizeof(dst->error));
  
@@@ -446,14 -433,6 +446,13 @@@ typedef struct AVFrame 
       */
      int flags;
  
- #if FF_API_AVFRAME_COLORSPACE
 +    /**
 +     * MPEG vs JPEG YUV range.
 +     * It must be accessed using av_frame_get_color_range() and
 +     * av_frame_set_color_range().
 +     * - encoding: Set by user
 +     * - decoding: Set by libavcodec
 +     */
      enum AVColorRange color_range;
  
      enum AVColorPrimaries color_primaries;
      enum AVColorSpace colorspace;
  
      enum AVChromaLocation chroma_location;
- #endif
 +
 +    /**
 +     * frame timestamp estimated using various heuristics, in stream time base
 +     * Code outside libavcodec should access this field using:
 +     * av_frame_get_best_effort_timestamp(frame)
 +     * - encoding: unused
 +     * - decoding: set by libavcodec, read by user.
 +     */
 +    int64_t best_effort_timestamp;
 +
 +    /**
 +     * reordered pos from the last AVPacket that has been input into the decoder
 +     * Code outside libavcodec should access this field using:
 +     * av_frame_get_pkt_pos(frame)
 +     * - encoding: unused
 +     * - decoding: Read by user.
 +     */
 +    int64_t pkt_pos;
 +
 +    /**
 +     * duration of the corresponding packet, expressed in
 +     * AVStream->time_base units, 0 if unknown.
 +     * Code outside libavcodec should access this field using:
 +     * av_frame_get_pkt_duration(frame)
 +     * - encoding: unused
 +     * - decoding: Read by user.
 +     */
 +    int64_t pkt_duration;
 +
 +    /**
 +     * metadata.
 +     * Code outside libavcodec should access this field using:
 +     * av_frame_get_metadata(frame)
 +     * - encoding: Set by user.
 +     * - decoding: Set by libavcodec.
 +     */
 +    AVDictionary *metadata;
 +
 +    /**
 +     * decode error flags of the frame, set to a combination of
 +     * FF_DECODE_ERROR_xxx flags if the decoder produced a frame, but there
 +     * were errors during the decoding.
 +     * Code outside libavcodec should access this field using:
 +     * av_frame_get_decode_error_flags(frame)
 +     * - encoding: unused
 +     * - decoding: set by libavcodec, read by user.
 +     */
 +    int decode_error_flags;
 +#define FF_DECODE_ERROR_INVALID_BITSTREAM   1
 +#define FF_DECODE_ERROR_MISSING_REFERENCE   2
 +
 +    /**
 +     * number of audio channels, only used for audio.
 +     * Code outside libavcodec should access this field using:
 +     * av_frame_get_channels(frame)
 +     * - encoding: unused
 +     * - decoding: Read by user.
 +     */
 +    int channels;
 +
 +    /**
 +     * size of the corresponding packet containing the compressed
 +     * frame. It must be accessed using av_frame_get_pkt_size() and
 +     * av_frame_set_pkt_size().
 +     * It is set to a negative value if unknown.
 +     * - encoding: unused
 +     * - decoding: set by libavcodec, read by user.
 +     */
 +    int pkt_size;
 +
 +    /**
 +     * Not to be accessed directly from outside libavutil
 +     */
 +    AVBufferRef *qp_table_buf;
  } AVFrame;
  
  /**
  #ifndef FF_API_OPT_TYPE_METADATA
  #define FF_API_OPT_TYPE_METADATA        (LIBAVUTIL_VERSION_MAJOR < 55)
  #endif
- #ifndef FF_API_AVFRAME_COLORSPACE
- #define FF_API_AVFRAME_COLORSPACE       (LIBAVUTIL_VERSION_MAJOR >= 52)
- #endif
  
  
 +#ifndef FF_CONST_AVUTIL53
 +#if LIBAVUTIL_VERSION_MAJOR >= 53
 +#define FF_CONST_AVUTIL53 const
 +#else
 +#define FF_CONST_AVUTIL53
 +#endif
 +#endif
 +
  /**
   * @}
   */