Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8276348: Use blessed modifier order in java.base
Reviewed-by: dfuchs, darcy, iris, rriggs, martin
  • Loading branch information
pavelrappo committed Nov 3, 2021
1 parent 465d350 commit 6150633
Show file tree
Hide file tree
Showing 21 changed files with 39 additions and 39 deletions.
4 changes: 2 additions & 2 deletions src/java.base/share/classes/java/io/ObjectInputFilter.java
Expand Up @@ -561,7 +561,7 @@ final class Config {
/**
* Lock object for filter and filter factory.
*/
private final static Object serialFilterLock = new Object();
private static final Object serialFilterLock = new Object();

/**
* The property name for the filter.
Expand Down Expand Up @@ -906,7 +906,7 @@ static ObjectInputFilter createFilter2(String pattern) {
* used for all ObjectInputStreams that do not set their own filters.
*
*/
final static class Global implements ObjectInputFilter {
static final class Global implements ObjectInputFilter {
/**
* The pattern used to create the filter.
*/
Expand Down
2 changes: 1 addition & 1 deletion src/java.base/share/classes/java/io/ObjectStreamClass.java
Expand Up @@ -2515,7 +2515,7 @@ synchronized MethodHandle putIfAbsentAndGet(ObjectStreamField[] fields, MethodHa
}

// a key composed of ObjectStreamField[] names and types
static abstract class Key {
abstract static class Key {
abstract int length();
abstract String fieldName(int i);
abstract Class<?> fieldType(int i);
Expand Down
2 changes: 1 addition & 1 deletion src/java.base/share/classes/java/lang/Object.java
Expand Up @@ -278,7 +278,7 @@ public String toString() {
* <li>By executing the body of a {@code synchronized} statement
* that synchronizes on the object.
* <li>For objects of type {@code Class,} by executing a
* synchronized static method of that class.
* static synchronized method of that class.
* </ul>
* <p>
* Only one thread at a time can own an object's monitor.
Expand Down
2 changes: 1 addition & 1 deletion src/java.base/share/classes/java/lang/Process.java
Expand Up @@ -845,7 +845,7 @@ public long skip(long n) throws IOException {
* A nested class to delay looking up the Charset for the native encoding.
*/
private static class CharsetHolder {
private final static Charset nativeCharset;
private static final Charset nativeCharset;
static {
Charset cs;
try {
Expand Down
6 changes: 3 additions & 3 deletions src/java.base/share/classes/java/lang/StackStreamFactory.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -118,7 +118,7 @@ enum WalkerState {
* For example, StackFrameInfo for StackWalker::walk or
* Class<?> for StackWalker::getCallerClass
*/
static abstract class AbstractStackWalker<R, T> {
abstract static class AbstractStackWalker<R, T> {
protected final StackWalker walker;
protected final Thread thread;
protected final int maxDepth;
Expand Down Expand Up @@ -783,7 +783,7 @@ protected void initFrameBuffer() {
*
* Each specialized AbstractStackWalker subclass may subclass the FrameBuffer.
*/
static abstract class FrameBuffer<F> {
abstract static class FrameBuffer<F> {
static final int START_POS = 2; // 0th and 1st elements are reserved

// buffers for VM to fill stack frame info
Expand Down
4 changes: 2 additions & 2 deletions src/java.base/share/classes/java/lang/System.java
Expand Up @@ -331,7 +331,7 @@ private static void checkIO() {
private static class CallersHolder {
// Remember callers of setSecurityManager() here so that warning
// is only printed once for each different caller
final static Map<Class<?>, Boolean> callers
static final Map<Class<?>, Boolean> callers
= Collections.synchronizedMap(new WeakHashMap<>());
}

Expand Down Expand Up @@ -1595,7 +1595,7 @@ public void log(Level level, ResourceBundle bundle, String format,
*
* @since 9
*/
public static abstract class LoggerFinder {
public abstract static class LoggerFinder {
/**
* The {@code RuntimePermission("loggerFinder")} is
* necessary to subclass and instantiate the {@code LoggerFinder} class,
Expand Down
4 changes: 2 additions & 2 deletions src/java.base/share/classes/java/lang/WeakPairMap.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2016, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -340,7 +340,7 @@ public boolean equals(Object obj) {
/**
* Common abstract supertype of a pair of WeakReference peers.
*/
private static abstract class WeakRefPeer<K> extends WeakReference<K> {
private abstract static class WeakRefPeer<K> extends WeakReference<K> {

WeakRefPeer(K k, ReferenceQueue<Object> queue) {
super(Objects.requireNonNull(k), queue);
Expand Down
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2017, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -244,7 +244,7 @@ public Iterator<Object> iterator() {
}
}

static abstract
abstract static
class WithCache extends AbstractConstantGroup {
@Stable final Object[] cache;

Expand Down
6 changes: 3 additions & 3 deletions src/java.base/share/classes/java/lang/invoke/CallSite.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2008, 2016, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2008, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -84,8 +84,8 @@ private static CallSite bootstrapDynamic(MethodHandles.Lookup caller, String nam
* @author John Rose, JSR 292 EG
* @since 1.7
*/
abstract
public class CallSite {
public
abstract class CallSite {

// The actual payload of this call site.
// Can be modified using {@link MethodHandleNatives#setCallSiteTargetNormal} or {@link MethodHandleNatives#setCallSiteTargetVolatile}.
Expand Down
2 changes: 1 addition & 1 deletion src/java.base/share/classes/java/net/InetAddress.java
Expand Up @@ -215,7 +215,7 @@ public class InetAddress implements java.io.Serializable {
@Native static final int IPv6 = 2;

/* Specify address family preference */
static transient final int preferIPv6Address;
static final transient int preferIPv6Address;

static class InetAddressHolder {
/**
Expand Down
2 changes: 1 addition & 1 deletion src/java.base/share/classes/java/security/Provider.java
Expand Up @@ -820,7 +820,7 @@ private void check(String directive) {
// legacy properties changed since last call to any services method?
private transient boolean legacyChanged;
// serviceMap changed since last call to getServices()
private volatile transient boolean servicesChanged;
private transient volatile boolean servicesChanged;

// Map<String,String> used to keep track of legacy registration
private transient Map<String,String> legacyStrings;
Expand Down
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2016, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2016, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -142,7 +142,7 @@ private ImmutableCollections() { }
static UnsupportedOperationException uoe() { return new UnsupportedOperationException(); }

@jdk.internal.ValueBased
static abstract class AbstractImmutableCollection<E> extends AbstractCollection<E> {
abstract static class AbstractImmutableCollection<E> extends AbstractCollection<E> {
// all mutating methods throw UnsupportedOperationException
@Override public boolean add(E e) { throw uoe(); }
@Override public boolean addAll(Collection<? extends E> c) { throw uoe(); }
Expand Down Expand Up @@ -249,7 +249,7 @@ static <E> List<E> listFromTrustedArrayNullsAllowed(Object... input) {
// ---------- List Implementations ----------

@jdk.internal.ValueBased
static abstract class AbstractImmutableList<E> extends AbstractImmutableCollection<E>
abstract static class AbstractImmutableList<E> extends AbstractImmutableCollection<E>
implements List<E>, RandomAccess {

// all mutating methods throw UnsupportedOperationException
Expand Down Expand Up @@ -742,7 +742,7 @@ public int lastIndexOf(Object o) {
// ---------- Set Implementations ----------

@jdk.internal.ValueBased
static abstract class AbstractImmutableSet<E> extends AbstractImmutableCollection<E>
abstract static class AbstractImmutableSet<E> extends AbstractImmutableCollection<E>
implements Set<E> {

@Override
Expand Down
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2019, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2019, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -624,7 +624,7 @@ public final int toBinary(OutputStream os) {
static final int NO_INDEX3_NULL_OFFSET = 0x7fff;
static final int NO_DATA_NULL_OFFSET = 0xfffff;

private static abstract class Data {
private abstract static class Data {
abstract ValueWidth getValueWidth();
abstract int getDataLength();
abstract int getFromIndex(int index);
Expand Down Expand Up @@ -768,7 +768,7 @@ private final int internalSmallIndex(Type type, int c) {
*
* @stable ICU 63
*/
public static abstract class Fast extends CodePointTrie {
public abstract static class Fast extends CodePointTrie {
private Fast(char[] index, Data data, int highStart,
int index3NullOffset, int dataNullOffset) {
super(index, data, highStart, index3NullOffset, dataNullOffset);
Expand Down Expand Up @@ -906,7 +906,7 @@ public boolean previous() {
*
* @stable ICU 63
*/
public static abstract class Small extends CodePointTrie {
public abstract static class Small extends CodePointTrie {
private Small(char[] index, Data data, int highStart,
int index3NullOffset, int dataNullOffset) {
super(index, data, highStart, index3NullOffset, dataNullOffset);
Expand Down
Expand Up @@ -61,11 +61,11 @@ public Boolean run() {
});
}

static private final boolean IS_64_BIT =
private static final boolean IS_64_BIT =
isSystemProperty("sun.arch.data.model", "64", "32");
static private final boolean USE_JVM_MAP =
private static final boolean USE_JVM_MAP =
isSystemProperty("jdk.image.use.jvm.map", "true", "true");
static private final boolean MAP_ALL =
private static final boolean MAP_ALL =
isSystemProperty("jdk.image.map.all", "true", IS_64_BIT ? "true" : "false");

private final Path imagePath;
Expand Down
Expand Up @@ -45,5 +45,5 @@ public Void run() {
});
}

native static ByteBuffer getNativeMap(String imagePath);
static native ByteBuffer getNativeMap(String imagePath);
}
Expand Up @@ -437,7 +437,7 @@ static final class Formatting {
getSimpleFormat(DEFAULT_FORMAT_PROP_KEY, null);

// Make it easier to wrap Logger...
static private final String[] skips;
private static final String[] skips;
static {
String additionalPkgs =
GetPropertyAction.privilegedGetProperty("jdk.logger.packages");
Expand Down
Expand Up @@ -121,7 +121,7 @@ static ModuleReference newExplodedModule(ModuleInfo.Attributes attrs,
* A base module reader that encapsulates machinery required to close the
* module reader safely.
*/
static abstract class SafeCloseModuleReader implements ModuleReader {
abstract static class SafeCloseModuleReader implements ModuleReader {

// RW lock to support safe close
private final ReadWriteLock lock = new ReentrantReadWriteLock();
Expand Down
Expand Up @@ -888,7 +888,7 @@ private int writeInternal(ByteBuffer src, long position) throws IOException {

// -- Memory-mapped buffers --

private static abstract class Unmapper
private abstract static class Unmapper
implements Runnable, UnmapperProxy
{
// may be required to close file
Expand Down
Expand Up @@ -71,7 +71,7 @@ public final class ProviderList {
static final ProviderList EMPTY = new ProviderList(PC0, true);

// list of all jdk.security.provider.preferred entries
static private PreferredList preferredPropList = null;
private static PreferredList preferredPropList = null;

// dummy provider object to use during initialization
// used to avoid explicit null checks in various places
Expand Down
Expand Up @@ -60,12 +60,12 @@ public final class RSAPrivateKeyImpl extends PKCS8Key implements RSAPrivateKey {
private final BigInteger n; // modulus
private final BigInteger d; // private exponent

private transient final KeyType type;
private final transient KeyType type;

// optional parameters associated with this RSA key
// specified in the encoding of its AlgorithmId.
// must be null for "RSA" keys.
private transient final AlgorithmParameterSpec keyParams;
private final transient AlgorithmParameterSpec keyParams;

/**
* Construct a key from its components. Used by the
Expand Down
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2006, 2019, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2006, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -550,7 +550,7 @@ static final void kickstart(HandshakeContext context) throws IOException {
/**
* A (transparent) specification of handshake message.
*/
static abstract class HandshakeMessage {
abstract static class HandshakeMessage {
final HandshakeContext handshakeContext;

HandshakeMessage(HandshakeContext handshakeContext) {
Expand Down

1 comment on commit 6150633

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.