Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

MSVC 2017 Compile Issues rttr_core #118

Closed
kannode opened this issue Feb 17, 2018 · 1 comment · Fixed by #120
Closed

MSVC 2017 Compile Issues rttr_core #118

kannode opened this issue Feb 17, 2018 · 1 comment · Fixed by #120

Comments

@kannode
Copy link

kannode commented Feb 17, 2018

When compiling main.cpp there is the following error and can be fixed by adding to the CMakeLists File.
if(MSVC)
add_definitions("-D_SILENCE_CXX17_UNCAUGHT_EXCEPTION_DEPRECATION_WARNING")
endif()

1>------ Build started: Project: unit_tests, Configuration: Release x64 ------
1>pch.cpp
1>main.cpp
1>D:\nodein\nusr\rttr\3rd_party\catch-1.8.2\catch/catch.hpp(8527): error C2220: warning treated as error - no 'object' file generated
1>D:\nodein\nusr\rttr\3rd_party\catch-1.8.2\catch/catch.hpp(8527): warning C4996: 'std::uncaught_exception': warning STL4006: std::uncaught_exception() is deprecated in C++17. It is superseded by std::uncaught_exceptions(), plural. You can define _SILENCE_CXX17_UNCAUGHT_EXCEPTION_DEPRECATION_WARNING or _SILENCE_ALL_CXX17_DEPRECATION_WARNINGS to acknowledge that you have received this warning.
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Tools\MSVC\14.12.25827\include\exception(17): note: see declaration of 'std::uncaught_exception'

@acki-m
Copy link
Contributor

acki-m commented Feb 17, 2018

On my tests it looks good:
https://ci.appveyor.com/project/acki-m/rttr/build/job/708vrlbbf6umts5i
Do you use the latest update of MSVC2017?
Anyway, I think I have to update catch before the 0.9.6 release

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

Successfully merging a pull request may close this issue.

2 participants