Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Help VC++ Code Analysis #30

Merged
merged 1 commit into from
Mar 19, 2021
Merged

Conversation

AaronRobinsonMSFT
Copy link
Contributor

@AaronRobinsonMSFT AaronRobinsonMSFT commented Mar 19, 2021

During some CA work in the dotnet runtime project we discovered the VC++ code analysis has trouble determining the following value is properly initialized prior to use. Instead of adding a suppression it seemed easier to simply initialize it.

See dotnet/runtime#49194 (comment) for context.

During some CA work in the dotnet runtime project we discovered the VC++ code analysis has trouble determining the follow value is properly initialized prior to use. Instead of adding a suppression it seemed easier to simply initialize it.

See dotnet/runtime#49194 (comment) for context.
@jtkukunas
Copy link
Contributor

LGTM.

@jtkukunas jtkukunas merged commit 26df674 into intel:master Mar 19, 2021
@AaronRobinsonMSFT AaronRobinsonMSFT deleted the patch-1 branch March 20, 2021 02:53
jtkukunas added a commit that referenced this pull request Apr 15, 2022
jtkukunas added a commit that referenced this pull request Apr 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants