[FFmpeg-devel] [PATCH 6/6 v2] avformat/movenc: add support for Immersive Audio Model and Formats in ISOBMFF

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Mon Feb 5 17:12:44 EET 2024


James Almer:
> On 2/3/2024 11:50 AM, Andreas Rheinhardt wrote:
>>> diff --git a/libavformat/movenc.h b/libavformat/movenc.h
>>> index 60363198c9..fee3e759e0 100644
>>> --- a/libavformat/movenc.h
>>> +++ b/libavformat/movenc.h
>>> @@ -25,7 +25,9 @@
>>>   #define AVFORMAT_MOVENC_H
>>>     #include "avformat.h"
>>> +#include "iamf.h"
>>>   #include "movenccenc.h"
>>> +#include "libavcodec/bsf.h"
>>
>> There is no need to include these here, as you don't need complete
>> types. This has the added benefit of forcing you to actually include the
>> files where you are using them (namely in movenc.c, where you forgot to
>> include bsf.h).
> 
> Ok, fixed locally.
> 
> Will push the set soon.

It seems you have not noticed my objection to the first version of your set.

- Andreas



More information about the ffmpeg-devel mailing list