[FFmpeg-devel] [PATCH] interprete bits_per_raw_sample==0 as »use default bit depth«

Michael Niedermayer michaelni
Mon Aug 31 13:33:45 CEST 2009


On Mon, Aug 31, 2009 at 08:11:18AM +0200, Lars T?uber wrote:
> Hi,
> 
> with this patch the standard bit depth is taken for all the colorspaces used in ffv1 when bits_per_raw_sample==0.

you move code around and change it at the same time, this makes a
review hard, also you explain what you change but not why

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

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- 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/20090831/ae58e884/attachment.pgp>



More information about the ffmpeg-devel mailing list