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

Rémi Forax forax at
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:

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


More information about the core-libs-dev mailing list