Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8046824

class SJavacTestUtil and *Wrapper are redundant and should be removed

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: P3 P3
    • 9
    • None
    • tools
    • b24
    • Verified

        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.

              jjg Jonathan Gibbons
              jjg Jonathan Gibbons
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Created:
                Updated:
                Resolved: