Skip to content

Analyzer fixes#1251

Merged
AArnott merged 2 commits into
microsoft:mainfrom
AArnott:analyzerFixes
Oct 25, 2023
Merged

Analyzer fixes#1251
AArnott merged 2 commits into
microsoft:mainfrom
AArnott:analyzerFixes

Conversation

@AArnott

@AArnott AArnott commented Oct 24, 2023

Copy link
Copy Markdown
Member

Recognize local functions in VSTHRD103

This prevents false positives in synchronous local functions that appear within async methods.

Fixes #1003

Add test to demonstrate that #881 no longer repros

Closes #881

This prevents false positives in synchronous local functions that appear within async methods.

Fixes microsoft#1003
@AArnott AArnott merged commit 4b595ea into microsoft:main Oct 25, 2023
@AArnott AArnott deleted the analyzerFixes branch October 25, 2023 18:10
@AArnott AArnott added this to the v17.9 milestone Oct 25, 2023
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.

VSTHRD103 mis-fires in synchronous local methods declared within an async member method VSTHRD110 mis-fires on null coalescing operator

2 participants