Merge commit '182d3f8221c23716ea4eafa460bdb94968f71f26'
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 26 Jun 2014 22:50:32 +0000 (00:50 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Thu, 26 Jun 2014 22:57:04 +0000 (00:57 +0200)
* commit '182d3f8221c23716ea4eafa460bdb94968f71f26':
  output example: rewrite encoding logic

Conflicts:
doc/examples/muxing.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
doc/examples/muxing.c

  #include <string.h>
  #include <math.h>
  
 -#include "libavutil/channel_layout.h"
 -#include "libavutil/mathematics.h"
 -#include "libavformat/avformat.h"
 -#include "libswscale/swscale.h"
 +#include <libavutil/avassert.h>
 +#include <libavutil/channel_layout.h>
 +#include <libavutil/opt.h>
 +#include <libavutil/mathematics.h>
 +#include <libavutil/timestamp.h>
 +#include <libavformat/avformat.h>
 +#include <libswscale/swscale.h>
 +#include <libswresample/swresample.h>
  
- static int audio_is_eof, video_is_eof;
 -/* 5 seconds stream duration */
 -#define STREAM_DURATION   5.0
 +#define STREAM_DURATION   10.0
  #define STREAM_FRAME_RATE 25 /* 25 images/s */
 -#define STREAM_NB_FRAMES  ((int)(STREAM_DURATION * STREAM_FRAME_RATE))
  #define STREAM_PIX_FMT    AV_PIX_FMT_YUV420P /* default pix_fmt */
  
  #define SCALE_FLAGS SWS_BICUBIC
@@@ -266,67 -182,82 +281,62 @@@ static int write_audio_frame(AVFormatCo
  {
      AVCodecContext *c;
      AVPacket pkt = { 0 }; // data and size must be 0;
-     int got_packet, ret;
-     int dst_nb_samples;
      AVFrame *frame;
++    int ret;
+     int got_packet;
++    int dst_nb_samples;
  
      av_init_packet(&pkt);
      c = ost->st->codec;
  
-     if (!flush) {
-         get_audio_frame(ost, ost->frame, c->channels);
+     frame = get_audio_frame(ost);
  
 -    avcodec_encode_audio2(c, &pkt, frame, &got_packet);
++    if (frame) {
 +        /* convert samples from native format to destination codec format, using the resampler */
 +        if (swr_ctx) {
 +            /* compute destination number of samples */
-             dst_nb_samples = av_rescale_rnd(swr_get_delay(swr_ctx, c->sample_rate) + ost->frame->nb_samples,
++            dst_nb_samples = av_rescale_rnd(swr_get_delay(swr_ctx, c->sample_rate) + frame->nb_samples,
 +                                            c->sample_rate, c->sample_rate, AV_ROUND_UP);
-             av_assert0(dst_nb_samples == ost->frame->nb_samples);
++            av_assert0(dst_nb_samples == frame->nb_samples);
 +
 +            /* convert to destination format */
 +            ret = swr_convert(swr_ctx,
 +                              ost->tmp_frame->data, dst_nb_samples,
-                               (const uint8_t **)ost->frame->data, ost->frame->nb_samples);
++                              (const uint8_t **)frame->data, frame->nb_samples);
 +            if (ret < 0) {
 +                fprintf(stderr, "Error while converting\n");
 +                exit(1);
 +            }
 +            frame = ost->tmp_frame;
 +        } else {
-             dst_nb_samples = ost->frame->nb_samples;
-             frame = ost->frame;
++            dst_nb_samples = frame->nb_samples;
 +        }
  
 -    if (got_packet) {
 -        pkt.stream_index = ost->st->index;
 +        frame->pts = av_rescale_q(samples_count, (AVRational){1, c->sample_rate}, c->time_base);
 +        samples_count += dst_nb_samples;
 +    }
 +
-     ret = avcodec_encode_audio2(c, &pkt, flush ? NULL : frame, &got_packet);
++    ret = avcodec_encode_audio2(c, &pkt, frame, &got_packet);
 +    if (ret < 0) {
 +        fprintf(stderr, "Error encoding audio frame: %s\n", av_err2str(ret));
 +        exit(1);
 +    }
  
-     if (!got_packet) {
-         if (flush)
-             audio_is_eof = 1;
-         return;
 -        /* Write the compressed frame to the media file. */
 -        if (av_interleaved_write_frame(oc, &pkt) != 0) {
 -            fprintf(stderr, "Error while writing audio frame\n");
++    if (got_packet) {
++        ret = write_frame(oc, &c->time_base, ost->st, &pkt);
++        if (ret < 0) {
++            fprintf(stderr, "Error while writing audio frame: %s\n",
++                    av_err2str(ret));
+             exit(1);
+         }
      }
  
-     ret = write_frame(oc, &c->time_base, ost->st, &pkt);
-     if (ret < 0) {
-         fprintf(stderr, "Error while writing audio frame: %s\n",
-                 av_err2str(ret));
-         exit(1);
-     }
+     return (frame || got_packet) ? 0 : 1;
  }
  
  /**************************************************************/
  /* video output */
  
- static int frame_count;
 -/* Add a video output stream. */
 -static void add_video_stream(OutputStream *ost, AVFormatContext *oc,
 -                             enum AVCodecID codec_id)
 -{
 -    AVCodecContext *c;
 -    AVCodec *codec;
 -
 -    /* find the video encoder */
 -    codec = avcodec_find_encoder(codec_id);
 -    if (!codec) {
 -        fprintf(stderr, "codec not found\n");
 -        exit(1);
 -    }
 -
 -    ost->st = avformat_new_stream(oc, codec);
 -    if (!ost->st) {
 -        fprintf(stderr, "Could not alloc stream\n");
 -        exit(1);
 -    }
 -
 -    c = ost->st->codec;
 -
 -    /* Put sample parameters. */
 -    c->bit_rate = 400000;
 -    /* Resolution must be a multiple of two. */
 -    c->width    = 352;
 -    c->height   = 288;
 -    /* timebase: This is the fundamental unit of time (in seconds) in terms
 -     * of which frame timestamps are represented. For fixed-fps content,
 -     * timebase should be 1/framerate and timestamp increments should be
 -     * identical to 1. */
 -    c->time_base.den = STREAM_FRAME_RATE;
 -    c->time_base.num = 1;
 -    c->gop_size      = 12; /* emit one intra frame every twelve frames at most */
 -    c->pix_fmt       = STREAM_PIX_FMT;
 -    if (c->codec_id == AV_CODEC_ID_MPEG2VIDEO) {
 -        /* just for testing, we also add B frames */
 -        c->max_b_frames = 2;
 -    }
 -    if (c->codec_id == AV_CODEC_ID_MPEG1VIDEO) {
 -        /* Needed to avoid using macroblocks in which some coeffs overflow.
 -         * This does not happen with normal video, it just happens here as
 -         * the motion of the chroma plane does not match the luma plane. */
 -        c->mb_decision = 2;
 -    }
 -    /* Some formats want stream headers to be separate. */
 -    if (oc->oformat->flags & AVFMT_GLOBALHEADER)
 -        c->flags |= CODEC_FLAG_GLOBAL_HEADER;
 -}
--
  static AVFrame *alloc_picture(enum AVPixelFormat pix_fmt, int width, int height)
  {
      AVFrame *picture;
@@@ -412,37 -343,59 +422,60 @@@ static void fill_yuv_image(AVFrame *pic
      }
  }
  
- static void write_video_frame(AVFormatContext *oc, OutputStream *ost, int flush)
+ static AVFrame *get_video_frame(OutputStream *ost)
  {
-     int ret;
      AVCodecContext *c = ost->st->codec;
  
-     if (!flush) {
-         if (c->pix_fmt != AV_PIX_FMT_YUV420P) {
-             /* as we only generate a YUV420P picture, we must convert it
-              * to the codec pixel format if needed */
+     /* check if we want to generate more frames */
+     if (av_compare_ts(ost->next_pts, ost->st->codec->time_base,
+                       STREAM_DURATION, (AVRational){ 1, 1 }) >= 0)
+         return NULL;
+     if (c->pix_fmt != AV_PIX_FMT_YUV420P) {
+         /* as we only generate a YUV420P picture, we must convert it
+          * to the codec pixel format if needed */
+         if (!ost->sws_ctx) {
+             ost->sws_ctx = sws_getContext(c->width, c->height,
+                                           AV_PIX_FMT_YUV420P,
+                                           c->width, c->height,
+                                           c->pix_fmt,
+                                           SCALE_FLAGS, NULL, NULL, NULL);
              if (!ost->sws_ctx) {
-                 ost->sws_ctx = sws_getContext(c->width, c->height, AV_PIX_FMT_YUV420P,
-                                               c->width, c->height,
-                                               c->pix_fmt,
-                                               SCALE_FLAGS, NULL, NULL, NULL);
-                 if (!ost->sws_ctx) {
-                     fprintf(stderr,
-                             "Could not initialize the conversion context\n");
-                     exit(1);
-                 }
+                 fprintf(stderr,
 -                        "Cannot initialize the conversion context\n");
++                        "Could not initialize the conversion context\n");
+                 exit(1);
              }
-             fill_yuv_image(ost->tmp_frame, frame_count, c->width, c->height);
-             sws_scale(ost->sws_ctx,
-                       (const uint8_t * const *)ost->tmp_frame->data, ost->tmp_frame->linesize,
-                       0, c->height, ost->frame->data, ost->frame->linesize);
-         } else {
-             fill_yuv_image(ost->frame, frame_count, c->width, c->height);
          }
 -        sws_scale(ost->sws_ctx, ost->tmp_frame->data, ost->tmp_frame->linesize,
+         fill_yuv_image(ost->tmp_frame, ost->next_pts, c->width, c->height);
++        sws_scale(ost->sws_ctx,
++                  (const uint8_t * const *)ost->tmp_frame->data, ost->tmp_frame->linesize,
+                   0, c->height, ost->frame->data, ost->frame->linesize);
+     } else {
+         fill_yuv_image(ost->frame, ost->next_pts, c->width, c->height);
      }
  
-     if (oc->oformat->flags & AVFMT_RAWPICTURE && !flush) {
-         /* Raw video case - directly store the picture in the packet */
+     ost->frame->pts = ost->next_pts++;
+     return ost->frame;
+ }
+ /*
+  * encode one video frame and send it to the muxer
+  * return 1 when encoding is finished, 0 otherwise
+  */
+ static int write_video_frame(AVFormatContext *oc, OutputStream *ost)
+ {
+     int ret;
+     AVCodecContext *c;
+     AVFrame *frame;
+     int got_packet = 0;
+     c = ost->st->codec;
+     frame = get_video_frame(ost);
+     if (oc->oformat->flags & AVFMT_RAWPICTURE) {
+         /* a hack to avoid data copy with some raw video muxers */
          AVPacket pkt;
          av_init_packet(&pkt);
  
          av_init_packet(&pkt);
  
          /* encode the image */
-         ost->frame->pts = frame_count;
-         ret = avcodec_encode_video2(c, &pkt, flush ? NULL : ost->frame, &got_packet);
+         ret = avcodec_encode_video2(c, &pkt, frame, &got_packet);
          if (ret < 0) {
 -            fprintf(stderr, "Error encoding a video frame\n");
 +            fprintf(stderr, "Error encoding video frame: %s\n", av_err2str(ret));
              exit(1);
          }
-         /* If size is zero, it means the image was buffered. */
  
          if (got_packet) {
 -            av_packet_rescale_ts(&pkt, c->time_base, ost->st->time_base);
 -            pkt.stream_index = ost->st->index;
 -
 -            /* Write the compressed frame to the media file. */
 -            ret = av_interleaved_write_frame(oc, &pkt);
 +            ret = write_frame(oc, &c->time_base, ost->st, &pkt);
 +        } else {
-             if (flush)
-                 video_is_eof = 1;
 +            ret = 0;
          }
      }
 -    if (ret != 0) {
 -        fprintf(stderr, "Error while writing video frame\n");
 +
 +    if (ret < 0) {
 +        fprintf(stderr, "Error while writing video frame: %s\n", av_err2str(ret));
          exit(1);
      }
-     frame_count++;
+     return (frame || got_packet) ? 0 : 1;
  }
  
  static void close_stream(AVFormatContext *oc, OutputStream *ost)
@@@ -499,10 -455,9 +534,10 @@@ int main(int argc, char **argv
      const char *filename;
      AVOutputFormat *fmt;
      AVFormatContext *oc;
-     double audio_time, video_time;
 +    AVCodec *audio_codec, *video_codec;
 +    int flush, ret;
      int have_video = 0, have_audio = 0;
 -    int i;
+     int encode_video = 0, encode_audio = 0;
  
      /* Initialize libavcodec, and register all codecs and formats. */
      av_register_all();
      /* Add the audio and video streams using the default format codecs
       * and initialize the codecs. */
      if (fmt->video_codec != AV_CODEC_ID_NONE) {
 -        add_video_stream(&video_st, oc, fmt->video_codec);
 +        add_stream(&video_st, oc, &video_codec, fmt->video_codec);
          have_video = 1;
+         encode_video = 1;
      }
      if (fmt->audio_codec != AV_CODEC_ID_NONE) {
 -        add_audio_stream(&audio_st, oc, fmt->audio_codec);
 +        add_stream(&audio_st, oc, &audio_codec, fmt->audio_codec);
          have_audio = 1;
+         encode_audio = 1;
      }
  
      /* Now that all the parameters are set, we can open the audio and
      }
  
      /* Write the stream header, if any. */
 -    avformat_write_header(oc, NULL);
 +    ret = avformat_write_header(oc, NULL);
 +    if (ret < 0) {
 +        fprintf(stderr, "Error occurred when opening output file: %s\n",
 +                av_err2str(ret));
 +        return 1;
 +    }
  
-     flush = 0;
-     while ((have_video && !video_is_eof) || (have_audio && !audio_is_eof)) {
-         /* Compute current audio and video time. */
-         audio_time = (have_audio && !audio_is_eof) ? audio_st.st->pts.val * av_q2d(audio_st.st->time_base) : INFINITY;
-         video_time = (have_video && !video_is_eof) ? video_st.st->pts.val * av_q2d(video_st.st->time_base) : INFINITY;
-         if (!flush &&
-             (!have_audio || audio_time >= STREAM_DURATION) &&
-             (!have_video || video_time >= STREAM_DURATION)) {
-             flush = 1;
-         }
-         /* write interleaved audio and video frames */
-         if (have_audio && !audio_is_eof && audio_time <= video_time) {
-             write_audio_frame(oc, &audio_st, flush);
-         } else if (have_video && !video_is_eof && video_time < audio_time) {
-             write_video_frame(oc, &video_st, flush);
+     while (encode_video || encode_audio) {
+         /* select the stream to encode */
+         if (encode_video &&
+             (!encode_audio || av_compare_ts(video_st.next_pts, video_st.st->codec->time_base,
+                                             audio_st.next_pts, audio_st.st->codec->time_base) <= 0)) {
+             encode_video = !write_video_frame(oc, &video_st);
+         } else {
+             encode_audio = !write_audio_frame(oc, &audio_st);
          }
      }