On Tue, 14 Sep 2021 13:15:18 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

>> I would leave it as an `@implNote` - or possibly `@implSpec`: depending on 
>> whether or not we want all implementations of the spec to behave in this 
>> way. However I don't think we would want to prevent subclasses from 
>> overriding this behavior and using their own business-logic ordering. So I 
>> believe the default behavior should be specified, if only so that subclasses 
>> can decide whether or not to override this method, without invalidating what 
>> subclasses might currently have implemented - or might wish to implement in 
>> the future. The CSR will be a good way to get feedback on whether 
>> `@implNote` or `@implSpec` is more appropriate. Also this is a change in 
>> behavior that needs to be made visible somewhere - and nobody reads release 
>> notes ;-)
>
> Done. I've updated the PR to include a `@implNote` specifying the order of 
> the properties. The CSR has been updated too to reflect this.

Thanks Jaikiran, - the `@implNote` looks good to me.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5372

Reply via email to