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

Alex Macdonald aptmac at openjdk.org
Tue May 23 15:27:49 UTC 2023


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

This pull request has now been integrated.

Changeset: ff2fba44
Author:    Alex Macdonald <aptmac at openjdk.org>
URL:       https://git.openjdk.org/jmc/commit/ff2fba4492009419761e85ae95bfcfd0ced82f60
Stats:     420 lines in 116 files changed: 97 ins; 119 del; 204 mod

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

Reviewed-by: bdutheil, clanger

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

PR: https://git.openjdk.org/jmc/pull/300


More information about the jmc-dev mailing list