This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: [PATCH] mdebugread.c: remove use of partial-stab.h


Checked in.

Elena


Elena Zannoni wrote:
> 
> This is removing the second occurrence of the infimous
> #include "partial-stab.h" from gdb.
> 
> This time the file is mdebugread.c.  Including the code directly
> allowed to get rid of code that was ifdeffed DBXREAD_ONLY.  Almost all
> of all the SOFUN_ADDRESS_MAYBE_MISSING instances were under
> DBXREAD_ONLY as well, except 2. So those are left for a second pass.
> I am not yet sure we can get rid of them.  I also suspect that the
> giant switch can be further simplified a bit.
> 
> Next step is to get rid of the last include "partial-stab.h" which is
> in dbxread.c.
> 
> As usual I'll wait a day or so before I commit this, in case somebody
> wants to take a look at it.
> 
> Jim (Blandy), I would be extremely grateful if you could wait until
> this in in before going ahead with another patch for the address sets.
> 
> I tested this on Irix 6.3.
> 
> Elena
> 
> 2001-10-25  Elena Zannoni <ezannoni@redhat.com>
> 
>         * mdebugread.c (CUR_SYMBOL_TYPE, CUR_SYMBOL_VALUE,
> START_PSYMTAB,
>         END_PSYMTAB, SET_NAMESTRING, HANDLE_RBRAC): Delete definitions.
>         (parse_partial_symbols): Don't include partial-stab.h any
>         more. Don't reuse code in partial-stab.h, include the code
>         directly, instead. Simplify code from partial-stab.h eliminating
>         ifdef DBXREAD_ONLY code.
> 
>         * Makefile.in (mdebugread.o): Remove dependency on
> partial-stab.h.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]