Skip to content

error C2665: 'CorSigUncompressData': none of the 3 overloads could convert all the argument types - #60967

Merged
davmason merged 1 commit into
dotnet:mainfrom
ww898:ww898-fix-poiter-conversion
Dec 6, 2021
Merged

error C2665: 'CorSigUncompressData': none of the 3 overloads could convert all the argument types#60967
davmason merged 1 commit into
dotnet:mainfrom
ww898:ww898-fix-poiter-conversion

Conversation

@ww898

Copy link
Copy Markdown
Contributor

Fix for cor.h(1951,8): error C2665: 'CorSigUncompressData': none of the 3 overloads could convert all the argument types on VS2019 v16.7 with warning level 4.

…the 3 overloads could convert all the argument types` on VS2019 with warning level 4.
@ghostghost added community-contribution Indicates that the PR has been added by a community member area-Infrastructure-coreclr Only use for closed issues labels Oct 28, 2021
@ghost

Copy link
Copy Markdown

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

Fix for cor.h(1951,8): error C2665: 'CorSigUncompressData': none of the 3 overloads could convert all the argument types on VS2019 v16.7 with warning level 4.

Author:ww898
Assignees:-
Labels:

area-Infrastructure-coreclr, community-contribution

Milestone:-

@ww898

Copy link
Copy Markdown
ContributorAuthor

I have the warning as errors option.

@davidwrightondavidwrighton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@davmasondavmason left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@davmason
davmason merged commit 5d9814e into dotnet:mainDec 6, 2021
viewizard added a commit to Samsung/netcoredbg that referenced this pull request Dec 10, 2021
Related to dotnet/runtime@fcd862e
Note, in case warning level 4, you also could need dotnet/runtime#60967 changes.
@ghostghost locked as resolved and limited conversation to collaborators Jan 6, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure-coreclrOnly use for closed issuescommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@ww898@davidwrighton@davmason