-
Notifications
You must be signed in to change notification settings - Fork 80
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
Exception stuff #511
Exception stuff #511
Conversation
} | ||
else | ||
{ | ||
printf("Something bad happened while reading SDLMgr data: %s", e.what()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where does this message go?
@@ -130,6 +130,22 @@ typedef int32_t hsError; | |||
# define hsDeprecated(message) | |||
#endif | |||
|
|||
#ifdef HAVE_OVERRIDE | |||
# define HS_OVERRIDE override |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has been around since MSVC++2012. Do we actually have to test for it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've specified our minimum MSVC compiler, but I don't know what we're targetting for GCC now that it has better support. IIRC, GCC only added override/final in version 4.7. I'm fine with making GCC 4.7 the minimum version though and cleaning this up :)
FYI, this minimum would exclude Ubuntu 12.04 LTS -- a distro I've usually still tried to support with libhsplasma.
This makes use of some exception information where it was previously being ignored, and derives HeadSpin exceptions from
std::exception
. It also adds compiler feature detection fornoexcept
,override
, andfinal
, and makes use ofnoexcept
in a few places where it may be useful (mostly the hsExceptions and plString)