RFR: 7308: Move non-Eclipse dependant classes from org.openjdk.jmc.ui.common to org.openjdk.jmc.common

Brice Dutheil github.com+803621+bric3 at openjdk.java.net
Tue Sep 14 12:43:06 UTC 2021


On Mon, 9 Aug 2021 21:30:22 GMT, Alex Macdonald <aptmac at openjdk.org> wrote:

> This PR addresses JMC-7308 [[0]](https://bugs.openjdk.java.net/browse/JMC-7308), in which it would be helpful to have some of the classes currently in jmc.ui.common shipped in core.
> 
> There are a number of classes currently in jmc.ui.common that would be a great asset to the core distribution (and the third-party applications that consume jmc-core), and these classes could live in jmc.common.
> 
> It isn't as straightforward as moving all of the packages to core, as there are still classes in these jmc.ui.common packages that have dependencies on Eclipse or rjmx. Having said that, the ones listed below can be moved without much difficulty:
> 
> - org.openjdk.jmc.ui.common.action (3)
>     Executable, IActionProvider, IUserAction
> 
> - org.openjdk.jmc.ui.common.jvm (5)
>     Connectable, JVMArch, JVMCommandLineToolkit, JVMDescriptor, JVMType
> 
> - org.openjdk.jmc.ui.common.resource (2)
>     IImageResource, Resource
> 
> - org.openjdk.jmc.ui.common.security (10)
>     ActionNotGrantedException, CredentialsNotAvailableException, FailedToSaveException, ICredentials, InMemoryCredentials, ISecurityManager, PersistentCredentials, SecurlyStoredByteArray, SecurityException, SecurityManagerFactory
> 
> - org.opendjk.jmc.ui.common.tree (3)
>     IArray, IChild, IParent
> 
> - org.openjdk.jmc.ui.common.util (4)
>     Environment, Filename, ICopyable, IObservable
> 
> - org.openjdk.jmc.ui.common.xydata (5)
>     DataSeries, DefautlTimestampedData, DefaultXYData, ITimeStampedData, IXYData
> 
> [0] https://bugs.openjdk.java.net/browse/JMC-7308

Marked as reviewed by bric3 at github.com (no known OpenJDK username).

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

PR: https://git.openjdk.java.net/jmc/pull/300


More information about the jmc-dev mailing list