Add block granularity progress indication - #262
Closed
trktby wants to merge 99 commits into
Closed
Conversation
… flag is f*ing hard
…or crate and codelenses only updating after reloading a file or editing it.
verifying different files or crates, not just the info of the most recent one is stored.
…lecting information but not verifying
Not yet actually depending on the verification's result.
…with displaying it
…s. However the textual decorators just stack in subsequent verifications so they will have to be removed at some point
… crash message upon kill.
Includes a draft of new icons.
This includes: - retaining information between selective runs - overhaul of the decorator related storage structure - processing of block messages - some bug fixes
Before, the current block may have been misleading. In the case of all paths through all methods currently being verified being stuck on a block, and those last block messages not having triggered the UI update (i.e. if they all arrived within the update interval), the last displayed blocks are not actually the hanging ones. Now anothe configurable interval forces an update. A new option for passing extra silicon arguments is added. In particular, when reporting block messages, the `--numberOfErrorsToReport` option is set to 0 by default.
- extra silicon args is an array now - keep providing code lens if the file is not dirty (mainly for tab changes) - simplify verification result parsing (since the filename prefix seems to not be useful) - mark verification results of methods as stale if they were not selected for the last verification run - only force update when there has been data and take tab changes into account
The ranges extracted from the line masks were off. Also some inputs to the extractor function were wrong. Path results that are "Unreachable" are treated as success for now.
Member
|
Superseded by #264. |
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
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is part of a practical work supervised by @Aurel300. It is based on a previous work's PR.
The main changes are: