RFR: 8146728: TestPrintGCDetailsVerbose is never run by jtreg
Bengt Rutisson
bengt.rutisson at oracle.com
Wed Jan 13 08:00:08 UTC 2016
Hi Kim,
Would it also make sense to rename the test now that it actually does
not use PrintGCDetails nor Verbose?
Maybe that should have been done as part of the UL changes. We can also
do it as a separate change after you have fixed the current issue.
WRT, the java.version/jdk.debug @requires support, I am all for removing
what we don't use yet and adding it back when we need it.
Bengt
On 2016-01-12 21:52, Kim Barrett wrote:
> On Jan 12, 2016, at 2:56 PM, Stefan Johansson <stefan.johansson at oracle.com> wrote:
>>> In which case, should I also remove the jdk.debug requires property?
>>> This is the only test presently using this feature. It can always be
>>> added back if some test needs it in the future.
>>>
>> I'm all for removing it, as you say, it's easy to add back in the future when needed.
>>
>>> I think this is also the only test using ~= (regex match) in a
>>> @requires.
>> Is this documented somewhere? Just so the functionality isn't forgotten when the only use is removed.
> It’s described in the jtreg documentation source, though the published web page (http://openjdk.java.net/jtreg/) seems a bit behind the latest hg version.
>
More information about the hotspot-gc-dev
mailing list