Review request for 6925868: Eliminate pack200's dependency on logging

Mandy Chung Mandy.Chung at Sun.COM
Fri Feb 12 16:13:22 UTC 2010


Rémi Forax wrote:
> No ok for me,

Sorry about putting an old webrev to cr.openjdk.java.net (forgot to 
update with the final one) .  In fact I fixed and ran the tests before 
sending out the review.
   http://cr.openjdk.java.net/~mchung/6925868/webrev.01/

Good spot.
Mandy

> Pack200Logger.warning(String) goes to an infinite loop,
> it should be:
>
> public void warning(String msg) {
>   getLogger().warning(msg);
> }
>
> Rémi
>
>




More information about the core-libs-dev mailing list