[FFmpeg-cvslog] r20518 - trunk/ffplay.c

[Uoti Urpala]

On Wed, 2009-11-11 at 15:52 -0800, Baptiste Coudurier wrote:
> On 11/11/2009 03:51 PM, Diego Biurrun wrote:
> > On Wed, Nov 11, 2009 at 03:47:08PM -0800, Baptiste Coudurier wrote:

> >> Are applications supposed to include "config.h" ?
> >
> > ffserver.c and ffmpeg.c already do it.  All three use definitions from
> > config.h all over the place...
> 
> I'm worried since it's supposed to be an example on how to use the 
> libraries.

Applications would usually include their *own* config.h or equivalent.
FFmpeg applications share one with the libraries.

> IMHO we should once and for all clean the mess and make the symbols not 
> part of the API _invisible_.

Visibility support would be good, but I think it's mostly orthogonal to
config.h definitions.