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

[Diego Biurrun]

On Wed, Nov 11, 2009 at 03:52:44PM -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.

Really?  Well, it sure never looked that way to me.  If you are
interested in this, make ffserver.c compile without -DHAVE_AV_CONFIG_H.

Diego