Add description to failure message if it is available

Aleksey Shipilev shade at redhat.com
Tue Dec 26 12:23:21 UTC 2017


On 12/26/2017 12:19 PM, Ivan Sopov wrote:
> In the final print of results ids of test failures are included, but
> descriptions are not (they are also called interpretations in the previous
> output). It seems to me that it would be convenient to have those
> descriptions right next to failure ids. I mean this:
> 
> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
> false, true, true, true (Put T, Put F, Remove F)
> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
> true, false, false, true (Put F, Put T, Cannot remove T)
> 
> instead of this:
> 
> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
> false, true, true, true
> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
> true, false, false, true

That makes sense.

> Patch is attached.

Please sign and submit the OCA, as per:
  http://openjdk.java.net/contribute/

Mention "OpenJDK" as the project, and this would be one-time hassle for entire OpenJDK.

Thanks,
-Aleksey




More information about the jcstress-dev mailing list