[FFmpeg-devel] IEC61937 compatible muxer

Michael Niedermayer michaelni
Wed Aug 19 12:08:10 CEST 2009


On Wed, Aug 19, 2009 at 10:59:04AM +0200, Bartlomiej Wolowiec wrote:
> Tuesday 18 August 2009 14:20:04 Michael Niedermayer napisa?(a):
> > >
> > >  Changelog                |    1
> > >  doc/general.texi         |    1
> > >  libavformat/Makefile     |    1
> > >  libavformat/allformats.c |    1
> > >  libavformat/avformat.h   |    2
> > >  libavformat/spdif.c      |  310
> > > +++++++++++++++++++++++++++++++++++++++++++++++ 6 files changed, 315
> > > insertions(+), 1 deletion(-)
> > > 5e1050033d862e1e5adcfc1cb8019129cc27a1cf  patch
> >
> > looks ok to me, given it works, has been tested and you add regression
> > tests
> >
> > [...]
> 
> I made some corrections suggested by Diego. 
> Yes, this code is working (not all MPEG options are tested, but they should 
> work), so should i commit this patch and make next commit with regression 
> tests or commit all at once ?

iam slightly in favor of 2 commits but thats very very slightly only

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090819/5fd690c5/attachment.pgp>



More information about the ffmpeg-devel mailing list