________________________________ From: ffmpeg-devel <[email protected]> on behalf of Anton Khirnov <[email protected]> Sent: Monday, October 5, 2020 1:50 PM To: FFmpeg development discussions and patches <[email protected]> Subject: Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/ac3tab: rename ff_ac3_sample_rate_tab to avpriv_ac3_sample_rate_tab so that it can be used in libavformat
Quoting Nachiket Tarate (2020-10-04 16:35:09) > This will be used by HLS demuxer to parse EC3SpecificBox (dec3) during > SAMPLE-AES decryption. > > Signed-off-by: Nachiket Tarate <[email protected]> Since the table is so small, it seems preferable to duplicate it in libavformat than add eat another private symbol. Private symbols are evil and should not exist. -- Anton Khirnov @Anton Khirnov <[email protected]> In the first version of the patch, I had implemented avpriv_eac3_parse_dec3() function in ac3_parser.c to avoid sharing or duplication of table across the libraries. I changed that implementation based on James Almer <[email protected]>'s review comments. @James Almer <[email protected]> What is your opinion about Anton Khirnov <[email protected]>'s review comments ? -- Nachiket Tarate _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
