Skip to content
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

Warnings not being properly disabled under Visual Studio #26

Closed
pefoley2 opened this issue May 6, 2016 · 5 comments
Closed

Warnings not being properly disabled under Visual Studio #26

pefoley2 opened this issue May 6, 2016 · 5 comments
Labels
Milestone

Comments

@pefoley2
Copy link
Contributor

pefoley2 commented May 6, 2016

The /wd flags in warnings.cmake are being outputted to the .vcxproj files, but not passed down to the cl.exe command line, causing a flood of warning spam.

@wrwilliams wrwilliams added this to the Warning cleanup milestone May 9, 2016
@wrwilliams
Copy link
Member

@pefoley2 Do you have any idea how to ensure that these flags get passed to the build tool appropriately in CMake?

@pefoley2
Copy link
Contributor Author

It looks like the actual issue was an incorrect pragma in InstructionAPI.h causing warning 4251 to be unconditionally enabled
See https://github.com/pefoley2/dyninst/commit/9ea3f33c682c4ed6ad8ceb93b2ef258c5fc4e880

@wrwilliams
Copy link
Member

Very nice, and a good catch. That was really dumb of past-me.

@pefoley2
Copy link
Contributor Author

It took a while to figure out why the flag wasn't applying, but it was obvious once I figured out what was happening :)

@wrwilliams
Copy link
Member

Verified that this is fixed on master; closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants