diff options
| author |  Clément Pit--Claudel <clement.pitclaudel@live.com> | 2015-07-27 11:28:39 -0700 |
|---|---|---|
| committer | Clément Pit--Claudel <clement.pitclaudel@live.com> | 2015-07-27 11:28:39 -0700 |
| commit | 8d1676a6d30e295d57e7e64fc58d103771459974 (patch) | |
| tree | a995ec33aa9fa1e4b129161b28c6571998fd4c4b | |
| parent | b61088661bea206390f487ed813fd350a50eb084 (diff) | |
Clarify the inlining warning.
| -rw-r--r-- | Source/Dafny/Translator.cs | 8 | ||||
| -rw-r--r-- | Test/dafny0/TriggerInPredicate.dfy.expect | 4 |
2 files changed, 4 insertions, 8 deletions
diff --git a/Source/Dafny/Translator.cs b/Source/Dafny/Translator.cs index e2c45558..83078fb5 100644 --- a/Source/Dafny/Translator.cs +++ b/Source/Dafny/Translator.cs @@ -12956,12 +12956,8 @@ namespace Microsoft.Dafny { } else {
// Skip inlining, as it would cause arbitrary expressions to pop up in the trigger
// CLEMENT: Report inlining issue in a VS plugin friendly way
- var info = new AdditionalInformation {
- Token = fexp.tok,
- Length = fexp.tok.val.Length,
- Text = "This call cannot be safely inlined.",
- };
- Resolver.DefaultInformationReporter(info);
+ //CLEMENT this should appear at the outmost call site, not at the innermost. See SnapshotableTrees.dfy
+ Dafny.Util.ReportIssue("Info", fexp.tok, "Some instances of this call cannot safely be inlined.");
}
}
diff --git a/Test/dafny0/TriggerInPredicate.dfy.expect b/Test/dafny0/TriggerInPredicate.dfy.expect index 47b3bdf9..7f6e0268 100644 --- a/Test/dafny0/TriggerInPredicate.dfy.expect +++ b/Test/dafny0/TriggerInPredicate.dfy.expect @@ -1,4 +1,4 @@ -TriggerInPredicate.dfy(9,20): Info: This call cannot be safely inlined.
-TriggerInPredicate.dfy(9,20): Info: This call cannot be safely inlined.
+TriggerInPredicate.dfy(9,20): Info: Some instances of this call cannot safely be inlined.
+TriggerInPredicate.dfy(9,20): Info: Some instances of this call cannot safely be inlined.
Dafny program verifier finished with 8 verified, 0 errors
|