Re: RFR: 8281585: Remove unused imports under test/lib and jtreg/gc [v2]

2022-02-14 Thread Leo Korinth
On Fri, 11 Feb 2022 08:54:51 GMT, Leo Korinth  wrote:

>> Remove unused imports under test/lib and jtreg/gc. They create lots of 
>> warnings if editing using an IDE. Tests in hotspot_gc passed.
>
> Leo Korinth has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   updating copyright

Thanks David and Serguei!

-

PR: https://git.openjdk.java.net/jdk/pull/7426


Re: RFR: 8281585: Remove unused imports under test/lib and jtreg/gc [v2]

2022-02-11 Thread David Holmes
On Fri, 11 Feb 2022 08:54:51 GMT, Leo Korinth  wrote:

>> Remove unused imports under test/lib and jtreg/gc. They create lots of 
>> warnings if editing using an IDE. Tests in hotspot_gc passed.
>
> Leo Korinth has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   updating copyright

Looks good.

Thanks.

-

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7426


Re: RFR: 8281585: Remove unused imports under test/lib and jtreg/gc [v2]

2022-02-11 Thread Serguei Spitsyn
On Fri, 11 Feb 2022 08:54:51 GMT, Leo Korinth  wrote:

>> Remove unused imports under test/lib and jtreg/gc. They create lots of 
>> warnings if editing using an IDE. Tests in hotspot_gc passed.
>
> Leo Korinth has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   updating copyright

Looks good.
Thanks,
Serguei

-

Marked as reviewed by sspitsyn (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7426


Re: RFR: 8281585: Remove unused imports under test/lib and jtreg/gc [v2]

2022-02-11 Thread Leo Korinth
> Remove unused imports under test/lib and jtreg/gc. They create lots of 
> warnings if editing using an IDE. Tests in hotspot_gc passed.

Leo Korinth has updated the pull request incrementally with one additional 
commit since the last revision:

  updating copyright

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7426/files
  - new: https://git.openjdk.java.net/jdk/pull/7426/files/6aaa1a3a..7d3e7a1b

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=7426=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=7426=00-01

  Stats: 59 lines in 59 files changed: 0 ins; 0 del; 59 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7426.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7426/head:pull/7426

PR: https://git.openjdk.java.net/jdk/pull/7426


Re: RFR: 8281585: Remove unused imports under test/lib and jtreg/gc [v2]

2022-02-11 Thread Leo Korinth
On Fri, 11 Feb 2022 08:54:51 GMT, Leo Korinth  wrote:

>> Remove unused imports under test/lib and jtreg/gc. They create lots of 
>> warnings if editing using an IDE. Tests in hotspot_gc passed.
>
> Leo Korinth has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   updating copyright

I have a maven project that compiles test/lib and jtreg/gc, so everything 
changed does compile, I should have mentioned that. I have updated copyright 
year on all files now as well.

-

PR: https://git.openjdk.java.net/jdk/pull/7426