RFR: 8263855: Use the blessed modifier order in java.management/naming
Claes Redestad
redestad at openjdk.java.net
Fri Mar 19 14:13:40 UTC 2021
On Thu, 18 Mar 2021 18:26:20 GMT, Alex Blewitt <github.com+76791+alblue at openjdk.org> wrote:
> As with #2993 changing the order of `final static` to `static final` for the `java.management`, `java.management.rmi` and `java.naming` modules.
Overall good - found a few cases where the private modifier is in the wrong place that ought to be fixed as well.
src/java.naming/share/classes/com/sun/jndi/ldap/EventQueue.java line 48:
> 46: */
> 47: final class EventQueue implements Runnable {
> 48: static final private boolean debug = false;
Move private to the front?
src/java.naming/share/classes/com/sun/jndi/ldap/EventSupport.java line 116:
> 114: */
> 115: final class EventSupport {
> 116: static final private boolean debug = false;
Move private to the front?
src/java.naming/share/classes/com/sun/jndi/ldap/LdapSchemaCtx.java line 389:
> 387: }
> 388:
> 389: static final private class SchemaInfo {
Move private to the front?
-------------
Changes requested by redestad (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/3078
More information about the core-libs-dev
mailing list