[FFmpeg-devel] [PATCH 05/44] avformat/mux: Move ff_choose_chroma_location to mxfenc, its only user

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Tue May 10 10:29:36 EEST 2022


Tomas Härdin:
> lör 2022-05-07 klockan 13:27 +0200 skrev Andreas Rheinhardt:
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
>> ---
>>  libavformat/internal.h |  5 -----
>>  libavformat/mux.c      | 31 -------------------------------
>>  libavformat/mxfenc.c   | 31 ++++++++++++++++++++++++++++++-
>>  3 files changed, 30 insertions(+), 37 deletions(-)
>>
>> diff --git a/libavformat/internal.h b/libavformat/internal.h
>> index 715e3ce33f..5e84942fb9 100644
>> --- a/libavformat/internal.h
>> +++ b/libavformat/internal.h
>> @@ -765,11 +765,6 @@ int ff_is_intra_only(enum AVCodecID id);
>>   */
>>  enum AVCodecID ff_get_pcm_codec_id(int bps, int flt, int be, int
>> sflags);
>>  
>> -/**
>> - * Chooses a timebase for muxing the specified stream.
>> - */
> 
> What's up with this comment?
> 

It's wrong. It seems to have been duplicated from ff_choose_timebase. Or
maybe there was a merge/rebase error? I didn't check where it came from,
I just removed it.

- Andreas


More information about the ffmpeg-devel mailing list