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

Chris Hegarty chris.hegarty at oracle.com
Wed Feb 22 11:16:29 UTC 2012


Kurchi,

Great work. I've been through the complete webrev and I think it looks 
good. Just a few minor comments:

  - there are API changes, but only in sun private implementation
    classes, so this should be fine.

  - Minor indentation nit where method declaration return type
    was generified. The method args on subsequent lines should be
    equally indented. Example LoaderHandler.java L152:

       public static Class<?> loadClass(String codebase, String name,
                               >>>>  ClassLoader defaultLoader)

  - There are opportunities to use auto boxing/unboxing

    >: diff RMIGenerator.java
    99c99
    <                     version = versionOptions.get(arg);
    ---
    >                     version = (versionOptions.get(arg)).intValue();

    ConnectionMultiplexer.java
    >: diff ConnectionMultiplexer.java ConnectionMultiplexer.java.1
    133a134
    <             Integer idObj;
    150c151,152
    >                     info = connectionTable.get(id);
    ---
    <                     idObj = new Integer(id);
    <                     info = connectionTable.get(idObj);
    158c160
    >                         connectionTable.put(id, info);
    ---
    <                         connectionTable.put(idObj, info);
    .....

-Chris.

On 22/02/2012 05:50, Kurchi Hazra wrote:
> Corrected the subject line.
>
>
> Hi,
>
> The following webrev removes warnings in sun.rmi.* packages. I have
> neglected nearly all
> deprecation warnings, since this code uses deprecated classes such as
> java.rmi.server.LogStream
> with no suggested replacements. I have included -Xlint:all,-deprecation
> as an option instead
> in the appropriate Makefiles.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7146763
> Webrev: http://cr.openjdk.java.net/~khazra/7146763/webrev.00/
>
>
> Thanks,
> Kurchi



More information about the core-libs-dev mailing list