diff options
author | Luca Barbato <lu_zero@gentoo.org> | 2012-10-01 02:49:18 +0400 |
---|---|---|
committer | Luca Barbato <lu_zero@gentoo.org> | 2012-10-01 21:57:57 +0400 |
commit | 1e46c63eb72be752e044ba32257d77f35cbd9dac (patch) | |
tree | c624a4c98905dacd9727d5c0592144da1b7132ad /libavformat/mux.c | |
parent | 86bbdf865e04bc5ddc2021b0620e6de634375253 (diff) |
avformat: refactor avformat_write_header
Split away option settings, sanity checks and general setup.
Diffstat (limited to 'libavformat/mux.c')
-rw-r--r-- | libavformat/mux.c | 53 |
1 files changed, 38 insertions, 15 deletions
diff --git a/libavformat/mux.c b/libavformat/mux.c index 775b5f136a..96eecb5a93 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -135,7 +135,8 @@ static int validate_codec_tag(AVFormatContext *s, AVStream *st) return 1; } -int avformat_write_header(AVFormatContext *s, AVDictionary **options) + +static int init_muxer(AVFormatContext *s, AVDictionary **options) { int ret = 0, i; AVStream *st; @@ -248,12 +249,23 @@ int avformat_write_header(AVFormatContext *s, AVDictionary **options) av_dict_set(&s->metadata, "encoder", LIBAVFORMAT_IDENT, 0); } - if (s->oformat->write_header) { - ret = s->oformat->write_header(s); - if (ret < 0) - goto fail; + if (options) { + av_dict_free(options); + *options = tmp; } + return 0; + +fail: + av_dict_free(&tmp); + return ret; +} + +static int init_pts(AVFormatContext *s) +{ + int i; + AVStream *st; + /* init PTS generation */ for (i = 0; i < s->nb_streams; i++) { int64_t den = AV_NOPTS_VALUE; @@ -270,22 +282,33 @@ int avformat_write_header(AVFormatContext *s, AVDictionary **options) break; } if (den != AV_NOPTS_VALUE) { - if (den <= 0) { - ret = AVERROR_INVALIDDATA; - goto fail; - } + if (den <= 0) + return AVERROR_INVALIDDATA; + frac_init(&st->pts, 0, 0, den); } } - if (options) { - av_dict_free(options); - *options = tmp; + return 0; +} + +int avformat_write_header(AVFormatContext *s, AVDictionary **options) +{ + int ret = 0; + + if (ret = init_muxer(s, options)) + return ret; + + if (s->oformat->write_header) { + ret = s->oformat->write_header(s); + if (ret < 0) + return ret; } + + if ((ret = init_pts(s) < 0)) + return ret; + return 0; -fail: - av_dict_free(&tmp); - return ret; } //FIXME merge with compute_pkt_fields |