This is the mail archive of the archer@sourceware.org mailing list for the Archer 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: [Archer] Re: [Archer] Re: [Archer] Stop the Insanity! LinespecRewrite

> This turned out to be pretty simple. collect_symbols was
> unconditionally adding all matching symbols to the results, which is
> appropriate only with DECODE_LINE_LIST_MODE. [I admit, I didn't
> actually know one could do that. "list a_global_variable" Learn
> something new everyday!]

Ah, yes, I came across the very same problem when I was helping Tom
with the Ada regressions with his linespec patche series...

> I pushed a patch for this. It now passes all of your new tests.
> 
> Any others?

I do not think so. That was fast! I will make sure to update my
sources and test again ASAP, hopefully this evening or tomorrow.

Great responsiveness, Keith. And also, the fact that you were able
to fix the problems easily seems to confirm the inherent quality
of your new code. It's great.

Cheers,
-- 
Joel
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]