Simplify push_handler and profile its malloc

* src/lisp.h (PUSH_HANDLER): Remove.
All callers changed to use push_handler directly.
* src/eval.c (internal_condition_case)
(internal_condition_case_1, internal_condition_case_2)
(internal_condition_case_n):
Use same pattern as for other invokers of push_handler.
(push_handler, push_handler_nosignal): Use call-by-value
instead of call-by-reference.  All uses changed.
(push_handler): Simplify by rewriting in terms of
push_handler_nosignal.
(push_handler_nosignal): Profile any newly allocated memory.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/emacs-module.c b/src/emacs-module.c
index f611c8ba600..e885af5de8f 100644
--- a/src/emacs-module.c
+++ b/src/emacs-module.c
@@ -194,8 +194,8 @@ static void module_wrong_type (emacs_env *, Lisp_Object, Lisp_Object);
 #define MODULE_SETJMP_1(handlertype, handlerfunc, retval, c, dummy)     \
   do {                                                                  \
     eassert (module_non_local_exit_check (env) == emacs_funcall_exit_return); \
-    struct handler *c;                                                  \
-    if (!push_handler_nosignal (&c, Qt, handlertype))                   \
+    struct handler *c = push_handler_nosignal (Qt, handlertype);        \
+    if (!c)                                                             \
       {                                                                 \
         module_out_of_memory (env);                                     \
         return retval;                                                  \