This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: (patch) hpjyg07: include/hp-symtab.h
- To: Jimmy Guo <guo at cup dot hp dot com>
- Subject: Re: (patch) hpjyg07: include/hp-symtab.h
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Wed, 15 Dec 1999 11:24:40 -0700
- cc: gdb-patches at sourceware dot cygnus dot com, binutils at sourceware dot cygnus dot com
- Reply-To: law at cygnus dot com
In message <Pine.LNX.4.10.9912150840410.29519-100000@hpcll168.cup.hp.com>you
write:
> > > 1999-11-03 Jimmy Guo <guo@cup.hp.com>
> > >
> > > * hp-symtab.h (HP_LANGUAGE_FORTRAN): Declare, replaces
> > > HP_LANGUAGE_F77.
> >I don't see the point behind making this change. Also note that it will
> >break gdb unless you change hp-psymtab-read.c at the same time.
>
> >Why are you making this change?
>
> We've added Fortran 90 support and this change is to cover Fortran90 as
> well.
Ah yes. That makes perfect sense.
> But it could be that hp-psymtab-read.c needs change at the same
> time ... I thought I tested each patch but one can never be sure.
grep HP_LANGUAGE_F77 will show one use in hp-psymtab-read.c so it
should be updated to the newer name. I'll take care of that.
It probably does not matter for this specific patch, but in general if we can
keep compatibility we should. I went ahead and updated the patch so that it
provides by HP_LANGUAGE_FORTRAN and HP_LANGUAGE_F77.
The changes should appear in the next gdb snapshot.
Thanks,
jeff