[FFmpeg-devel] [PATCH] Create a libavformat/version.h header

[Baptiste Coudurier]

Michael Niedermayer wrote:
> On Tue, Feb 24, 2009 at 01:40:31PM -0800, Baptiste Coudurier wrote:
>> Hi,
>>
>> On 2/24/2009 11:20 AM, Michael Niedermayer wrote:
>>> On Tue, Feb 24, 2009 at 08:03:51PM +0100, Stefano Sabatini wrote:
>>>> On date Monday 2009-02-23 00:39:31 +0100, Diego Biurrun encoded:
>>>>> On Sun, Feb 22, 2009 at 04:37:43PM +0100, Stefano Sabatini wrote:
>>>>>> the patch creates a separate version.h file, which makes possible to
>>>>>> include in a project avio.h without to include before avformat.h.
>>>>> Looks good to me, but you will need Michael to approve it.
>>>> Michael?
>>>>
>>>> If we decide this is the way to go, then IMO this should be done
>>>> before the release (currently being avio.h inclusion borken).
>>> IMHO avio.h is not supposed to be included directly in user apps
>>> is there a problem left after this assumtation?
>>> did any app include avio.h directly? if so why?
>> The very usefull ByteIOContext and get_be/le functions.
> 
> what are they usefull for without avformat.h ?
> 

You can use the API to open a file (url_fopen) and read content using
get_be/le.

Very useful, I actually use it.

It doesn't need avformat.h, you can include <libavformat/avio.h> directly.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org