RFR: 8276401: Use blessed modifier order in java.net.http

Mark Sheppard msheppar at openjdk.java.net
Wed Nov 3 14:30:10 UTC 2021


On Wed, 3 Nov 2021 10:11:31 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

> Hi, 
> 
> Please find here a trivial cleanup change that updates classes in the `java.net.http` module to use the "blessed modifier order".
> 
> The changeset was obtained by running `sh ./bin/blessed-modifier-order.sh src/java.net.http`.
> 
> best regards,
> 
> -- daniel

a general comment on the static abstract class  changes to abstract static class. For me the former and current declarations seems more appropriate, that is, static abstract class,  as the static modifier immediately conveys a significant and strong structural relationship with outer or containing class. While abstract has a qualification on class i.e. the type of class and appearing directly before class is more natural (to me !!). As such, abstract qualifies the static relationship.
The placement and ordering of the modifier should be to assist in covering semantics when scanning code, and conveys a certain level of "importance" of the qualifier's semantics

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

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


More information about the net-dev mailing list