Skip to content

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

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.

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

Merged
merged 1 commit into from
Dec 6, 2021
Merged

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

merged 1 commit into from
Dec 6, 2021

Conversation

ww898
Copy link
Contributor

@ww898 ww898 commented Oct 28, 2021

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.
@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-Infrastructure-coreclr labels Oct 28, 2021
@ghost
Copy link

ghost commented Oct 28, 2021

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
Contributor Author

ww898 commented Oct 29, 2021

I have the warning as errors option.

Copy link
Member

@davidwrighton davidwrighton left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Contributor

@davmason davmason left a comment

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:main Dec 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.
@ghost ghost locked as resolved and limited conversation to collaborators Jan 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure-coreclr community-contribution Indicates 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