This is the mail archive of the gdb-patches@sourceware.org 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]
Other format: [Raw text]

Re: Performance regression (12x): Re: RFC: add relative file name handling for linespecs


Hi Tom,

> Jan> I have noticed nightly regression testing to start randomly timing
> Jan> out a lot for gdb.ada/* testcases, narrowed it down to this
> Jan> check-in.
> 
> I looked into this.

I missed this message! You should have told me and I would have looked
into it.

> 2012-01-31  Tom Tromey  <tromey@redhat.com>
> 
> 	* ada-lang.c (ada_lookup_symbol_list_full): New function, from
> 	ada_lookup_symbol_list.  Add 'full_search' argument.
> 	(ada_lookup_symbol_list): Rewrite in terms of
> 	ada_lookup_symbol_list_full.
> 	(ada_iterate_over_symbols): Use ada_lookup_symbol_list_full.

Thanks a lot for the patch. I agree that the patch looks correct,
but I'm a little confused by the name of the new function (the _full
suffix suggests that the lookup is always a full search).

Traditionally, we've used a _1 suffix instead, which is very
unimaginative and yet so convenient.  I was going to change
your patch this way, but in the end, I'm just thinking I should
just add the parameter and update all callers. There are not that
many. I suspect you might have thought about this too, only to
just try the minimum change.

Would it be OK with you if I made that change instead?

-- 
Joel


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