1998-10-13 Jason Molenda (jsm@bugshack.cygnus.com)

        * blockframe.c (find_pc_sect_partial_function): Add braces to avoid
        possible nested-if confusion.
        * breakpoint.c (breakpoint_here_p): Ditto.
        (breakpoint_inserted_here_p): Ditto.
        (breakpoint_thread_match): Ditto.

        * gnu-regex.c: Define _REGEX_RE_COMP only if it isn't already defined.
        * gnu-regex.h: Define _REGEX_RE_COMP to pick up old compatability
        prototypes.

        * symtab.h: Add prototype for _initialize_source.
        * value.h: Add prototype for _initialize_value.

        * defs.h: Include sys/types.h or stddef.h to get size_t.
        (make_cleanup): Add make_cleanup_func typedef and switch to using
        a prototype for this function.
        (mfree): Add prototypes for mmalloc, mrealloc, mfree if we aren't
        using mmalloc.

        * ax-gdb.c breakpoint.c coffread.c corelow.c dbxread.c dwarf2read.c
        dwarfread.c elfread.c eval.c exec.c gdbtk-cmds.c gdbtk.c infcmd.c
        infrun.c mipsread.c nlmread.c os9kread.c parse.c printcmd.c symfile.c
        symmisc.c symtab.c thread.c top.c tracepoint.c typeprint.c valops.c:
        Cast parameters passed to make_cleanup to use the new
        make_cleanup_func typedef.


More warning cleanups.  There are still a bunch of places where the first
argument to make_cleanup is not cast to (make_cleanup_func); these are
either due to the function fitting the make_cleanup_func specification
already (e.g. free()) or they are in files that weren't compiled when
I did my make on a Linux native system.  Bwahahaha.  You can see them
like this:

grep make_cleanup\  * | grep -v make_cleanup_func

I'll surely go back and clean up the remaining suspicious calls in
GDB one of these days. :-)

1 files changed, 5 insertions, 5 deletions

diff --git a/gdb/ax-gdb.c b/gdb/ax-gdb.c
index a55df32..de6f870 100644
--- a/gdb/ax-gdb.c
+++ b/gdb/ax-gdb.c
@@ -1155,7 +1155,7 @@ gen_deref (ax, value)
      actually emit any code; we just change the type from "Pointer to
      T" to "T", and mark the value as an lvalue in memory.  Leave it
      to the consumer to actually dereference it.  */
-  value->type = TYPE_TARGET_TYPE (value->type);
+  value->type = check_typedef (TYPE_TARGET_TYPE (value->type));
   value->kind = ((value->type->code == TYPE_CODE_FUNC)
 		 ? axs_rvalue : axs_lvalue_memory);
 }
@@ -1782,7 +1782,7 @@ expr_to_agent (expr, value)
   struct agent_expr *ax = new_agent_expr ();
   union exp_element *pc;
 
-  old_chain = make_cleanup (free_agent_expr, ax);
+  old_chain = make_cleanup ((make_cleanup_func) free_agent_expr, ax);
 
   pc = expr->elts;
   trace_kludge = 0;
@@ -1836,7 +1836,7 @@ gen_trace_for_expr (expr)
   union exp_element *pc;
   struct axs_value value;
 
-  old_chain = make_cleanup (free_agent_expr, ax);
+  old_chain = make_cleanup ((make_cleanup_func) free_agent_expr, ax);
 
   pc = expr->elts;
   trace_kludge = 1;
@@ -1905,9 +1905,9 @@ agent_command (exp, from_tty)
     error_no_arg ("expression to translate");
   
   expr = parse_expression (exp);
-  old_chain = make_cleanup (free_current_contents, &expr);
+  old_chain = make_cleanup ((make_cleanup_func) free_current_contents, &expr);
   agent = gen_trace_for_expr (expr);
-  make_cleanup (free_agent_expr, agent);
+  make_cleanup ((make_cleanup_func) free_agent_expr, agent);
   ax_print (gdb_stdout, agent);
   ax_reqs (agent, &reqs);