[FFmpeg-devel] [PATCH] add _XOPEN_SOURCE definition to tools/trasher.c

Diego Biurrun diego
Tue Mar 24 17:31:46 CET 2009


On Tue, Mar 24, 2009 at 05:17:19PM +0100, Michael Niedermayer wrote:
> On Tue, Mar 24, 2009 at 08:15:26AM +0100, Diego Biurrun wrote:
> > On Tue, Mar 24, 2009 at 01:29:32AM +0100, Michael Niedermayer wrote:
> > > On Tue, Mar 24, 2009 at 12:01:04AM +0100, Diego Biurrun wrote:
> > > > On Mon, Mar 23, 2009 at 10:32:16PM +0000, M?ns Rullg?rd wrote:
> > > > > Diego Biurrun <diego at biurrun.de> writes:
> > > > > 
> > > > > > When compiling tools/trasher.c the following warnings appear:
> > > > > >
> > > > > > tools/trasher.c: In function 'main':
> > > > > > tools/trasher.c:44: warning: implicit declaration of function 'srandom'
> > > > > > tools/trasher.c:51: warning: implicit declaration of function 'random'
> > > > > >
> > > > > > According to
> > > > > >
> > > > > > http://www.opengroup.org/onlinepubs/009695399/functions/srandom.html
> > > > > >
> > > > > > (s)random is an XSI extension, so adding
> > > > > >
> > > > > > #define _XOPEN_SOURCE 600
> > > > > >
> > > > > > to the file seems to be the correct solution.
> > > > > 
> > > > > You attached an empty file.  I don't think I've seen that before.  And
> > > > > in the same breath as commenting on somebody else's lack of patch too...
> > > > 
> > > > Clearly, I was destined to come up with clever new ways to botch patch
> > > > attachment.  Just forgetting it for the 999th time would not give
> > > > anybody comic relief...
> > > 
> > > you could post the patch and attach the reply next time
> > 
> > :)
> > 
> > > about the patch, why not use lfg lcg or any other generate we have?
> > 
> > It would considerably bloat the size of the program.  I think the idea
> > is for those tiny tools to be standalone.  random() seems to be good
> > enough for this use case.
> 
> what about marsaglias KISS99 generator or even
> a LCG, that would just be state = state*C0 + C1
> 
> random() has the disadvantage that it returns different values on different
> platorms, making debuging and reproducing issues harder.

It's your program, improve it whichever way you want.  I'm not
particularly interested in in doing that, I just want the build system
part of it to be correct.

Thus, I have committed my trivial and obviously correct patch.

Diego



More information about the ffmpeg-devel mailing list