FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add -a/--all flag to show all SMT verification conditions by CatarinaGamboa · Pull Request #243 · liquid-java/liquidjava · GitHub

Add -a/--all flag to show all SMT verification conditions - #243

Draft
CatarinaGamboa wants to merge 3 commits into
mainfrom
worktree-issue-240
Draft

Add -a/--all flag to show all SMT verification conditions#243
CatarinaGamboa wants to merge 3 commits into
mainfrom
worktree-issue-240

Conversation

Copy link
Copy Markdown
Collaborator

Description

Closes #240

Now we show all the VCs that were sent to the smt solver even if there are no errors.

Example

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring

Checklist

  • Added/updated tests under liquidjava-example/src/main/java/testSuite/ (Correct* / Error*)
  • mvn test passes locally
  • Updated docs/README if behavior or API changed

CatarinaGamboa requested a review from rcosta358 June 2, 2026 15:34
public boolean debugMode;

@Option(names = { "-a", "--all" }, description = "Show every verification condition sent to the SMT solver")
public boolean showAllVCs;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Maybe rename to something like debugAllMode.

* {@code --debug} / {@code -d} or the lighter {@code --all} / {@code -a}, which shows every query without the rest
* of the debug output (e.g. simplification passes).
*/
public static boolean smtEnabled() {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Maybe rename to loggingEnabled.

CatarinaGamboa marked this pull request as draft June 3, 2026 14:19

Copy link
Copy Markdown
Collaborator Author

We need more information for the cases we don't even have verification starting when there was supposed to be some, I'll work on this next

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a flag -a to show all verification conditions sent to the smt solver

2 participants


Back | FazBrowse Home | New Git URL