Skip to content

Comment out unused variable to fix building with VS 2022 17.14#2347

Merged
tian-lt merged 1 commit intomicrosoft:mainfrom
jketema:unused-variable
Jun 24, 2025
Merged

Comment out unused variable to fix building with VS 2022 17.14#2347
tian-lt merged 1 commit intomicrosoft:mainfrom
jketema:unused-variable

Conversation

@jketema
Copy link
Contributor

@jketema jketema commented Jun 23, 2025

With the 17.14 release of Visual Studio 2022, I see the following error:

...\src\GraphingImpl\Mocks\Graph.h(47,47): warning C4100: 'variableName': unreferenced parameter [...\src\GraphingImpl\GraphingImpl.vcxproj
...
...\src\GraphingImpl\Mocks\Graph.h(47,47): error C2220: the following warning is treated as an error [...\src\GraphingImpl\GraphingImpl.vcxproj

Comment out the unused variable to resolve this.

Fixes #.

Description of the changes:

How changes were validated:

With the 17.14 release of Visual Studio 2022, I see the following error:
```
...\src\GraphingImpl\Mocks\Graph.h(47,47): warning C4100: 'variableName': unreferenced parameter [...\src\GraphingImpl\GraphingImpl.vcxproj
...
...\src\GraphingImpl\Mocks\Graph.h(47,47): error C2220: the following warning is treated as an error [...\src\GraphingImpl\GraphingImpl.vcxproj
```
Comment out the unused variable to resolve this.
@jketema jketema marked this pull request as ready for review June 23, 2025 13:32
@jketema jketema changed the title Comment out unused variable Comment out unused variable to fix building with VS 2022 17.14 Jun 23, 2025
@jketema
Copy link
Contributor Author

jketema commented Jun 23, 2025

@tian-lt @oneonezhang @hanzhang54

@tian-lt tian-lt merged commit 544e7a5 into microsoft:main Jun 24, 2025
5 checks passed
@jketema
Copy link
Contributor Author

jketema commented Jun 24, 2025

Thanks for the approval and merge!

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