RFR: 8198317: Enhance JavacTool.getTask for flexibility

Jonathan Gibbons jjg at openjdk.java.net
Fri Dec 25 20:53:58 UTC 2020


On Fri, 25 Dec 2020 16:26:39 GMT, Guoxiong Li <github.com+13688759+lgxbslgx at openjdk.org> wrote:

> Hi all,
> 
> This little patch enhances the setting of `Log` in `JavacTool.getTask`.
> 
> Thank you for taking the time to review.
> 
> Best Regards.

As a general rule, all changes should have an accompanying regression test, unless there is a good reason why not.   The set of good reasons has a corresponding list of JBS labels, shown here: http://openjdk.java.net/guide/#noreg

The closest match here would be `noreg-trivial` but I don't think it is close enough to claim that. In other words, I think this change needs a corresponding new test.  It is also not uncommon for small changes like this to more effort to write the test than fixing the underlying issue.

--

* *Unit test*: insurance against having to writing a regression test later on
* *Regression test*: the penalty for not writing a unit test in the first place

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

Changes requested by jjg (Reviewer).

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


More information about the compiler-dev mailing list