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

[Diego Biurrun]

On Thu, Nov 12, 2009 at 12:54:37AM +0100, Stefano Sabatini wrote:
> On date Wednesday 2009-11-11 15:52:44 -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:
> >>> On 11/11/2009 03:46 PM, diego wrote:
> >>>>
> >>>> Log:
> >>>> Add #include for required header config.h.
> >>>>
> >>>> --- trunk/ffplay.c	Wed Nov 11 23:36:17 2009	(r20517)
> >>>> +++ trunk/ffplay.c	Thu Nov 12 00:46:11 2009	(r20518)
> >>>> @@ -19,6 +19,7 @@
> >>>>
> >>>> +#include "config.h"
> >>>>   #include<math.h>
> >>>>   #include<limits.h>
> >>>
> >>> 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.
> 
> +1.
> 
> BTW, why is such inclusion *required*, and which are the symbols in
> config.h used by ffmpeg.c and ffserver.c?

grep is your friend, try looking for HAVE_ etc.

Diego