Code review request: 7146763: Warnings cleanup in the sun.rmi and related packages

Rémi Forax forax at univ-mlv.fr
Fri Feb 24 21:45:08 UTC 2012


On 02/24/2012 08:01 PM, Kurchi Hazra wrote:
> Hi Stuart,
>
>    Thanks for the detailed explanation. Here is an updated webrev:
> http://cr.openjdk.java.net/~khazra/7146763/webrev.02/

Hi Kurchi,
the field logClassConstructor should not be changed after all,
it should be declared as a Constructor<? extends LogFile> so you can remove
the cast to LogFile at line 542.

with this change, the patch looks good to me.

>
>
> - Kurchi

cheers,
Rémi




More information about the core-libs-dev mailing list