RFR: 7308: Move non-Eclipse dependant classes from org.openjdk.jmc.ui.common to org.openjdk.jmc.common [v6]
Alex Macdonald
aptmac at openjdk.org
Fri May 19 21:18:01 UTC 2023
On Wed, 17 May 2023 09:37:10 GMT, Brice Dutheil <bdutheil at openjdk.org> wrote:
>> Alex Macdonald has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 18 commits:
>>
>> - add last missing copyright year update
>> - add missing license year updates
>> - Merge branch 'master' of https://git.openjdk.org/jmc into PR-7308
>> - fix RjmxTestCase imports
>> - update license headers to 2023
>> - Merge master
>> - fix imports for PersistentCredentials
>>
>> I must have missed this when I rebased onto the master branch. Fixing it
>> up here.
>> - update license headers to 2022
>> - update license headers
>> - update agent plugin imports
>> - ... and 8 more: https://git.openjdk.org/jmc/compare/ae2fbf35...2536c4a7
>
> core/org.openjdk.jmc.common/src/main/java/org/openjdk/jmc/common/util/IObservable.java line 36:
>
>> 34:
>> 35: import java.util.Observable;
>> 36: import java.util.Observer;
>
> **thought:** Should this `IObservable` be even removed, both `Observer` and `Observable` are deprecated since JDK 9.
Good point, think that might deserve it's own issue in JIRA though
-------------
PR Review Comment: https://git.openjdk.org/jmc/pull/300#discussion_r1199405837
More information about the jmc-dev
mailing list