Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactored static analysis support #316

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

maximmenshikov
Copy link
Collaborator

This is a pull request created mostly for discussion.
The static analysis support is reimplemented using asp/simple library. All reports are passed back to RuC.

There are a few issues:

  1. The biggest one is related to handling of source locations. Unfortunately, it is hard to use internal source location matching mechanism, so the location is taken directly from the analyzer, which can do it intelligently.
  2. Analysis is actually performed twice: first by RuC, second by static analyzer using RuC. That's weird.
  3. The library is currently not open-source. This is to be reconsidered as the project was naturally intended to be open-source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant