Skip to content

Commit

Permalink
8314738: Remove all occurrences of and support for @Revised
Browse files Browse the repository at this point in the history
Reviewed-by: mr
  • Loading branch information
pavelrappo committed Aug 22, 2023
1 parent 6b9df03 commit f39fc0a
Show file tree
Hide file tree
Showing 28 changed files with 8 additions and 124 deletions.
1 change: 0 additions & 1 deletion make/Docs.gmk
Expand Up @@ -67,7 +67,6 @@ MODULES_SOURCE_PATH := $(call PathList, $(call GetModuleSrcPath) )
# ordering of tags as the tags are otherwise ordered in order of definition.
JAVADOC_TAGS := \
-tag beaninfo:X \
-tag revised:X \
-tag since.unbundled:X \
-tag Note:X \
-tag ToDo:X \
Expand Down
2 changes: 0 additions & 2 deletions src/java.base/share/classes/java/io/FileInputStream.java
Expand Up @@ -499,8 +499,6 @@ public int available() throws IOException {
* this method should be prepared to handle possible reentrant invocation.
*
* @throws IOException {@inheritDoc}
*
* @revised 1.4
*/
@Override
public void close() throws IOException {
Expand Down
2 changes: 0 additions & 2 deletions src/java.base/share/classes/java/io/FileOutputStream.java
Expand Up @@ -392,8 +392,6 @@ public void write(byte[] b, int off, int len) throws IOException {
* this method should be prepared to handle possible reentrant invocation.
*
* @throws IOException if an I/O error occurs.
*
* @revised 1.4
*/
@Override
public void close() throws IOException {
Expand Down
2 changes: 0 additions & 2 deletions src/java.base/share/classes/java/io/InputStreamReader.java
Expand Up @@ -157,8 +157,6 @@ public InputStreamReader(InputStream in, CharsetDecoder dec) {
* {@code null} if the stream has been closed
*
* @see Charset
*
* @revised 1.4
*/
public String getEncoding() {
return sd.getEncoding();
Expand Down
2 changes: 0 additions & 2 deletions src/java.base/share/classes/java/io/OutputStreamWriter.java
Expand Up @@ -176,8 +176,6 @@ public OutputStreamWriter(OutputStream out, CharsetEncoder enc) {
* {@code null} if the stream has been closed
*
* @see Charset
*
* @revised 1.4
*/
public String getEncoding() {
return se.getEncoding();
Expand Down
4 changes: 0 additions & 4 deletions src/java.base/share/classes/java/io/RandomAccessFile.java
Expand Up @@ -129,7 +129,6 @@ public class RandomAccessFile implements DataOutput, DataInput, Closeable {
* @see java.lang.SecurityException
* @see java.lang.SecurityManager#checkRead(java.lang.String)
* @see java.lang.SecurityManager#checkWrite(java.lang.String)
* @revised 1.4
*/
public RandomAccessFile(String name, String mode)
throws FileNotFoundException
Expand Down Expand Up @@ -215,7 +214,6 @@ public RandomAccessFile(String name, String mode)
* @see java.lang.SecurityManager#checkRead(java.lang.String)
* @see java.lang.SecurityManager#checkWrite(java.lang.String)
* @see java.nio.channels.FileChannel#force(boolean)
* @revised 1.4
*/
public RandomAccessFile(File file, String mode)
throws FileNotFoundException
Expand Down Expand Up @@ -703,8 +701,6 @@ public void setLength(long newLength) throws IOException {
* this method should be prepared to handle possible reentrant invocation.
*
* @throws IOException if an I/O error occurs.
*
* @revised 1.4
*/
public void close() throws IOException {
if (closed) {
Expand Down
3 changes: 0 additions & 3 deletions src/java.base/share/classes/java/lang/Class.java
Expand Up @@ -1168,7 +1168,6 @@ public Type getGenericSuperclass() {
* this method returns {@code null}.
*
* @return the package of this class.
* @revised 9
*/
public Package getPackage() {
if (isPrimitive() || isArray()) {
Expand Down Expand Up @@ -3029,7 +3028,6 @@ public Constructor<T> getDeclaredConstructor(Class<?>... parameterTypes)
*
* @see Module#getResourceAsStream(String)
* @since 1.1
* @revised 9
*/
@CallerSensitive
public InputStream getResourceAsStream(String name) {
Expand Down Expand Up @@ -3125,7 +3123,6 @@ public InputStream getResourceAsStream(String name) {
* manager.
* @throws NullPointerException If {@code name} is {@code null}
* @since 1.1
* @revised 9
*/
@CallerSensitive
public URL getResource(String name) {
Expand Down
19 changes: 0 additions & 19 deletions src/java.base/share/classes/java/lang/ClassLoader.java
Expand Up @@ -227,7 +227,6 @@
* @jls 13.1 The Form of a Binary
* @see #resolveClass(Class)
* @since 1.0
* @revised 9
*/
public abstract class ClassLoader {

Expand Down Expand Up @@ -881,7 +880,6 @@ protected final Class<?> defineClass(byte[] b, int off, int len)
* @see java.security.SecureClassLoader
*
* @since 1.1
* @revised 9
*/
protected final Class<?> defineClass(String name, byte[] b, int off, int len)
throws ClassFormatError
Expand Down Expand Up @@ -1015,8 +1013,6 @@ private void postDefineClass(Class<?> c, ProtectionDomain pd) {
* certificates than this class, or if {@code name} begins with
* "{@code java.}" and this class loader is not the platform
* class loader or its ancestor.
*
* @revised 9
*/
protected final Class<?> defineClass(String name, byte[] b, int off, int len,
ProtectionDomain protectionDomain)
Expand Down Expand Up @@ -1091,7 +1087,6 @@ protected final Class<?> defineClass(String name, byte[] b, int off, int len,
* @see #defineClass(String, byte[], int, int, ProtectionDomain)
*
* @since 1.5
* @revised 9
*/
protected final Class<?> defineClass(String name, java.nio.ByteBuffer b,
ProtectionDomain protectionDomain)
Expand Down Expand Up @@ -1404,7 +1399,6 @@ protected URL findResource(String moduleName, String name) throws IOException {
* @throws NullPointerException If {@code name} is {@code null}
*
* @since 1.1
* @revised 9
*/
public URL getResource(String name) {
Objects.requireNonNull(name);
Expand Down Expand Up @@ -1469,7 +1463,6 @@ public URL getResource(String name) {
* @throws NullPointerException If {@code name} is {@code null}
*
* @since 1.2
* @revised 9
*/
public Enumeration<URL> getResources(String name) throws IOException {
Objects.requireNonNull(name);
Expand Down Expand Up @@ -1567,7 +1560,6 @@ public Stream<URL> resources(String name) {
* denied by the security manager.
*
* @since 1.2
* @revised 9
*/
protected URL findResource(String name) {
return null;
Expand Down Expand Up @@ -1602,7 +1594,6 @@ protected URL findResource(String name) {
* If I/O errors occur
*
* @since 1.2
* @revised 9
*/
protected Enumeration<URL> findResources(String name) throws IOException {
return Collections.emptyEnumeration();
Expand Down Expand Up @@ -1687,7 +1678,6 @@ public final boolean isRegisteredAsParallelCapable() {
* denied by the security manager.
*
* @since 1.1
* @revised 9
*/
public static URL getSystemResource(String name) {
return getSystemClassLoader().getResource(name);
Expand Down Expand Up @@ -1723,7 +1713,6 @@ public static URL getSystemResource(String name) {
* If I/O errors occur
*
* @since 1.2
* @revised 9
*/
public static Enumeration<URL> getSystemResources(String name)
throws IOException
Expand Down Expand Up @@ -1755,7 +1744,6 @@ public static Enumeration<URL> getSystemResources(String name)
* @throws NullPointerException If {@code name} is {@code null}
*
* @since 1.1
* @revised 9
*/
public InputStream getResourceAsStream(String name) {
Objects.requireNonNull(name);
Expand Down Expand Up @@ -1788,7 +1776,6 @@ public InputStream getResourceAsStream(String name) {
* denied by the security manager.
*
* @since 1.1
* @revised 9
*/
public static InputStream getSystemResourceAsStream(String name) {
URL url = getSystemResource(name);
Expand Down Expand Up @@ -1948,9 +1935,6 @@ public static ClassLoader getPlatformClassLoader() {
* exception is thrown by that constructor when it is invoked. The
* underlying cause of the error can be retrieved via the
* {@link Throwable#getCause()} method.
*
* @revised 1.4
* @revised 9
*/
@CallerSensitive
public static ClassLoader getSystemClassLoader() {
Expand Down Expand Up @@ -2217,7 +2201,6 @@ private Package toPackage(String name, NamedPackage p, Module m) {
*
*
* @since 1.2
* @revised 9
*
* @jvms 5.3 Creation and Loading
* @see <a href="{@docRoot}/../specs/jar/jar.html#package-sealing">
Expand Down Expand Up @@ -2326,7 +2309,6 @@ public final Package[] getDefinedPackages() {
* @see ClassLoader#getDefinedPackage(String)
*
* @since 1.2
* @revised 9
*/
@Deprecated(since="9")
protected Package getPackage(String name) {
Expand Down Expand Up @@ -2361,7 +2343,6 @@ protected Package getPackage(String name) {
* @see ClassLoader#getDefinedPackages()
*
* @since 1.2
* @revised 9
*/
protected Package[] getPackages() {
Stream<Package> pkgs = packages();
Expand Down
9 changes: 1 addition & 8 deletions src/java.base/share/classes/java/lang/Package.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 1997, 2021, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 1997, 2023, 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 @@ -114,7 +114,6 @@
* @see ClassLoader#definePackage(String, String, String, String, String, String, String, URL)
*
* @since 1.2
* @revised 9
*/
public class Package extends NamedPackage implements java.lang.reflect.AnnotatedElement {
/**
Expand Down Expand Up @@ -211,8 +210,6 @@ public String getImplementationVersion() {
* is returned if it is not known.
* @return the vendor that implemented this package, {@code null}
* is returned if it is not known.
*
* @revised 9
*/
public String getImplementationVendor() {
return versionInfo.implVendor;
Expand Down Expand Up @@ -355,8 +352,6 @@ public boolean isCompatibleWith(String desired)
* a {@code Package} for the specified class loader.
*
* @see ClassLoader#getDefinedPackage
*
* @revised 9
*/
@CallerSensitive
@Deprecated(since="9")
Expand All @@ -379,8 +374,6 @@ public static Package getPackage(String name) {
* class loader and its ancestors
*
* @see ClassLoader#getDefinedPackages
*
* @revised 9
*/
@CallerSensitive
public static Package[] getPackages() {
Expand Down
6 changes: 1 addition & 5 deletions src/java.base/share/classes/java/lang/StackTraceElement.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2000, 2022, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2000, 2023, 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,6 @@ public final class StackTraceElement implements java.io.Serializable {
* @throws NullPointerException if {@code declaringClass} or
* {@code methodName} is null
* @since 1.5
* @revised 9
*/
public StackTraceElement(String declaringClass, String methodName,
String fileName, int lineNumber) {
Expand Down Expand Up @@ -355,7 +354,6 @@ public boolean isNativeMethod() {
* {@link java.lang.StackWalker.StackFrame}, where an implementation may
* choose to omit some element in the returned string.
*
* @revised 9
* @see Throwable#printStackTrace()
*/
@Override
Expand Down Expand Up @@ -426,8 +424,6 @@ private static int length(String s) {
* @return true if the specified object is another
* {@code StackTraceElement} instance representing the same
* execution point as this instance.
*
* @revised 9
*/
public boolean equals(Object obj) {
if (obj==this)
Expand Down
4 changes: 0 additions & 4 deletions src/java.base/share/classes/java/lang/Thread.java
Expand Up @@ -1693,8 +1693,6 @@ public final void stop() {
*
* @throws SecurityException
* if the current thread cannot modify this thread
*
* @revised 6.0, 14
*/
public void interrupt() {
if (this != Thread.currentThread()) {
Expand Down Expand Up @@ -1726,7 +1724,6 @@ public void interrupt() {
* @return {@code true} if the current thread has been interrupted;
* {@code false} otherwise.
* @see #isInterrupted()
* @revised 6.0, 14
*/
public static boolean interrupted() {
return currentThread().getAndClearInterrupt();
Expand All @@ -1739,7 +1736,6 @@ public static boolean interrupted() {
* @return {@code true} if this thread has been interrupted;
* {@code false} otherwise.
* @see #interrupted()
* @revised 6.0, 14
*/
public boolean isInterrupted() {
return interrupted;
Expand Down
Expand Up @@ -175,8 +175,6 @@ private static Lookup lookup(Class<?> caller) {
* Also, it cannot access
* <a href="MethodHandles.Lookup.html#callsens">caller sensitive methods</a>.
* @return a lookup object which is trusted minimally
*
* @revised 9
*/
public static Lookup publicLookup() {
return Lookup.PUBLIC_LOOKUP;
Expand Down Expand Up @@ -1437,8 +1435,6 @@ public static <T extends Member> T reflectAs(Class<T> expected, MethodHandle tar
* so that there can be a secure foundation for lookups.
* Nearly all other methods in the JSR 292 API rely on lookup
* objects to check access requests.
*
* @revised 9
*/
public static final
class Lookup {
Expand Down Expand Up @@ -1621,8 +1617,6 @@ private Class<?> lookupClassOrNull() {
* @return the lookup modes, which limit the kinds of access performed by this lookup object
* @see #in
* @see #dropLookupMode
*
* @revised 9
*/
public int lookupModes() {
return allowedModes & ALL_MODES;
Expand Down Expand Up @@ -1703,7 +1697,6 @@ private static Lookup newLookup(Class<?> lookupClass, Class<?> prevLookupClass,
* @throws IllegalArgumentException if {@code requestedLookupClass} is a primitive type or void or array class
* @throws NullPointerException if the argument is null
*
* @revised 9
* @see #accessClass(Class)
* @see <a href="#cross-module-lookup">Cross-module lookups</a>
*/
Expand Down Expand Up @@ -2599,8 +2592,6 @@ private static void checkUnprivilegedlookupClass(Class<?> lookupClass) {
* because it requires a direct subclass relationship between
* caller and callee.)
* @see #in
*
* @revised 9
*/
@Override
public String toString() {
Expand Down
Expand Up @@ -74,7 +74,6 @@
* @spec jni/index.html Java Native Interface Specification
* @jls 6.6 Access Control
* @since 1.2
* @revised 9
*/
public class AccessibleObject implements AnnotatedElement {
static {
Expand Down Expand Up @@ -119,7 +118,6 @@ static void checkPermission() {
* java.lang.Class}
* @see SecurityManager#checkPermission
* @see ReflectPermission
* @revised 9
*/
@CallerSensitive
public static void setAccessible(AccessibleObject[] array, boolean flag) {
Expand Down Expand Up @@ -207,7 +205,6 @@ public static void setAccessible(AccessibleObject[] array, boolean flag) {
* @spec jni/index.html Java Native Interface Specification
* @see #trySetAccessible
* @see java.lang.invoke.MethodHandles#privateLookupIn
* @revised 9
*/
@CallerSensitive // overrides in Method/Field/Constructor are @CS
public void setAccessible(boolean flag) {
Expand Down Expand Up @@ -424,8 +421,6 @@ String toShortString() {
* This method may return {@code false} on a reflected object that is
* accessible to the caller. To test if this reflected object is accessible,
* it should use {@link #canAccess(Object)}.
*
* @revised 9
*/
@Deprecated(since="9")
public boolean isAccessible() {
Expand Down

1 comment on commit f39fc0a

@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.