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

[Måns Rullgård]

Diego Biurrun <diego at biurrun.de> writes:

> 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.

Michael says they shouldn't...

> All three use definitions from config.h all over the place...

Someone/something is wrong then...

-- 
M?ns Rullg?rd
mans at mansr.com