Code review request: 7146763: Warnings cleanup in the sun.rmi and related packages
Darryl Mocek
darryl.mocek at oracle.com
Wed Feb 22 18:14:21 UTC 2012
Hi Kurchi,
overall the changes look good. I have a few comments:
- TCPTransport: Lines #552/553: I actually prefer the way it was as I
think it's more readable.
- RMIMasterSocketFactory: Line #244: The comment was removed, was there
a reason for this?
- HttpInputStream: Line #73: Why even have this if statement? I'd make
this a comment instead, something like if contentLengthFound, we should
probably do something in this case.
- CGIHandler: Line #288: Why even have this if statement? I'd make this
a comment instead, something like if contentLengthFound, we should
probably do something in this case.
Darryl
On 02/21/2012 09:50 PM, 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