[FFmpeg-devel] [PATCH] Parsing ALS object type in MPEG-4

[Thilo Borgmann]

Baptiste Coudurier schrieb:
> On 10/9/09 1:48 PM, Alex Converse wrote:
>> On Sun, Aug 23, 2009 at 3:51 PM, Thilo Borgmann
>> <thilo.borgmann at googlemail.com>  wrote:
>>> Index: libavcodec/mpeg4audio.h
>>> ===================================================================
>>> --- libavcodec/mpeg4audio.h     (revision 19689)
>>> +++ libavcodec/mpeg4audio.h     (working copy)
>>> @@ -36,6 +36,7 @@
>>>      int ext_sampling_index;
>>>      int ext_sample_rate;
>>>      int ext_chan_config;
>>> +    int channels;
>>>   } MPEG4AudioConfig;
>>
>> Can we put "channels" directly below "chan_config" without API/ABI
>> breakage? If so I'd like to try to keep related thing together.
> 
> We could since this struct is not part of the public API, though since
> libavformat uses libavcodec, it is not safe.
> 

Patch updated.


> This reminds me, I think avcodec_register_all and av_register_all should
> put requirements for libavcodec and libavutil versions respectively, to
> avoid shared libraries problems, because soname only contains major
> version and API changes only update minor version.
> 



-Thilo
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: als_readconfig.rev7.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091011/71db009e/attachment.asc>