RFR: 8252778: remove jdk.test.lib.FileInstaller action from compiler/c2/stemmer test

Igor Ignatyev iignatyev at openjdk.java.net
Sun Sep 6 16:37:51 UTC 2020


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

-------------

Commit messages:
 - 8252778: remove jdk.test.lib.FileInstaller action from compiler/c2/stemmer test

Changes: https://git.openjdk.java.net/jdk/pull/33/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=33&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8252778
  Stats: 2 lines in 1 file changed: 0 ins; 1 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/33.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/33/head:pull/33

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


More information about the hotspot-compiler-dev mailing list