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: [patch] Fix "ambiguous linespec" regression: break lineno


>>>>> "Jan" == Jan Kratochvil <jan.kratochvil@redhat.com> writes:

Jan> It seems somehow clear to me, OK to check it in?

I agree it is an improvement.

However, why should this apply to linespecs used by 'break' but not by
other ones?

Jan> +      if (last_displayed_sal_is_valid ()

linespec.c:initialize_defaults has:

      struct symtab_and_line cursal = 
	get_current_source_symtab_and_line ();

It seems like we have two similar notions here -- the "current" source
line and the "last displayed" source line.

This doesn't make sense to me.  Can we not just have a single notion and
use it everywhere?

If we really need two, can we do the processing in linespec.c?

I realize you're just reverting a bit of code - but is that ObjC hack
really needed?  I'd like us to get away from this kind of thing.

Tom


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