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