[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