RFR(s): 8060192: Add default method Collection.toArray(generator)
Stuart Marks
stuart.marks at oracle.com
Fri Dec 8 00:47:03 UTC 2017
On 12/7/17 3:50 PM, Jonathan Bluett-Duncan wrote:
> Looking over
> http://cr.openjdk.java.net/~smarks/reviews/8060192/webrev.2/src/java.base/share/classes/java/util/ArrayList.java.cdiff.html,
> I'm wondering if the method `ArrayList.toArray(IntFunction)` should have an
> `@Override` to make it clear that it's overriding the default method in
> Collection. What do you think?
I guess it wouldn't hurt. I had thought about adding it, but most of the methods
in ArrayList that override implementations from supertypes *don't* have
@Override. Looking more closely, it appears that the ones that override
interface default methods *do* have @Override.
I don't think there's a rule that says that @Override should be used when
overriding interface default methods but not when overriding implementations
from a superclass or when implementing an abstract interface method. Frankly, I
think the JDK is just sloppy here. Most of the existing implementations didn't
use @Override -- possibly because they were introduced before annotations
existed -- and later on, whoever implemented the overrides of the Java 8 default
methods decided to add @Override.
s'marks
More information about the core-libs-dev
mailing list