On Wed, 16 Dec 2020 01:19:43 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Patricio Chilano Mateo has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   include 'VM Warning' in stderr search
>
> test/lib/jdk/test/lib/process/OutputAnalyzer.java line 43:
> 
>> 41:     private static final String jvmwarningmsg = ".* VM warning:.*";
>> 42: 
>> 43:     private static final String deprecatedmsg = ".* deprecated.*";
> 
> This might be too generic, perhaps combine it with "VM warning" so we don't 
> get accidental hits on other deprecation output eg from javac?
> Also I'm not sure if only deprecation warnings should be handled but I guess 
> we can adjust if that need arises.

Sounds good. I added "VM warning" to the search.

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

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

Reply via email to