generic-library/jsoncpp
1
0
Fork 0
mirror of https://github.com/open-source-parsers/jsoncpp.git synced 2025-01-07 09:48:03 +01:00
Code Issues Projects Releases Wiki Activity

Added NORETURN for throw functions.

Browse Source 
Resolve Issue - https://github.com/open-source-parsers/jsoncpp/issues/389
This commit is contained in:
Gaurav 2016-03-16 11:15:09 +05:30
parent e29b671ed5
commit d97ea5bf8d
1 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
include/json/value.h
View File

@ -22,6 +22,19 @@
#include <cpptl/forwards.h> #include <cpptl/forwards.h>
#endif #endif
//Conditional NORETURN attribute on the throw functions would:
// a) suppress false positives from static code analysis
// b) possibly improve optimization opportunities.
#if !defined(JSONCPP_NORETURN)
# if defined(_MSC_VER)
# define JSONCPP_NORETURN __declspec(noreturn)
# elif defined(__GNUC__)
# define JSONCPP_NORETURN __attribute__ ((__noreturn__))
# else
# define JSONCPP_NORETURN
# endif
#endif
// Disable warning C4251: <data member>: <type> needs to have dll-interface to // Disable warning C4251: <data member>: <type> needs to have dll-interface to
// be used by... // be used by...
#if defined(JSONCPP_DISABLE_DLL_INTERFACE_WARNING) #if defined(JSONCPP_DISABLE_DLL_INTERFACE_WARNING)
@ -69,9 +82,9 @@ public:
}; };
/// used internally /// used internally
void throwRuntimeError(JSONCPP_STRING const& msg); JSONCPP_NORETURN void throwRuntimeError(JSONCPP_STRING const& msg);
/// used internally /// used internally
void throwLogicError(JSONCPP_STRING const& msg); JSONCPP_NORETURN void throwLogicError(JSONCPP_STRING const& msg);
/** \brief Type of the value held by a Value object. /** \brief Type of the value held by a Value object.
*/ */