diff options
author | Andres Noetzli <andres.noetzli@gmail.com> | 2018-10-22 09:12:47 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-10-22 09:12:47 -0700 |
commit | 8e58fec53e1bc9f1b9349a235e2ad76bda4d9dd9 (patch) | |
tree | cad404d821e33fdaf7218584adc67b62b368380f /src | |
parent | 0b515de430771d9bdc15fc29d41b391f1cc7c1fd (diff) |
Recover from wrong use of get-info :reason-unknown (#2667)
Fixes #2584. Currently, we are immediately terminating CVC4 if the user
issues a `(get-info :reason-unknown)` command if it didn't succeed a
`(check-sat)` call returning `unknown`. This commit changes the behavior
to return an `(error ...)` but continue executing afterwards. It turns
the `ModalException` thrown in this case into a
`RecoverableModalException` and adds a check in
`GetInfoCommand::invoke()` to turn it into a
`CommandRecoverableFailure`, which solves the issue.
Diffstat (limited to 'src')
-rw-r--r-- | src/smt/command.cpp | 4 | ||||
-rw-r--r-- | src/smt/smt_engine.cpp | 5 |
2 files changed, 7 insertions, 2 deletions
diff --git a/src/smt/command.cpp b/src/smt/command.cpp index 51cb6663f..5198ea2d1 100644 --- a/src/smt/command.cpp +++ b/src/smt/command.cpp @@ -2438,6 +2438,10 @@ void GetInfoCommand::invoke(SmtEngine* smtEngine) { d_commandStatus = new CommandUnsupported(); } + catch (RecoverableModalException& e) + { + d_commandStatus = new CommandRecoverableFailure(e.what()); + } catch (exception& e) { d_commandStatus = new CommandFailure(e.what()); diff --git a/src/smt/smt_engine.cpp b/src/smt/smt_engine.cpp index 149d8bb35..9a0d969d8 100644 --- a/src/smt/smt_engine.cpp +++ b/src/smt/smt_engine.cpp @@ -2412,8 +2412,9 @@ CVC4::SExpr SmtEngine::getInfo(const std::string& key) const { transform(s.begin(), s.end(), s.begin(), ::tolower); return SExpr(SExpr::Keyword(s)); } else { - throw ModalException("Can't get-info :reason-unknown when the " - "last result wasn't unknown!"); + throw RecoverableModalException( + "Can't get-info :reason-unknown when the " + "last result wasn't unknown!"); } } else if(key == "assertion-stack-levels") { AlwaysAssert(d_userLevels.size() <= |