-
Bug
-
Resolution: Fixed
-
P3
-
None
-
b24
-
Verified
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8056869 | emb-9 | Unassigned | P3 | Resolved | Fixed | b24 |
For some reason, the code in SJavacTestUtil thinks it necessary to check if the .class file for a .java file is up to date and compiles it if necessary.
Since this is standard functionality for jtreg, the code should be removed and replaced by the equivalent jtreg @build actions.
That leaves almost no reason for the *Wrapper classes to exist. The one remaining check left in SJavacTestUtil (to see if sjavac is on the classpath) can be done as a library call in each of the tests, without requiring a whole extra wrapper class.
Since this is standard functionality for jtreg, the code should be removed and replaced by the equivalent jtreg @build actions.
That leaves almost no reason for the *Wrapper classes to exist. The one remaining check left in SJavacTestUtil (to see if sjavac is on the classpath) can be done as a library call in each of the tests, without requiring a whole extra wrapper class.
- backported by
-
JDK-8056869 class SJavacTestUtil and *Wrapper are redundant and should be removed
-
- Resolved
-
- relates to
-
CODETOOLS-7900963 @requires should support class:classname as a predicate
-
- New
-