[FFmpeg-devel] [PATCH 5/5] avformat/rawenc: Use init instead of write_header function
Andreas Rheinhardt
andreas.rheinhardt at gmail.com
Thu Mar 11 02:51:46 EET 2021
Andreas Rheinhardt:
> force_one_stream() does not write anything.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> Is it it actually intended that the mp2 muxer does not use
> force_one_stream at all? And is it intended that force_one_stream
> does only check codec_type and not the actual codec?
> And wouldn't it be better to inform the caller about the fact that
> the format supports only one stream by adding a new
> AVOutputFormat.flags?
>
> libavformat/rawenc.c | 60 ++++++++++++++++++++++----------------------
> 1 file changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/libavformat/rawenc.c b/libavformat/rawenc.c
> index 32704f9bfd..caec297f4a 100644
> --- a/libavformat/rawenc.c
> +++ b/libavformat/rawenc.c
> @@ -64,7 +64,7 @@ AVOutputFormat ff_ac3_muxer = {
> .extensions = "ac3",
> .audio_codec = AV_CODEC_ID_AC3,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -96,7 +96,7 @@ AVOutputFormat ff_adx_muxer = {
> .extensions = "adx",
> .audio_codec = AV_CODEC_ID_ADPCM_ADX,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .write_trailer = adx_write_trailer,
> .flags = AVFMT_NOTIMESTAMPS,
> @@ -110,7 +110,7 @@ AVOutputFormat ff_aptx_muxer = {
> .extensions = "aptx",
> .audio_codec = AV_CODEC_ID_APTX,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -123,7 +123,7 @@ AVOutputFormat ff_aptx_hd_muxer = {
> .extensions = "aptxhd",
> .audio_codec = AV_CODEC_ID_APTX_HD,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -136,7 +136,7 @@ AVOutputFormat ff_avs2_muxer = {
> .extensions = "avs,avs2",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_AVS2,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -149,7 +149,7 @@ AVOutputFormat ff_cavsvideo_muxer = {
> .extensions = "cavs",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_CAVS,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -161,7 +161,7 @@ AVOutputFormat ff_codec2raw_muxer = {
> .long_name = NULL_IF_CONFIG_SMALL("raw codec2 muxer"),
> .audio_codec = AV_CODEC_ID_CODEC2,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -172,7 +172,7 @@ AVOutputFormat ff_codec2raw_muxer = {
> AVOutputFormat ff_data_muxer = {
> .name = "data",
> .long_name = NULL_IF_CONFIG_SMALL("raw data"),
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -185,7 +185,7 @@ AVOutputFormat ff_dirac_muxer = {
> .extensions = "drc,vc2",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_DIRAC,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -198,7 +198,7 @@ AVOutputFormat ff_dnxhd_muxer = {
> .extensions = "dnxhd,dnxhr",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_DNXHD,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -212,7 +212,7 @@ AVOutputFormat ff_dts_muxer = {
> .extensions = "dts",
> .audio_codec = AV_CODEC_ID_DTS,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -226,7 +226,7 @@ AVOutputFormat ff_eac3_muxer = {
> .extensions = "eac3",
> .audio_codec = AV_CODEC_ID_EAC3,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -240,7 +240,7 @@ AVOutputFormat ff_g722_muxer = {
> .extensions = "g722",
> .audio_codec = AV_CODEC_ID_ADPCM_G722,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -254,7 +254,7 @@ AVOutputFormat ff_g723_1_muxer = {
> .extensions = "tco,rco",
> .audio_codec = AV_CODEC_ID_G723_1,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -266,7 +266,7 @@ AVOutputFormat ff_g726_muxer = {
> .long_name = NULL_IF_CONFIG_SMALL("raw big-endian G.726 (\"left-justified\")"),
> .audio_codec = AV_CODEC_ID_ADPCM_G726,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -278,7 +278,7 @@ AVOutputFormat ff_g726le_muxer = {
> .long_name = NULL_IF_CONFIG_SMALL("raw little-endian G.726 (\"right-justified\")"),
> .audio_codec = AV_CODEC_ID_ADPCM_G726LE,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -292,7 +292,7 @@ AVOutputFormat ff_gsm_muxer = {
> .extensions = "gsm",
> .audio_codec = AV_CODEC_ID_GSM,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -306,7 +306,7 @@ AVOutputFormat ff_h261_muxer = {
> .extensions = "h261",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_H261,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -320,7 +320,7 @@ AVOutputFormat ff_h263_muxer = {
> .extensions = "h263",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_H263,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -342,7 +342,7 @@ AVOutputFormat ff_h264_muxer = {
> .extensions = "h264,264",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_H264,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .check_bitstream = h264_check_bitstream,
> .flags = AVFMT_NOTIMESTAMPS,
> @@ -365,7 +365,7 @@ AVOutputFormat ff_hevc_muxer = {
> .extensions = "hevc,h265,265",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_HEVC,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .check_bitstream = hevc_check_bitstream,
> .flags = AVFMT_NOTIMESTAMPS,
> @@ -379,7 +379,7 @@ AVOutputFormat ff_m4v_muxer = {
> .extensions = "m4v",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_MPEG4,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -393,7 +393,7 @@ AVOutputFormat ff_mjpeg_muxer = {
> .extensions = "mjpg,mjpeg",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_MJPEG,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -406,9 +406,9 @@ AVOutputFormat ff_singlejpeg_muxer = {
> .mime_type = "image/jpeg",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_MJPEG,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> - .write_header = force_one_stream,
> };
> #endif
>
> @@ -419,7 +419,7 @@ AVOutputFormat ff_mlp_muxer = {
> .extensions = "mlp",
> .audio_codec = AV_CODEC_ID_MLP,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -446,7 +446,7 @@ AVOutputFormat ff_mpeg1video_muxer = {
> .extensions = "mpg,mpeg,m1v",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_MPEG1VIDEO,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -459,7 +459,7 @@ AVOutputFormat ff_mpeg2video_muxer = {
> .extensions = "m2v",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_MPEG2VIDEO,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -484,7 +484,7 @@ AVOutputFormat ff_sbc_muxer = {
> .mime_type = "audio/x-sbc",
> .extensions = "sbc,msbc",
> .audio_codec = AV_CODEC_ID_SBC,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -497,7 +497,7 @@ AVOutputFormat ff_truehd_muxer = {
> .extensions = "thd",
> .audio_codec = AV_CODEC_ID_TRUEHD,
> .video_codec = AV_CODEC_ID_NONE,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
> @@ -510,7 +510,7 @@ AVOutputFormat ff_vc1_muxer = {
> .extensions = "vc1",
> .audio_codec = AV_CODEC_ID_NONE,
> .video_codec = AV_CODEC_ID_VC1,
> - .write_header = force_one_stream,
> + .init = force_one_stream,
> .write_packet = ff_raw_write_packet,
> .flags = AVFMT_NOTIMESTAMPS,
> };
>
Will apply this tonight unless there are objections.
- Andreas
More information about the ffmpeg-devel
mailing list