From dbdf5a2a04a2649613e7fdf15b53f960f4dfa1e4 Mon Sep 17 00:00:00 2001 From: Jim Kingdon Date: Fri, 19 Nov 1993 21:16:16 +0000 Subject: [PATCH] * symtab.c (decode_line_1): Add comment about use of return_to_top_level directly instead of error. Add comment saying that the '' should not be needed--that the completer should be fixed. --- gdb/ChangeLog | 6 ++++++ gdb/symtab.c | 7 +++++++ 2 files changed, 13 insertions(+) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index af3ace2c95c..03fd96594c7 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +Fri Nov 19 15:08:47 1993 Jim Kingdon (kingdon@lioth.cygnus.com) + + * symtab.c (decode_line_1): Add comment about use of + return_to_top_level directly instead of error. Add comment saying + that the '' should not be needed--that the completer should be fixed. + Fri Nov 19 11:00:33 1993 Kung Hsu (kung@cirdan.cygnus.com) * symtab.c(decode_line_1): fix the inconsistency of setting diff --git a/gdb/symtab.c b/gdb/symtab.c index 32791492e2f..99811542f5b 100644 --- a/gdb/symtab.c +++ b/gdb/symtab.c @@ -1644,6 +1644,11 @@ build_canonical_line_spec (sal, symname, canonical) if no file is validly specified. Callers must check that. Also, the line number returned may be invalid. */ +/* We allow single quotes in various places. This is a hideous + kludge, which exists because the completer can't yet deal with the + lack of single quotes. FIXME: write a linespec_completer which we + can use as appropriate instead of make_symbol_completion_list. */ + struct symtabs_and_lines decode_line_1 (argptr, funfirstline, default_symtab, default_line, canonical) char **argptr; @@ -1742,6 +1747,8 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line, canonical) while(!++p && *p != '>'); if (!p) { + /* FIXME: Why warning() and then return_to_top_level? + What's wrong with error()? */ warning("non-matching '<' and '>' in command"); return_to_top_level (RETURN_ERROR); } -- 2.30.2