RFR: 8231640: (prop) Canonical property storage [v7]
Jaikiran Pai
jpai at openjdk.java.net
Sat Sep 18 03:52:18 UTC 2021
On Fri, 17 Sep 2021 17:10:10 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> "different set implementation" as part of the spec may challenge the compatibility test developers to prove or disprove that statement.
>> The type of an instance is frequently understood to be the "implementation".
>> The visible type returned from Properties.entrySet() is SynchronizedSet.
>> Anyone can create one of those.
>> That statement might mislead a subclass into thinking they can not/must not return a SynchronizedSet if they want the built-in sorting.
>> I'm thinking that wording it in other term of the subclass might be better:
>> "...sort order of the keys in the entrySet() unless entrySet() is overridden by a subclass to return a different value than 'super.entrySet'(). "
>> The existing implementation is fine.
>
> I agree that this is a better formulation than what I suggested :-)
Done. I've updated the PR to use Roger's suggested text and yes it's much more precise than what we had so far.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5372
More information about the core-libs-dev
mailing list