Prefix property descriptions of panics with "Panic:"#4045
Prefix property descriptions of panics with "Panic:"#4045tautschnig wants to merge 1 commit intomodel-checking:mainfrom
Conversation
This will reduce the ambiguity between cases of Kani reporting a safety (undefined behavior) violation vs Kani demonstrating the possibility of a panic (defined behavior).
|
Is it not possible to rely on the property class for this disambiguation? We've been trying to avoid modifying assertion messages that come from the compiler. |
I guess that would require also printing the property class when reporting results? |
|
I guess what I'm missing is the target audience of this change. Is it for human consumption? If not, Kani's CBMC property parser processes the property class: kani/kani-driver/src/cbmc_output_parser.rs Line 101 in 7209765 |
My current reason for putting forth this change proposal is that we were reviewing results on the standard library and didn't know whether we were seeing reports of undefined behavior or possible panics. |
And this cannot be inferred from the first line of a check's output? e.g.: which shows that this is a (Sorry for asking too many questions. I just want to make sure this change is warranted.) |
|
Closing as no longer needed. |
This will reduce the ambiguity between cases of Kani reporting a safety (undefined behavior) violation vs Kani demonstrating the possibility of a panic (defined behavior).
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.