[FFmpeg-cvslog] lavf: eliminate ff_get_audio_frame_size()

Anton Khirnov git at videolan.org
Thu Aug 14 03:24:40 CEST 2014


ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Mon Jul 28 13:27:57 2014 +0000| [d92550d19129f230d534c832c5a41591682ebbf8] | committer: Michael Niedermayer

lavf: eliminate ff_get_audio_frame_size()

It is basically a wrapper around av_get_audio_frame_duration(), with a
fallback to AVCodecContext.frame_size. However, that field is set only
when the stream codec context is actually used for encoding or decoding,
which is discouraged.

For muxing, it is generally the responsibility of the caller to set the
packet duration.
For demuxing, if the duration is not stored at the container level, it
should be set by the parser.

Therefore, removing the frame_size fallback should not break any
important case.
(cherry picked from commit 30e50c50274f88f0f5ae829f401cd3c7f5266719)

Conflicts:

	libavformat/utils.c

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d92550d19129f230d534c832c5a41591682ebbf8
---

 libavformat/internal.h |    2 --
 libavformat/mux.c      |    2 +-
 libavformat/utils.c    |   17 +----------------
 3 files changed, 2 insertions(+), 19 deletions(-)

diff --git a/libavformat/internal.h b/libavformat/internal.h
index 08987c7..3011706 100644
--- a/libavformat/internal.h
+++ b/libavformat/internal.h
@@ -337,8 +337,6 @@ void ff_free_stream(AVFormatContext *s, AVStream *st);
 void ff_compute_frame_duration(int *pnum, int *pden, AVStream *st,
                                AVCodecParserContext *pc, AVPacket *pkt);
 
-int ff_get_audio_frame_size(AVCodecContext *enc, int size, int mux);
-
 unsigned int ff_codec_get_tag(const AVCodecTag *tags, enum AVCodecID id);
 
 enum AVCodecID ff_codec_get_id(const AVCodecTag *tags, unsigned int tag);
diff --git a/libavformat/mux.c b/libavformat/mux.c
index dbb9b55..55add43 100644
--- a/libavformat/mux.c
+++ b/libavformat/mux.c
@@ -521,7 +521,7 @@ static int compute_pkt_fields2(AVFormatContext *s, AVStream *st, AVPacket *pkt)
     case AVMEDIA_TYPE_AUDIO:
         frame_size = (pkt->flags & AV_PKT_FLAG_UNCODED_FRAME) ?
                      ((AVFrame *)pkt->data)->nb_samples :
-                     ff_get_audio_frame_size(st->codec, pkt->size, 1);
+                     av_get_audio_frame_duration(st->codec, pkt->size);
 
         /* HACK/FIXME, we skip the initial 0 size packets as they are most
          * likely equal to the encoder delay, but it would be better if we
diff --git a/libavformat/utils.c b/libavformat/utils.c
index 364796a..5d146d6 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -713,21 +713,6 @@ static int determinable_frame_size(AVCodecContext *avctx)
 }
 
 /**
- * Get the number of samples of an audio frame. Return -1 on error.
- */
-int ff_get_audio_frame_size(AVCodecContext *enc, int size, int mux)
-{
-    int frame_size;
-
-    if ((frame_size = av_get_audio_frame_duration(enc, size)) > 0)
-        return frame_size;
-
-
-
-    return -1;
-}
-
-/**
  * Return the frame duration in seconds. Return 0 if not available.
  */
 void ff_compute_frame_duration(int *pnum, int *pden, AVStream *st,
@@ -762,7 +747,7 @@ void ff_compute_frame_duration(int *pnum, int *pden, AVStream *st,
         }
         break;
     case AVMEDIA_TYPE_AUDIO:
-        frame_size = ff_get_audio_frame_size(st->codec, pkt->size, 0);
+        frame_size = av_get_audio_frame_duration(st->codec, pkt->size);
         if (frame_size <= 0 || st->codec->sample_rate <= 0)
             break;
         *pnum = frame_size;



More information about the ffmpeg-cvslog mailing list