Skip to content

Commit

Permalink
avformat/wav: Deduplicate codec tags lists
Browse files Browse the repository at this point in the history
Also saves relocations.

Signed-off-by: Andreas Rheinhardt <[email protected]>
  • Loading branch information
mkver committed Feb 23, 2021
1 parent 09e6d3a commit 1df1083
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 4 deletions.
4 changes: 4 additions & 0 deletions libavformat/riff.c
Original file line number Diff line number Diff line change
Expand Up @@ -596,6 +596,10 @@ const AVCodecTag *const ff_riff_codec_tags_list[] = {
};
#endif

#if CONFIG_WAV_DEMUXER || CONFIG_WAV_MUXER || CONFIG_W64_DEMUXER || CONFIG_W64_MUXER
const AVCodecTag *const ff_wav_codec_tags_list[] = { ff_codec_wav_tags, NULL };
#endif

const AVMetadataConv ff_riff_info_conv[] = {
{ "IART", "artist" },
{ "ICMT", "comment" },
Expand Down
2 changes: 2 additions & 0 deletions libavformat/riff.h
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,8 @@ extern const AVCodecTag ff_codec_bmp_tags[]; // exposed through avformat_get_rif
extern const AVCodecTag ff_codec_wav_tags[];
/* The following list contains both ff_codec_bmp_tags and ff_codec_wav_tags. */
extern const AVCodecTag *const ff_riff_codec_tags_list[];
/* The following list contains only ff_codec_wav_tags. */
extern const AVCodecTag *const ff_wav_codec_tags_list[];

extern const AVCodecTag ff_codec_bmp_tags_unofficial[];

Expand Down
4 changes: 2 additions & 2 deletions libavformat/wavdec.c
Original file line number Diff line number Diff line change
Expand Up @@ -821,7 +821,7 @@ AVInputFormat ff_wav_demuxer = {
.read_packet = wav_read_packet,
.read_seek = wav_read_seek,
.flags = AVFMT_GENERIC_INDEX,
.codec_tag = (const AVCodecTag * const []) { ff_codec_wav_tags, 0 },
.codec_tag = ff_wav_codec_tags_list,
.priv_class = &wav_demuxer_class,
};
#endif /* CONFIG_WAV_DEMUXER */
Expand Down Expand Up @@ -978,7 +978,7 @@ AVInputFormat ff_w64_demuxer = {
.read_packet = wav_read_packet,
.read_seek = wav_read_seek,
.flags = AVFMT_GENERIC_INDEX,
.codec_tag = (const AVCodecTag * const []) { ff_codec_wav_tags, 0 },
.codec_tag = ff_wav_codec_tags_list,
.priv_class = &w64_demuxer_class,
};
#endif /* CONFIG_W64_DEMUXER */
4 changes: 2 additions & 2 deletions libavformat/wavenc.c
Original file line number Diff line number Diff line change
Expand Up @@ -514,7 +514,7 @@ AVOutputFormat ff_wav_muxer = {
.write_trailer = wav_write_trailer,
.deinit = wav_deinit,
.flags = AVFMT_TS_NONSTRICT,
.codec_tag = (const AVCodecTag* const []){ ff_codec_wav_tags, 0 },
.codec_tag = ff_wav_codec_tags_list,
.priv_class = &wav_muxer_class,
};
#endif /* CONFIG_WAV_MUXER */
Expand Down Expand Up @@ -612,6 +612,6 @@ AVOutputFormat ff_w64_muxer = {
.write_packet = wav_write_packet,
.write_trailer = w64_write_trailer,
.flags = AVFMT_TS_NONSTRICT,
.codec_tag = (const AVCodecTag* const []){ ff_codec_wav_tags, 0 },
.codec_tag = ff_wav_codec_tags_list,
};
#endif /* CONFIG_W64_MUXER */

0 comments on commit 1df1083

Please sign in to comment.