Integrated: 8252778: remove jdk.test.lib.FileInstaller action from compiler/c2/stemmer test
Igor Ignatyev
iignatyev at openjdk.java.net
Thu Sep 10 13:25:32 UTC 2020
On Sun, 6 Sep 2020 16:30:29 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:
> pre-Skara RFR [thread](https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-September/039878.html)
>
> Hi all,
>
> could you please review this small and trivial cleanup?
>
> from [JBS](https://bugs.openjdk.java.net/browse/JDK-8252778):
>> `compiler/c2/stemmer` test uses `jdk.test.lib.FileInstaller` to copy "words" file from the test source directory to the
>> current working directory, `compiler.c2.stemmer.Stemmer` can read this file. yet, `c.c.s.Stemmer` class treats its 1st
>> argument as a path to the file, given this isn't needed and we can pass "${test.src}/words" instead of "words"
>
> testing: compiler/c2/stemmer on {linux,windows,macos}-x64
This pull request has now been integrated.
Changeset: 5b30a831
Author: Igor Ignatyev <iignatyev at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/5b30a831
Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
8252778: remove jdk.test.lib.FileInstaller action from compiler/c2/stemmer test
Reviewed-by: shade, epavlova
-------------
PR: https://git.openjdk.java.net/jdk/pull/33
More information about the hotspot-compiler-dev
mailing list