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