nerdcruft/libcruft-util
1
0
Fork 0
Code Issues Releases Activity

debug: remove nullary panic function

Browse Source 
panic without message tended to promote useless console output. if it's
worth panicing over it's worth writing a oneliner.
This commit is contained in:
Danny Robson 2016-04-19 14:50:13 +10:00
parent 3a6faca98f
commit 568af11a0a
2 changed files with 2 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
debug.hpp
View File

@ -68,7 +68,7 @@
#define CHECK(C) do { \ #define CHECK(C) do { \
DEBUG_ONLY( \ DEBUG_ONLY( \
if (!(C)) \ if (!(C)) \
panic (); \ panic (#C); \
); \ ); \
} while (0) } while (0)
@ -273,7 +273,6 @@
/////////////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////////////
constexpr void panic [[noreturn]] (const char*); constexpr void panic [[noreturn]] (const char*);
constexpr void panic [[noreturn]] (void);
template <typename ...Args> template <typename ...Args>
constexpr void panic [[noreturn]] (const char *fmt, const Args&...); constexpr void panic [[noreturn]] (const char *fmt, const Args&...);

9
debug.ipp
View File

@ -83,13 +83,6 @@ constexpr void unreachable [[noreturn]] (const char *msg)
/////////////////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////////////////
constexpr void panic [[noreturn]] (void)
{
panic ("nfi");
}
//-----------------------------------------------------------------------------
constexpr void panic [[noreturn]] (const char *msg) constexpr void panic [[noreturn]] (const char *msg)
{ {
! msg ! msg
@ -105,6 +98,6 @@ void
panic [[noreturn]] (const char *fmt, const Args& ...args) panic [[noreturn]] (const char *fmt, const Args& ...args)
{ {
! fmt ! fmt
? panic () ? panic ("unreachable constexpr panic helper")
: util::debug::detail::panic (fmt, args...); : util::debug::detail::panic (fmt, args...);
} }