[FFmpeg-devel] [PATCH] graph2dot tool

Stefano Sabatini stefano.sabatini-lala
Sun Feb 21 16:53:22 CET 2010


On date Friday 2010-01-08 03:40:45 +0100, Michael Niedermayer encoded:
> On Wed, Jan 06, 2010 at 09:02:10PM +0100, Stefano Sabatini wrote:
> > On date Monday 2009-12-28 13:08:16 +0100, Diego Biurrun encoded:
> > > On Sun, Dec 27, 2009 at 10:56:27PM +0100, Stefano Sabatini wrote:
> > > > 
> > > > static void usage()
> > > 
> > > usage(void)
> > 
> > Fixed, as well as other header inclusions / HAVE_AV_CONFIG_H issues.
> 
> i just wanted to say that iam in principle ok with putting this in tools.
> And i probably wont review it because its code quality is rather irrelevant
> to ffmpeg.

I removed the sink/source auto-insertion stuff, as I consider more
robust to have the user to explicitely specify sources and sinks, and
added some documentation for it.

> Besides, since when is C the ideal choice for parsing stuff ?

I'm just using C for printing stuff, the parsing is done by dot.

Regards.
-- 
FFmpeg = Frenzy Foolish Mere Power EniGma
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-graph2dot-tool.patch
Type: text/x-diff
Size: 7581 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100221/09ddfcf9/attachment.patch>



More information about the ffmpeg-devel mailing list