This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: [RFA] Prevent a warning when compiling varobj.c
- To: fnasser at cygnus dot com
- Subject: Re: [RFA] Prevent a warning when compiling varobj.c
- From: Eli Zaretskii <eliz at delorie dot com>
- Date: Mon, 15 May 2000 03:17:16 -0400 (EDT)
- CC: gdb-patches at sourceware dot cygnus dot com
- References: <200005132132.RAA16282@indy.delorie.com> <391DB3A7.A3EEA6F8@cygnus.com>
- Reply-to: Eli Zaretskii <eliz at is dot elta dot co dot il>
> Date: Sat, 13 May 2000 19:57:27 +0000
> From: Fernando Nasser <fnasser@cygnus.com>
>
> The wrapper.h part is approved.
I committed this (and the no-need-to-approve Makefile.in patch).
> Thanks for fixing this.
>
> Fernando
>
>
> Eli Zaretskii wrote:
> >
> > The following patch prevents GCC from printing a warning about
> > gdb_value_subscript being implicitly declared. While at that, I
> > noticed that some files which include wrapper.h don't list it in their
> > dependencies, so I fixed that as well.
> >
> > Okay to commit?
> >
> > 2000-05-12 Eli Zaretskii <eliz@is.elta.co.il>
> >
> > * Makefile.in (gdbtypes.o, varobj.o): Depend on wrapper.h.
> >
> > * wrapper.h (gdb_value_subscript, wrap_value_subscript): Add
> > prototypes.
> >
> > --- gdb/Makefile.i~3 Fri May 12 18:50:44 2000
> > +++ gdb/Makefile.in Fri May 12 20:22:14 2000
> > @@ -1369,7 +1369,7 @@ arch-utils.o: arch-utils.c $(defs_h) $(b
> >
> > gdbtypes.o: gdbtypes.c $(bfd_h) complaints.h $(defs_h) $(expression_h) \
> > $(gdbtypes_h) language.h objfiles.h symfile.h $(symtab_h) target.h \
> > - $(value_h) gdb_string.h
> > + $(value_h) gdb_string.h wrapper.h
> >
> > go32-nat.o: go32-nat.c $(defs_h) $(inferior_h) gdb_wait.h $(gdbcore_h) \
> > $(command_h) $(floatformat_h) target.h
> > @@ -1943,7 +1943,7 @@ gdb-events.o: gdb-events.c gdb-events.h
> > ui-out.o: ui-out.c $(defs_h) $(ui_out_h) expression.h language.h
> > cli-out.o: cli-out.c $(defs_h) $(ui_out_h) $(cli_out_h)
> > varobj.o: varobj.c $(defs_h) $(frame_h) $(value_h) \
> > - $(language_h) valprint.h varobj.h
> > + $(language_h) valprint.h varobj.h wrapper.h
> > $(CC) -c $(INTERNAL_WARN_CFLAGS) $(NO_WERROR_CFLAGS) $<
> > wrapper.o: wrapper.c $(defs_h) $(frame_h) $(value_h) wrapper.h
> >
> > --- gdb/wrapper.h~0 Sat Apr 8 18:11:16 2000
> > +++ gdb/wrapper.h Fri May 12 20:17:20 2000
> > @@ -35,6 +35,9 @@ extern int wrap_value_fetch_lazy PARAMS
> > extern int gdb_value_equal PARAMS ((value_ptr, value_ptr, int *));
> > extern int wrap_value_equal PARAMS ((char *));
> >
> > +extern int gdb_value_subscript PARAMS ((value_ptr, value_ptr, value_ptr *));
> > +extern int wrap_value_subscript PARAMS ((char *));
> > +
> > extern int gdb_value_ind PARAMS ((value_ptr val, value_ptr * rval));
> > extern int wrap_value_ind PARAMS ((char *opaque_arg));
> >
>
> --
> Fernando Nasser
> Cygnus Solutions (a Red Hat company) E-Mail: fnasser@cygnus.com
> 2323 Yonge Street, Suite #300 Tel: 416-482-2661 ext. 311
> Toronto, Ontario M4P 2C9 Fax: 416-482-6299
>