Request for Review of 7116890 (Warning Cleanup java.io) (original) (raw)

Sebastian Sickelmann sebastian.sickelmann at gmx.de
Fri Dec 2 06:15:36 UTC 2011

Am 02.12.2011 01:22, schrieb Stuart Marks:

On 12/1/11 2:13 PM, Stuart Marks wrote:

On 12/1/11 12:38 PM, Alan Bateman wrote:

On 01/12/2011 18:35, Sebastian Sickelmann wrote:

: Thanks Alan, L67-68 was a backporting (from a more complex solution to a small warning cleanup) issue. I missed the type parameters here. I changed L119-120 also.

New webrev is here: http://dl.dropbox.com/u/43692695/oss-patches/openjdk8/WarningCleanupJavaio/CR71168901/index.html

Looks fine to me. Hi Sebastian! The change looks fine to me too. I'll apply the patch, build it, and push it in. Thanks for your contribution! Well, turns out this patch had an error in it. :-( In ExpiringCache.java the constructor that takes a long parameter has code that looks like this: map = new LinkedHashMap<>() { ... } It turns out that it's illegal to use diamond in the construction of an anonymous class! Didn't you compile the code, Sebastian? :-) To be fair, neither of us reviewers caught this either. The compiler is the ultimate reviewer in this case. I've taken the liberty of expanding the type parameters to map = new LinkedHashMap<String,Entry>() { ... } and I've gone ahead and pushed the changeset. Again, thanks for your contribution. s'marks Oh, i am sorry about that. I thought i had compiled it. But it looks like i have not.

Again, thanks for the nice cleanup event.

-- Sebastian

More information about the core-libs-dev mailing list