8200125: Fix some classloader/module typos

Reviewed-by: alanb
This commit is contained in:
Martin Buchholz 2018-04-05 09:33:27 -07:00
parent 97d7cfb14e
commit 184bbf01dd
8 changed files with 28 additions and 28 deletions

@ -699,7 +699,7 @@ public abstract class ClassLoader {
/**
* Finds the class with the given <a href="#binary-name">binary name</a>
* in a module defined to this class loader.
* Class loader implementations that support the loading from modules
* Class loader implementations that support loading from modules
* should override this method.
*
* @apiNote This method returns {@code null} rather than throwing
@ -1281,7 +1281,7 @@ public abstract class ClassLoader {
/**
* Returns a URL to a resource in a module defined to this class loader.
* Class loader implementations that support the loading from modules
* Class loader implementations that support loading from modules
* should override this method.
*
* @apiNote This method is the basis for the {@link
@ -1417,12 +1417,12 @@ public abstract class ClassLoader {
* @param name
* The resource name
*
* @return An enumeration of {@link java.net.URL URL} objects for
* the resource. If no resources could be found, the enumeration
* will be empty. Resources for which a {@code URL} cannot be
* constructed, are in package that is not opened unconditionally,
* or access to the resource is denied by the security manager,
* are not returned in the enumeration.
* @return An enumeration of {@link java.net.URL URL} objects for the
* resource. If no resources could be found, the enumeration will
* be empty. Resources for which a {@code URL} cannot be
* constructed, are in a package that is not opened
* unconditionally, or access to the resource is denied by the
* security manager, are not returned in the enumeration.
*
* @throws IOException
* If I/O errors occur

@ -152,7 +152,7 @@ public final class ModuleLayer {
private static final ModuleLayer EMPTY_LAYER
= new ModuleLayer(Configuration.empty(), List.of(), null);
// the configuration from which this ;ayer was created
// the configuration from which this layer was created
private final Configuration cf;
// parent layers, empty in the case of the empty layer
@ -498,7 +498,7 @@ public final class ModuleLayer {
try {
Loader loader = new Loader(cf.modules(), parentLoader);
loader.initRemotePackageMap(cf, parents);
ModuleLayer layer = new ModuleLayer(cf, parents, mn -> loader);
ModuleLayer layer = new ModuleLayer(cf, parents, mn -> loader);
return new Controller(layer);
} catch (IllegalArgumentException | IllegalStateException e) {
throw new LayerInstantiationException(e.getMessage());

@ -1411,7 +1411,7 @@ public final class URL implements java.io.Serializable {
// Check with factory if another thread set a
// factory since our last check
if (!checkedWithFactory && (fac = factory) != null) {
handler2 = fac.createURLStreamHandler(protocol);
handler2 = fac.createURLStreamHandler(protocol);
}
if (handler2 != null) {

@ -764,7 +764,7 @@ public class URLClassLoader extends SecureClassLoader implements Closeable {
path = ParseUtil.decode(path);
if (path.endsWith(File.separator))
path += "-";
p = new FilePermission(path, SecurityConstants.FILE_READ_ACTION);
p = new FilePermission(path, SecurityConstants.FILE_READ_ACTION);
} else {
/**
* Not loading from a 'file:' URL so we want to give the class

@ -100,8 +100,8 @@ public class BootLoader {
}
/**
* Register a module with this class loader so that its classes (and
* resources) become visible via this class loader.
* Registers a module with this class loader so that its classes
* (and resources) become visible via this class loader.
*/
public static void loadModule(ModuleReference mref) {
ClassLoaders.bootLoader().loadModule(mref);

@ -584,7 +584,7 @@ public class BuiltinClassLoader
}
/**
* A variation of {@code loadCass} to load a class with the specified
* A variation of {@code loadClass} to load a class with the specified
* binary name. This method returns {@code null} when the class is not
* found.
*/
@ -633,16 +633,16 @@ public class BuiltinClassLoader
}
/**
* A variation of {@code loadCass} to load a class with the specified
* A variation of {@code loadClass} to load a class with the specified
* binary name. This method returns {@code null} when the class is not
* found.
*/
protected Class<?> loadClassOrNull(String cn) {
protected Class<?> loadClassOrNull(String cn) {
return loadClassOrNull(cn, false);
}
/**
* Find the candidate loaded module for the given class name.
* Finds the candidate loaded module for the given class name.
* Returns {@code null} if none of the modules defined to this
* class loader contain the API package for the class.
*/
@ -656,7 +656,7 @@ public class BuiltinClassLoader
}
/**
* Find the candidate loaded module for the given class name
* Finds the candidate loaded module for the given class name
* in the named module. Returns {@code null} if the named module
* is not defined to this class loader or does not contain
* the API package for the class.
@ -832,8 +832,8 @@ public class BuiltinClassLoader
}
/**
* Get the Package with the specified package name. If defined
* then verify that it against the manifest and code source.
* Gets the Package with the specified package name. If defined
* then verifies it against the manifest and code source.
*
* @throws SecurityException if there is a sealing violation (JAR spec)
*/
@ -859,7 +859,7 @@ public class BuiltinClassLoader
/**
* Defines a new package in this ClassLoader. The attributes in the specified
* Manifest are use to get the package version and sealing information.
* Manifest are used to get the package version and sealing information.
*
* @throws IllegalArgumentException if the package name duplicates an
* existing package either in this class loader or one of its ancestors
@ -976,7 +976,7 @@ public class BuiltinClassLoader
// -- miscellaneous supporting methods
/**
* Returns the ModuleReader for the given module, creating it if needed
* Returns the ModuleReader for the given module, creating it if needed.
*/
private ModuleReader moduleReaderFor(ModuleReference mref) {
ModuleReader reader = moduleToReader.get(mref);

@ -90,7 +90,7 @@ public final class Loader extends SecureClassLoader {
ClassLoader.registerAsParallelCapable();
}
// the loader pool is in a pool, can be null
// the pool this loader is a member of; can be null
private final LoaderPool pool;
// parent ClassLoader, can be null
@ -487,7 +487,7 @@ public final class Loader extends SecureClassLoader {
}
/**
* Finds the class with the specified binary name in a given module.
* Finds the class with the specified binary name in the given module.
* This method returns {@code null} if the class cannot be found.
*/
@Override

@ -173,7 +173,7 @@ public class ModuleHashesBuilder {
}
/**
* Traverse this graph and performs the given action in topological order
* Traverses this graph and performs the given action in topological order.
*/
public void ordered(Consumer<T> action) {
TopoSorter<T> sorter = new TopoSorter<>(this);
@ -181,7 +181,7 @@ public class ModuleHashesBuilder {
}
/**
* Traverses this graph and performs the given action in reverse topological order
* Traverses this graph and performs the given action in reverse topological order.
*/
public void reverse(Consumer<T> action) {
TopoSorter<T> sorter = new TopoSorter<>(this);
@ -189,7 +189,7 @@ public class ModuleHashesBuilder {
}
/**
* Returns a transposed graph from this graph
* Returns a transposed graph from this graph.
*/
public Graph<T> transpose() {
Builder<T> builder = new Builder<>();