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