New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Add AK1008 - Check for multiple Stash.Stash() invocation #105

Open

Arkatufus wants to merge 2 commits into akkadotnet:dev from Arkatufus:Add-AK1008-multiple-stash-invocation

Contributor

Arkatufus commented Nov 13, 2024

Attempt to detect multiple Stash.Stash() invocation in a single code flow

Problems:

To detect this, we would need ControlFlowGraph to be able to check all possible code flow
ControlFlowGraph is an evolving API, there has been changes throughout the API versions
Problems with IOperation.Children property renamed to IOperation.ChildOperations from Roslyn 3.11 to Roslyn 4
Problems with analyzing inline anonymous lambda functions, this can only be done in Roslyn 4.9 which we don't support.

Arkatufus and others added 2 commits

November 13, 2024 22:43


          Add AK1008 - Check for multiple Stash.Stash() invocation

6760a69


          Merge branch 'dev' into Add-AK1008-multiple-stash-invocation

a8edf1b

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet