Skip to content

WIP ineff-assign and unused-assign analyzer#450

Draft
bluesign wants to merge 2 commits intoonflow:masterfrom
bluesign:master
Draft

WIP ineff-assign and unused-assign analyzer#450
bluesign wants to merge 2 commits intoonflow:masterfrom
bluesign:master

Conversation

@bluesign
Copy link
Copy Markdown
Contributor

Description

WIP

adds unused variable and ineffectual assign analyzer.

a bit WIP some places I am not sure 100%


  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@turbolent
Copy link
Copy Markdown
Member

Great work Deniz! Is this analysis inspired by some existing algorithm, like e.g. https://github.com/gordonklaus/ineffassign?

@bluesign
Copy link
Copy Markdown
Contributor Author

I don't remember exactly, but I think you directed me this algorithm before, I tried to make it similar probably.

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.

2 participants