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

[Stefano Sabatini]

On date Tuesday 2009-02-24 23:10:25 +0100, Michael Niedermayer encoded:
> 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 ?

So I'd say to simply mark the file as "installed but internal", like
in the attached patches.

I consider it an API fix, so I'm bumping micro.

Regards.
-- 
FFmpeg = Fostering and Friendly Moronic Prodigious Extensive God
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-avio-notice.patch
Type: text/x-diff
Size: 566 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090226/607939ad/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: specify-avio-h-usage.patch
Type: text/x-diff
Size: 1068 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090226/607939ad/attachment-0001.patch>