Merge commit '167ea1fbf15ecefa30729f9b8d091ed431bf43bd'
authorHendrik Leppkes <h.leppkes@gmail.com>
Mon, 24 Aug 2015 08:37:50 +0000 (10:37 +0200)
committerHendrik Leppkes <h.leppkes@gmail.com>
Mon, 24 Aug 2015 08:37:50 +0000 (10:37 +0200)
* commit '167ea1fbf15ecefa30729f9b8d091ed431bf43bd':
  xavs: Do not try to set the bitrate tolerance without a bitrate

Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
1  2 
libavcodec/libxavs.c

diff --combined libavcodec/libxavs.c
index a4d2c79c64e3d6ff602232503eee4539a728a05c,43b0915c52480de0b24a911eef27160c080edaa2..45b4dcb76fe39fa9e88bd85a73ed3bef409d28d3
@@@ -2,20 -2,20 +2,20 @@@
   * AVS encoding using the xavs library
   * Copyright (C) 2010 Amanda, Y.N. Wu <amanda11192003@gmail.com>
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -89,8 -89,10 +89,8 @@@ static int encode_nals(AVCodecContext *
      for (i = 0; i < nnal; i++)
          size += nals[i].i_payload;
  
 -    if ((ret = ff_alloc_packet(pkt, size)) < 0) {
 -        av_log(ctx, AV_LOG_ERROR, "Error getting output packet of size %d.\n", size);
 +    if ((ret = ff_alloc_packet2(ctx, pkt, size, 0)) < 0)
          return ret;
 -    }
      p = pkt->data;
  
      /* Write the SEI as part of the first frame. */
@@@ -118,7 -120,7 +118,7 @@@ static int XAVS_frame(AVCodecContext *a
      xavs_nal_t *nal;
      int nnal, i, ret;
      xavs_picture_t pic_out;
 -    uint8_t *sd;
 +    int pict_type;
  
      x4->pic.img.i_csp   = XAVS_CSP_I420;
      x4->pic.img.i_plane = 3;
  
      if (!ret) {
          if (!frame && !(x4->end_of_stream)) {
 -            if ((ret = ff_alloc_packet(pkt, 4)) < 0)
 +            if ((ret = ff_alloc_packet2(avctx, pkt, 4, 0)) < 0)
                  return ret;
  
              pkt->data[0] = 0x0;
              pkt->data[2] = 0x01;
              pkt->data[3] = 0xb1;
              pkt->dts = 2*x4->pts_buffer[(x4->out_frame_count-1)%(avctx->max_b_frames+1)] -
 -                       x4->pts_buffer[(x4->out_frame_count-2)%(avctx->max_b_frames+1)];
 +                         x4->pts_buffer[(x4->out_frame_count-2)%(avctx->max_b_frames+1)];
              x4->end_of_stream = END_OF_STREAM;
              *got_packet = 1;
          }
@@@ -174,24 -176,21 +174,24 @@@ FF_ENABLE_DEPRECATION_WARNING
      } else
          pkt->dts = pkt->pts;
  
 -#if FF_API_CODED_FRAME
 -FF_DISABLE_DEPRECATION_WARNINGS
      switch (pic_out.i_type) {
      case XAVS_TYPE_IDR:
      case XAVS_TYPE_I:
 -        avctx->coded_frame->pict_type = AV_PICTURE_TYPE_I;
 +        pict_type = AV_PICTURE_TYPE_I;
          break;
      case XAVS_TYPE_P:
 -        avctx->coded_frame->pict_type = AV_PICTURE_TYPE_P;
 +        pict_type = AV_PICTURE_TYPE_P;
          break;
      case XAVS_TYPE_B:
      case XAVS_TYPE_BREF:
 -        avctx->coded_frame->pict_type = AV_PICTURE_TYPE_B;
 +        pict_type = AV_PICTURE_TYPE_B;
          break;
 +    default:
 +        pict_type = AV_PICTURE_TYPE_NONE;
      }
 +#if FF_API_CODED_FRAME
 +FF_DISABLE_DEPRECATION_WARNINGS
 +    avctx->coded_frame->pict_type = pict_type;
  FF_ENABLE_DEPRECATION_WARNINGS
  #endif
  
@@@ -212,7 -211,10 +212,7 @@@ FF_DISABLE_DEPRECATION_WARNING
  FF_ENABLE_DEPRECATION_WARNINGS
  #endif
  
 -    sd = av_packet_new_side_data(pkt, AV_PKT_DATA_QUALITY_FACTOR, sizeof(int));
 -    if (!sd)
 -        return AVERROR(ENOMEM);
 -    *(int *)sd = (pic_out.i_qpplus1 - 1) * FF_QP2LAMBDA;
 +    ff_side_data_set_encoder_stats(pkt, (pic_out.i_qpplus1 - 1) * FF_QP2LAMBDA, NULL, 0, pict_type);
  
      x4->out_frame_count++;
      *got_packet = ret;
@@@ -224,7 -226,7 +224,7 @@@ static av_cold int XAVS_close(AVCodecCo
      XavsContext *x4 = avctx->priv_data;
  
      av_freep(&avctx->extradata);
 -    av_free(x4->sei);
 +    av_freep(&x4->sei);
      av_freep(&x4->pts_buffer);
  
      if (x4->enc)
@@@ -356,7 -358,7 +356,7 @@@ FF_ENABLE_DEPRECATION_WARNING
  
      if (avctx->bit_rate > 0)
          x4->params.rc.f_rate_tolerance =
-             (float)avctx->bit_rate_tolerance/avctx->bit_rate;
+             (float)avctx->bit_rate_tolerance / avctx->bit_rate;
  
      if ((avctx->rc_buffer_size) &&
          (avctx->rc_initial_buffer_occupancy <= avctx->rc_buffer_size)) {
      if (!x4->enc)
          return -1;
  
 -    if (!(x4->pts_buffer = av_mallocz((avctx->max_b_frames+1) * sizeof(*x4->pts_buffer))))
 +    if (!(x4->pts_buffer = av_mallocz_array((avctx->max_b_frames+1), sizeof(*x4->pts_buffer))))
          return AVERROR(ENOMEM);
  
      /* TAG: Do we have GLOBAL HEADER in AVS */
      /* We Have PPS and SPS in AVS */
 -    if (avctx->flags & AV_CODEC_FLAG_GLOBAL_HEADER) {
 +    if (avctx->flags & AV_CODEC_FLAG_GLOBAL_HEADER && 0) {
          xavs_nal_t *nal;
          int nnal, s, i, size;
          uint8_t *p;
  #define OFFSET(x) offsetof(XavsContext, x)
  #define VE AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_ENCODING_PARAM
  static const AVOption options[] = {
 -    { "crf",           "Select the quality for constant quality mode",    OFFSET(crf),           AV_OPT_TYPE_FLOAT,  {-1 }, -1, FLT_MAX, VE },
 +    { "crf",           "Select the quality for constant quality mode",    OFFSET(crf),           AV_OPT_TYPE_FLOAT,  {.dbl = -1 }, -1, FLT_MAX, VE },
      { "qp",            "Constant quantization parameter rate control method",OFFSET(cqp),        AV_OPT_TYPE_INT,    {.i64 = -1 }, -1, INT_MAX, VE },
      { "b-bias",        "Influences how often B-frames are used",          OFFSET(b_bias),        AV_OPT_TYPE_INT,    {.i64 = INT_MIN}, INT_MIN, INT_MAX, VE },
 -    { "cplxblur",      "Reduce fluctuations in QP (before curve compression)", OFFSET(cplxblur), AV_OPT_TYPE_FLOAT,  {-1 }, -1, FLT_MAX, VE},
 +    { "cplxblur",      "Reduce fluctuations in QP (before curve compression)", OFFSET(cplxblur), AV_OPT_TYPE_FLOAT,  {.dbl = -1 }, -1, FLT_MAX, VE},
      { "direct-pred",   "Direct MV prediction mode",                       OFFSET(direct_pred),   AV_OPT_TYPE_INT,    {.i64 = -1 }, -1, INT_MAX, VE, "direct-pred" },
      { "none",          NULL,      0,    AV_OPT_TYPE_CONST, { .i64 = XAVS_DIRECT_PRED_NONE },     0, 0, VE, "direct-pred" },
      { "spatial",       NULL,      0,    AV_OPT_TYPE_CONST, { .i64 = XAVS_DIRECT_PRED_SPATIAL },  0, 0, VE, "direct-pred" },
      { NULL },
  };
  
 -static const AVClass class = {
 +static const AVClass xavs_class = {
      .class_name = "libxavs",
      .item_name  = av_default_item_name,
      .option     = options,
@@@ -464,6 -466,6 +464,6 @@@ AVCodec ff_libxavs_encoder = 
      .close          = XAVS_close,
      .capabilities   = AV_CODEC_CAP_DELAY | AV_CODEC_CAP_AUTO_THREADS,
      .pix_fmts       = (const enum AVPixelFormat[]) { AV_PIX_FMT_YUV420P, AV_PIX_FMT_NONE },
 -    .priv_class     = &class,
 +    .priv_class     = &xavs_class,
      .defaults       = xavs_defaults,
  };