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.

Sign up for GitHub

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

Jump to bottom

S2222 performance: Improve early bailout logic #5375

Closed
pavel-mikula-sonarsource opened this issue Feb 9, 2022 · 0 comments · Fixed by #5405 or #5410
Closed

S2222 performance: Improve early bailout logic #5375

pavel-mikula-sonarsource opened this issue Feb 9, 2022 · 0 comments · Fixed by #5405 or #5410
Assignees
@costin-zaharia-sonarsource
Labels
Area: CFG/SE CFG and SE related issues.
Milestone
8.36

Comments

@pavel-mikula-sonarsource
Copy link
Contributor

pavel-mikula-sonarsource commented Feb 9, 2022 •
edited
Loading

Improve LocksReleasedAllPathsBase.ShouldExecute() logic to execute the engine only if there exist related pair of acquire&release methods.

We should stop exploring when we meet any lambda or nested function.

This needs to be done for C# and also VB.NET
@pavel-mikula-sonarsource pavel-mikula-sonarsource added this to the 8.36 milestone Feb 9, 2022
Merged
Merged
Merged
@costin-zaharia-sonarsource costin-zaharia-sonarsource linked a pull request Feb 18, 2022 that will close this issue
Implement early bail-out #5410
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@costin-zaharia-sonarsource costin-zaharia-sonarsource

Labels
Area: CFG/SE CFG and SE related issues.
Projects
None yet
Milestone
8.36
2 participants
@costin-zaharia-sonarsource @pavel-mikula-sonarsource