RFR: CODETOOLS-7903317: Fix code to compile extra property definitions
Jonathan Gibbons
jjg at openjdk.org
Wed Sep 21 23:04:38 UTC 2022
On Sat, 17 Sep 2022 05:05:25 GMT, Iris Clark <iris at openjdk.org> wrote:
>> Please review a fairly simple update to eliminate false positives in comments when looking for the name of the class in a Java source file.
>>
>> The fix addresses a performance issue when running tests, because the "extra property definition" files were compiled every time jtreg was run, instead of as needed, which is the intent.
>>
>> The fix is to ignore comments in the source code. The code to do this is copied from equivalent functionality in the langtools `ToolBox` class, which has been in use for many years.
>>
>> The existing tests for "extra property definition" files have been extended with a test case to ensure that the false positives no longer occur and that the files are not recompiled unnecessarily.
>>
>> Minor style issues in the main source file have also been fixed.
>
> test/extra-props/comments/TEST.ROOT line 2:
>
>> 1: /*
>> 2: * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
>
> Copyright year?
Borderline, since it's a copy ...
-------------
PR: https://git.openjdk.org/jtreg/pull/122
More information about the jtreg-dev
mailing list