[FFmpeg-devel] Prepwork for DVCPRO HD inclusion

Michael Niedermayer michaelni
Sun Aug 10 23:30:55 CEST 2008


On Sun, Aug 10, 2008 at 02:00:52PM -0700, Roman V. Shaposhnik wrote:
> On Sun, 2008-08-10 at 22:33 +0200, Michael Niedermayer wrote:
> > > > cosmetic and functional mix, please split the patch in a cosmetic and
> > > > a functional patch. This is very hard to decypher otherwise
> > > 
> > > Which part of this patch have you considered to be cosmetic? The
> > > indentation changes because of the extra loop (and that's functional)
> > > and the table name is changed because the table now has nothing
> > > to do with shifts.
> > 
> > the indention and whitespace change primarely, it makes reviewing hard because
> > i have to check each line if something changed or not ...
> 
> I guess the problem is -- even if indentation would stay the same the
> diff will pretty much look the same. There's not much left from the
> original code since the table's indicies got changed. I'm attaching
> the result of "diff -w", but as you can see the only lines that
> are now "intact" are comments. Please advise. 

ok, ive no objections to the patch

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

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- 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/20080810/7d829ffb/attachment.pgp>



More information about the ffmpeg-devel mailing list