RFR: 8305846: Support compilation in Proc test utility
David Holmes
dholmes at openjdk.org
Wed Apr 12 02:59:33 UTC 2023
On Tue, 11 Apr 2023 13:40:39 GMT, Weijun Wang <weijun at openjdk.org> wrote:
> Enhance the `Proc` utility to support compilation.
test/lib/jdk/test/lib/process/Proc.java line 331:
> 329:
> 330: if (compile) {
> 331: var comp = CompilerUtils.compile(
I don't find this use of `var` helpful - `var` is useful for saying "some type I don't really care about and can't be bothered to spell out". But here we just have a boolean so it is clearer to just say `boolean compiled` (or `success` or `ok` or some such name that reflects what the return value actually means). Thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13425#discussion_r1163528639
More information about the core-libs-dev
mailing list