From df049a5831c7c573698b98bfd4bc0ec8dc854c1f Mon Sep 17 00:00:00 2001 From: Doug Evans Date: Fri, 17 Jan 2014 10:00:07 -0800 Subject: [PATCH] Move ASSERT_FUNCTION to FUNCTION_NAME. * common/common-utils.h (FUNCTION_NAME): Renamed from ASSERT_FUNCTION, and moved here ... * common/gdb_assert.h (ASSERT_FUNCTION): ... from here. #include "common-utils.h". (gdb_assert, gdb_assert_fail, gdb_assert_not_reached): Update. * common/vec.h (VEC_ASSERT_PASS): Update. * darwin-nat.h: Replace #include of gdb_assert.h with common-utils.h. (MACH_CHECK_ERROR): Update. --- gdb/ChangeLog | 11 +++++++++++ gdb/common/common-utils.h | 13 +++++++++++++ gdb/common/gdb_assert.h | 23 ++++++----------------- gdb/common/vec.h | 2 +- gdb/darwin-nat.h | 8 ++++---- 5 files changed, 35 insertions(+), 22 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a2ebabf8a14..d7d54e235bf 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,14 @@ +2014-01-17 Doug Evans + + * common/common-utils.h (FUNCTION_NAME): Renamed from ASSERT_FUNCTION, + and moved here ... + * common/gdb_assert.h (ASSERT_FUNCTION): ... from here. + #include "common-utils.h". + (gdb_assert, gdb_assert_fail, gdb_assert_not_reached): Update. + * common/vec.h (VEC_ASSERT_PASS): Update. + * darwin-nat.h: Replace #include of gdb_assert.h with common-utils.h. + (MACH_CHECK_ERROR): Update. + 2014-01-17 Simon Marchi * gdbarch.sh (gdbarch_address_class_name_to_type_flags): Add diff --git a/gdb/common/common-utils.h b/gdb/common/common-utils.h index 9777ac0f166..2d995490b91 100644 --- a/gdb/common/common-utils.h +++ b/gdb/common/common-utils.h @@ -25,6 +25,19 @@ #include #include +/* Version 2.4 and later of GCC define a magical variable `__PRETTY_FUNCTION__' + which contains the name of the function currently being defined. + This is broken in G++ before version 2.6. + C9x has a similar variable called __func__, but prefer the GCC one since + it demangles C++ function names. */ +#if (GCC_VERSION >= 2004) +#define FUNCTION_NAME __PRETTY_FUNCTION__ +#else +#if defined __STDC_VERSION__ && __STDC_VERSION__ >= 199901L +#define FUNCTION_NAME __func__ +#endif +#endif + extern void malloc_failure (long size) ATTRIBUTE_NORETURN; extern void internal_error (const char *file, int line, const char *, ...) ATTRIBUTE_NORETURN ATTRIBUTE_PRINTF (3, 4); diff --git a/gdb/common/gdb_assert.h b/gdb/common/gdb_assert.h index 50f7bf2be9b..4869fc45d40 100644 --- a/gdb/common/gdb_assert.h +++ b/gdb/common/gdb_assert.h @@ -19,6 +19,8 @@ #ifndef GDB_ASSERT_H #define GDB_ASSERT_H +#include "common-utils.h" /* For FUNCTION_NAME. */ + /* A static assertion. This will cause a compile-time error if EXPR, which must be a compile-time constant, is false. */ @@ -32,24 +34,11 @@ #define gdb_assert(expr) \ ((void) ((expr) ? 0 : \ - (gdb_assert_fail (#expr, __FILE__, __LINE__, ASSERT_FUNCTION), 0))) - -/* Version 2.4 and later of GCC define a magical variable `__PRETTY_FUNCTION__' - which contains the name of the function currently being defined. - This is broken in G++ before version 2.6. - C9x has a similar variable called __func__, but prefer the GCC one since - it demangles C++ function names. */ -#if (GCC_VERSION >= 2004) -#define ASSERT_FUNCTION __PRETTY_FUNCTION__ -#else -#if defined __STDC_VERSION__ && __STDC_VERSION__ >= 199901L -#define ASSERT_FUNCTION __func__ -#endif -#endif + (gdb_assert_fail (#expr, __FILE__, __LINE__, FUNCTION_NAME), 0))) /* This prints an "Assertion failed" message, asking the user if they want to continue, dump core, or just exit. */ -#if defined (ASSERT_FUNCTION) +#if defined (FUNCTION_NAME) #define gdb_assert_fail(assertion, file, line, function) \ internal_error (file, line, _("%s: Assertion `%s' failed."), \ function, assertion) @@ -62,9 +51,9 @@ /* The canonical form of gdb_assert (0). MESSAGE is a string to include in the error message. */ -#if defined (ASSERT_FUNCTION) +#if defined (FUNCTION_NAME) #define gdb_assert_not_reached(message) \ - internal_error (__FILE__, __LINE__, "%s: %s", ASSERT_FUNCTION, _(message)) + internal_error (__FILE__, __LINE__, "%s: %s", FUNCTION_NAME, _(message)) #else #define gdb_assert_not_reached(message) \ internal_error (__FILE__, __LINE__, _(message)) diff --git a/gdb/common/vec.h b/gdb/common/vec.h index 987457301b1..7bae2ff9494 100644 --- a/gdb/common/vec.h +++ b/gdb/common/vec.h @@ -398,7 +398,7 @@ extern void *vec_o_reserve (void *, int, size_t, size_t); #define VEC_ASSERT_PASS ,file_,line_ #define vec_assert(expr, op) \ ((void)((expr) ? 0 : (gdb_assert_fail (op, file_, line_, \ - ASSERT_FUNCTION), 0))) + FUNCTION_NAME), 0))) #define VEC(T) VEC_##T #define VEC_OP(T,OP) VEC_##T##_##OP diff --git a/gdb/darwin-nat.h b/gdb/darwin-nat.h index 6d42a3bb816..84f4be1413a 100644 --- a/gdb/darwin-nat.h +++ b/gdb/darwin-nat.h @@ -18,7 +18,7 @@ #define __DARWIN_NAT_H__ #include -#include "gdb_assert.h" +#include "common-utils.h" /* For FUNCTION_NAME. */ /* Describe the mach exception handling state for a task. This state is saved before being changed and restored when a process is detached. @@ -136,10 +136,10 @@ extern mach_port_t darwin_port_set; /* A copy of mach_host_self (). */ extern mach_port_t darwin_host_self; -/* ASSERT_FUNCTION is defined in gdb_assert.h (or not). */ -#ifdef ASSERT_FUNCTION +/* FUNCTION_NAME is defined in common-utils.h (or not). */ +#ifdef FUNCTION_NAME #define MACH_CHECK_ERROR(ret) \ - mach_check_error (ret, __FILE__, __LINE__, ASSERT_FUNCTION) + mach_check_error (ret, __FILE__, __LINE__, FUNCTION_NAME) #else #define MACH_CHECK_ERROR(ret) \ mach_check_error (ret, __FILE__, __LINE__, "??") -- 2.30.2