Hello Mikhail,

 the fix looks fine to me.

Thanks,
Andrew

On 6/9/2014 10:32 PM, mikhail cherkasov wrote:
Hi all,

please review the fix:
http://cr.openjdk.java.net/~mcherkas/8041460/webrev.02/
for the following bug:
https://bugs.openjdk.java.net/browse/JDK-8041460

The problem is that during merging of metadata trees we take index of IHDR_bitDepths and set this value to IHDR_bitDepth, but IHDR_bitDepth requires a real number of bits. IHDR_bitDepths - store all valid values: "1", "2", "4", "8", "16", so to get
valid value we need: 1 << index.

Thanks,
Mikhail.


Reply via email to