This is the mail archive of the insight@sources.redhat.com mailing list for the Insight project.


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

Re: Patch: search `directory' path for `break' files


Tom Tromey writes:
 > >>>>> "Elena" == Elena Zannoni <ezannoni@cygnus.com> writes:
 > 
 > Elena> 1. Could you please check in the cleanup of lookup_symtab_1 as a
 > Elena>    separate obvious fix?
 > 
 > Done.
 > 
 > Thanks for your reviews.  I'll try soon to address the points you
 > raised and resubmit the patches.  It might be a little while since any
 > gdb hacking I do is on my own time.
 > 
 > Elena> 2. I was about to approve your patch, which is ok, when I got
 > Elena> curious about why symtab_to_filename wasn't used
 > Elena> anywhere. Seeing it was introduced by Stu Grossman, I thought
 > Elena> of gdbtk, and voila' there it is. Have you looked at
 > Elena> gdbtk-cmds.c:full_lookup_symtab() ?  Does it do what you need?
 > Elena> If so, could we avoid code duplication by having only one copy
 > Elena> of the same function?
 > 
 > I did run across this function after submitting my patch.  I think my
 > patch obsoletes this function, and we can investigate removing it.
 > full_lookup_symtab() doesn't really do the same thing as my patch
 > though, at least as I understand it.  I'm not 100% certain about what
 > is going on there since I don't know what the typical inputs to
 > full_lookup_symtab() look like.

I am cc-ing the insight list on this. They might know.

 > 
 > When you say the patch is ok does that mean I can check in the rest of
 > it?  Or should I try to handle psymtabs and directory names first?
 > 

No, no need to do the psymtabs now. I would like to get rid of code
duplication with gdbtk, if possible, first, though.

Elena


 > Tom


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