Explicit locations: introduce explicit locations
[deliverable/binutils-gdb.git] / gdb / linespec.c
index 8ee721b239948fbd6cca3984ec6fe19efe401217..84e518fc719802bb24a73f060349ee2462fe863a 100644 (file)
@@ -1,8 +1,6 @@
 /* Parser for linespec for the GNU debugger, GDB.
 
-   Copyright (C) 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995,
-   1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2007, 2008,
-   2009, 2010, 2011 Free Software Foundation, Inc.
+   Copyright (C) 1986-2015 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -35,7 +33,6 @@
 #include "block.h"
 #include "objc-lang.h"
 #include "linespec.h"
-#include "exceptions.h"
 #include "language.h"
 #include "interps.h"
 #include "mi/mi-cmds.h"
 #include "arch-utils.h"
 #include <ctype.h>
 #include "cli/cli-utils.h"
+#include "filenames.h"
+#include "ada-lang.h"
+#include "stack.h"
+#include "location.h"
 
-/* We share this one with symtab.c, but it is not exported widely.  */
+typedef struct symbol *symbolp;
+DEF_VEC_P (symbolp);
 
-extern char *operator_chars (char *, char **);
+typedef struct type *typep;
+DEF_VEC_P (typep);
 
-/* Prototypes for local functions.  */
-
-static void initialize_defaults (struct symtab **default_symtab,
-                                int *default_line);
+/* An address entry is used to ensure that any given location is only
+   added to the result a single time.  It holds an address and the
+   program space from which the address came.  */
 
-static struct symtabs_and_lines decode_indirect (char **argptr);
+struct address_entry
+{
+  struct program_space *pspace;
+  CORE_ADDR addr;
+};
 
-static char *locate_first_half (char **argptr, int *is_quote_enclosed);
+typedef struct bound_minimal_symbol bound_minimal_symbol_d;
 
-static struct symtabs_and_lines decode_objc (char **argptr,
-                                            int funfirstline,
-                                            struct symtab *file_symtab,
-                                            struct linespec_result *canonical,
-                                            char *saved_arg);
+DEF_VEC_O (bound_minimal_symbol_d);
 
-static struct symtabs_and_lines decode_compound (char **argptr,
-                                                int funfirstline,
-                                                struct linespec_result *canonical,
-                                                char *saved_arg,
-                                                char *p);
+/* A linespec.  Elements of this structure are filled in by a parser
+   (either parse_linespec or some other function).  The structure is
+   then converted into SALs by convert_linespec_to_sals.  */
 
-static struct symbol *lookup_prefix_sym (char **argptr, char *p);
+struct linespec
+{
+  /* An explicit location describing the SaLs.  */
+  struct explicit_location explicit;
+
+  /* The list of symtabs to search to which to limit the search.  May not
+     be NULL.  If explicit.SOURCE_FILENAME is NULL (no user-specified
+     filename), FILE_SYMTABS should contain one single NULL member.  This
+     will cause the code to use the default symtab.  */
+  VEC (symtab_ptr) *file_symtabs;
+
+  /* A list of matching function symbols and minimal symbols.  Both lists
+     may be NULL if no matching symbols were found.  */
+  VEC (symbolp) *function_symbols;
+  VEC (bound_minimal_symbol_d) *minimal_symbols;
+
+  /* A structure of matching label symbols and the corresponding
+     function symbol in which the label was found.  Both may be NULL
+     or both must be non-NULL.  */
+  struct
+  {
+    VEC (symbolp) *label_symbols;
+    VEC (symbolp) *function_symbols;
+  } labels;
+};
+typedef struct linespec *linespec_p;
 
-static struct symtabs_and_lines find_method (int funfirstline,
-                                            struct linespec_result *canonical,
-                                            char *saved_arg,
-                                            char *copy,
-                                            struct type *t,
-                                            struct symbol *sym_class);
+/* A canonical linespec represented as a symtab-related string.
 
-static void cplusplus_error (const char *name, const char *fmt, ...)
-     ATTRIBUTE_NORETURN ATTRIBUTE_PRINTF (2, 3);
+   Each entry represents the "SYMTAB:SUFFIX" linespec string.
+   SYMTAB can be converted for example by symtab_to_fullname or
+   symtab_to_filename_for_display as needed.  */
 
-static int total_number_of_methods (struct type *type);
+struct linespec_canonical_name
+{
+  /* Remaining text part of the linespec string.  */
+  char *suffix;
 
-static int find_methods (struct type *, char *,
-                        enum language, struct symbol **);
+  /* If NULL then SUFFIX is the whole linespec string.  */
+  struct symtab *symtab;
+};
 
-static int add_matching_methods (int method_counter, struct type *t,
-                                enum language language,
-                                struct symbol **sym_arr);
+/* An instance of this is used to keep all state while linespec
+   operates.  This instance is passed around as a 'this' pointer to
+   the various implementation methods.  */
 
-static int add_constructors (int method_counter, struct type *t,
-                            enum language language,
-                            struct symbol **sym_arr);
+struct linespec_state
+{
+  /* The language in use during linespec processing.  */
+  const struct language_defn *language;
 
-static void build_canonical_line_spec (struct symtab_and_line *,
-                                      char *, struct linespec_result *);
+  /* The program space as seen when the module was entered.  */
+  struct program_space *program_space;
 
-static char *find_toplevel_char (char *s, char c);
+  /* The default symtab to use, if no other symtab is specified.  */
+  struct symtab *default_symtab;
 
-static int is_objc_method_format (const char *s);
+  /* The default line to use.  */
+  int default_line;
 
-static struct symtabs_and_lines decode_line_2 (struct symbol *[],
-                                              int, int,
-                                              struct linespec_result *);
+  /* The 'funfirstline' value that was passed in to decode_line_1 or
+     decode_line_full.  */
+  int funfirstline;
 
-static struct symtab *symtab_from_filename (char **argptr,
-                                           char *p, int is_quote_enclosed);
+  /* Nonzero if we are running in 'list' mode; see decode_line_list.  */
+  int list_mode;
 
-static struct symbol *find_function_symbol (char **argptr, char *p,
-                                           int is_quote_enclosed);
+  /* The 'canonical' value passed to decode_line_full, or NULL.  */
+  struct linespec_result *canonical;
 
-static struct
-symtabs_and_lines decode_all_digits (char **argptr,
-                                    struct symtab *default_symtab,
-                                    int default_line,
-                                    struct linespec_result *canonical,
-                                    struct symtab *file_symtab,
-                                    char *q);
+  /* Canonical strings that mirror the symtabs_and_lines result.  */
+  struct linespec_canonical_name *canonical_names;
 
-static struct symtabs_and_lines decode_dollar (char *copy,
-                                              int funfirstline,
-                                              struct symtab *default_symtab,
-                                              struct linespec_result *canonical,
-                                              struct symtab *file_symtab);
+  /* This is a set of address_entry objects which is used to prevent
+     duplicate symbols from being entered into the result.  */
+  htab_t addr_set;
 
-static int decode_label (struct symbol *function_symbol,
-                        char *copy, struct linespec_result *canonical,
-                        struct symtabs_and_lines *result);
+  /* Are we building a linespec?  */
+  int is_linespec;
+};
 
-static struct symtabs_and_lines decode_variable (char *copy,
-                                                int funfirstline,
-                                                struct linespec_result *canonical,
-                                                struct symtab *file_symtab);
+/* This is a helper object that is used when collecting symbols into a
+   result.  */
 
-static struct
-symtabs_and_lines symbol_found (int funfirstline,
-                               struct linespec_result *canonical,
-                               char *copy,
-                               struct symbol *sym,
-                               struct symtab *file_symtab,
-                               struct symbol *function_symbol);
+struct collect_info
+{
+  /* The linespec object in use.  */
+  struct linespec_state *state;
 
-static struct
-symtabs_and_lines minsym_found (int funfirstline,
-                               struct minimal_symbol *msymbol);
+  /* A list of symtabs to which to restrict matches.  */
+  VEC (symtab_ptr) *file_symtabs;
 
-/* Helper functions.  */
+  /* The result being accumulated.  */
+  struct
+  {
+    VEC (symbolp) *symbols;
+    VEC (bound_minimal_symbol_d) *minimal_symbols;
+  } result;
+};
 
-/* Issue a helpful hint on using the command completion feature on
-   single quoted demangled C++ symbols as part of the completion
-   error.  */
+/* Token types  */
 
-static void
-cplusplus_error (const char *name, const char *fmt, ...)
+enum ls_token_type
 {
-  struct ui_file *tmp_stream;
-  char *message;
+  /* A keyword  */
+  LSTOKEN_KEYWORD = 0,
 
-  tmp_stream = mem_fileopen ();
-  make_cleanup_ui_file_delete (tmp_stream);
+  /* A colon "separator"  */
+  LSTOKEN_COLON,
 
-  {
-    va_list args;
+  /* A string  */
+  LSTOKEN_STRING,
 
-    va_start (args, fmt);
-    vfprintf_unfiltered (tmp_stream, fmt, args);
-    va_end (args);
-  }
+  /* A number  */
+  LSTOKEN_NUMBER,
 
-  while (*name == '\'')
-    name++;
-  fprintf_unfiltered (tmp_stream,
-                     ("Hint: try '%s<TAB> or '%s<ESC-?>\n"
-                      "(Note leading single quote.)"),
-                     name, name);
+  /* A comma  */
+  LSTOKEN_COMMA,
 
-  message = ui_file_xstrdup (tmp_stream, NULL);
-  make_cleanup (xfree, message);
-  throw_error (NOT_FOUND_ERROR, "%s", message);
-}
+  /* EOI (end of input)  */
+  LSTOKEN_EOI,
 
-/* Return the number of methods described for TYPE, including the
-   methods from types it derives from.  This can't be done in the symbol
-   reader because the type of the baseclass might still be stubbed
-   when the definition of the derived class is parsed.  */
+  /* Consumed token  */
+  LSTOKEN_CONSUMED
+};
+typedef enum ls_token_type linespec_token_type;
 
-static int
-total_number_of_methods (struct type *type)
-{
-  int n;
-  int count;
+/* List of keywords  */
 
-  CHECK_TYPEDEF (type);
-  if (! HAVE_CPLUS_STRUCT (type))
-    return 0;
-  count = TYPE_NFN_FIELDS_TOTAL (type);
+static const char * const linespec_keywords[] = { "if", "thread", "task" };
+#define IF_KEYWORD_INDEX 0
 
-  for (n = 0; n < TYPE_N_BASECLASSES (type); n++)
-    count += total_number_of_methods (TYPE_BASECLASS (type, n));
+/* A token of the linespec lexer  */
 
-  return count;
-}
+struct ls_token
+{
+  /* The type of the token  */
+  linespec_token_type type;
 
-/* Recursive helper function for decode_line_1.
-   Look for methods named NAME in type T.
-   Return number of matches.
-   Put matches in SYM_ARR, which should have been allocated with
-   a size of total_number_of_methods (T) * sizeof (struct symbol *).
-   Note that this function is g++ specific.  */
+  /* Data for the token  */
+  union
+  {
+    /* A string, given as a stoken  */
+    struct stoken string;
 
-static int
-find_methods (struct type *t, char *name, enum language language,
-             struct symbol **sym_arr)
-{
-  int i1 = 0;
-  int ibase;
-  char *class_name = type_name_no_tag (t);
-  struct cleanup *cleanup;
-  char *canon;
+    /* A keyword  */
+    const char *keyword;
+  } data;
+};
+typedef struct ls_token linespec_token;
 
-  /* NAME is typed by the user: it needs to be canonicalized before
-     passing to lookup_symbol.  */
-  canon = cp_canonicalize_string (name);
-  if (canon != NULL)
-    {
-      name = canon;
-      cleanup = make_cleanup (xfree, name);
-    }
-  else
-    cleanup = make_cleanup (null_cleanup, NULL);
+#define LS_TOKEN_STOKEN(TOK) (TOK).data.string
+#define LS_TOKEN_KEYWORD(TOK) (TOK).data.keyword
 
-  /* Ignore this class if it doesn't have a name.  This is ugly, but
-     unless we figure out how to get the physname without the name of
-     the class, then the loop can't do any good.  */
-  if (class_name
-      && (lookup_symbol_in_language (class_name, (struct block *) NULL,
-                        STRUCT_DOMAIN, language, (int *) NULL)))
-    {
-      int method_counter;
-      int name_len = strlen (name);
+/* An instance of the linespec parser.  */
 
-      CHECK_TYPEDEF (t);
+struct ls_parser
+{
+  /* Lexer internal data  */
+  struct
+  {
+    /* Save head of input stream.  */
+    const char *saved_arg;
 
-      /* Loop over each method name.  At this level, all overloads of a name
-         are counted as a single name.  There is an inner loop which loops over
-         each overload.  */
+    /* Head of the input stream.  */
+    const char *stream;
+#define PARSER_STREAM(P) ((P)->lexer.stream)
 
-      for (method_counter = TYPE_NFN_FIELDS (t) - 1;
-          method_counter >= 0;
-          --method_counter)
-       {
-         char *method_name = TYPE_FN_FIELDLIST_NAME (t, method_counter);
-         char dem_opname[64];
+    /* The current token.  */
+    linespec_token current;
+  } lexer;
 
-         if (strncmp (method_name, "__", 2) == 0 ||
-             strncmp (method_name, "op", 2) == 0 ||
-             strncmp (method_name, "type", 4) == 0)
-           {
-             if (cplus_demangle_opname (method_name, dem_opname, DMGL_ANSI))
-               method_name = dem_opname;
-             else if (cplus_demangle_opname (method_name, dem_opname, 0))
-               method_name = dem_opname;
-           }
+  /* Is the entire linespec quote-enclosed?  */
+  int is_quote_enclosed;
 
-         if (strcmp_iw (name, method_name) == 0)
-           /* Find all the overloaded methods with that name.  */
-           i1 += add_matching_methods (method_counter, t, language,
-                                       sym_arr + i1);
-         else if (strncmp (class_name, name, name_len) == 0
-                  && (class_name[name_len] == '\0'
-                      || class_name[name_len] == '<'))
-           i1 += add_constructors (method_counter, t, language,
-                                   sym_arr + i1);
-       }
-    }
+  /* The state of the parse.  */
+  struct linespec_state state;
+#define PARSER_STATE(PPTR) (&(PPTR)->state)
 
-  /* Only search baseclasses if there is no match yet, since names in
-     derived classes override those in baseclasses.
+  /* The result of the parse.  */
+  struct linespec result;
+#define PARSER_RESULT(PPTR) (&(PPTR)->result)
+};
+typedef struct ls_parser linespec_parser;
 
-     FIXME: The above is not true; it is only true of member functions
-     if they have the same number of arguments (??? - section 13.1 of the
-     ARM says the function members are not in the same scope but doesn't
-     really spell out the rules in a way I understand.  In any case, if
-     the number of arguments differ this is a case in which we can overload
-     rather than hiding without any problem, and gcc 2.4.5 does overload
-     rather than hiding in this case).  */
+/* A convenience macro for accessing the explicit location result of
+   the parser.  */
+#define PARSER_EXPLICIT(PPTR) (&PARSER_RESULT ((PPTR))->explicit)
 
-  if (i1 == 0)
-    for (ibase = 0; ibase < TYPE_N_BASECLASSES (t); ibase++)
-      i1 += find_methods (TYPE_BASECLASS (t, ibase), name,
-                         language, sym_arr + i1);
+/* Prototypes for local functions.  */
 
-  do_cleanups (cleanup);
-  return i1;
-}
+static void iterate_over_file_blocks (struct symtab *symtab,
+                                     const char *name, domain_enum domain,
+                                     symbol_found_callback_ftype *callback,
+                                     void *data);
 
-/* Add the symbols associated to methods of the class whose type is T
-   and whose name matches the method indexed by METHOD_COUNTER in the
-   array SYM_ARR.  Return the number of methods added.  */
+static void initialize_defaults (struct symtab **default_symtab,
+                                int *default_line);
 
-static int
-add_matching_methods (int method_counter, struct type *t,
-                     enum language language, struct symbol **sym_arr)
-{
-  int field_counter;
-  int i1 = 0;
+CORE_ADDR linespec_expression_to_pc (const char **exp_ptr);
 
-  for (field_counter = TYPE_FN_FIELDLIST_LENGTH (t, method_counter) - 1;
-       field_counter >= 0;
-       --field_counter)
-    {
-      struct fn_field *f;
-      const char *phys_name;
+static struct symtabs_and_lines decode_objc (struct linespec_state *self,
+                                            linespec_p ls,
+                                            const char *arg);
 
-      f = TYPE_FN_FIELDLIST1 (t, method_counter);
+static VEC (symtab_ptr) *symtabs_from_filename (const char *);
 
-      if (TYPE_FN_FIELD_STUB (f, field_counter))
-       {
-         char *tmp_name, *tmp2;
-
-         tmp_name = gdb_mangle_name (t,
-                                     method_counter,
-                                     field_counter);
-         tmp2 = alloca (strlen (tmp_name) + 1);
-         strcpy (tmp2, tmp_name);
-         xfree (tmp_name);
-         phys_name = tmp2;
-       }
-      else
-       phys_name = TYPE_FN_FIELD_PHYSNAME (f, field_counter);
-
-      sym_arr[i1] = lookup_symbol_in_language (phys_name,
-                                  NULL, VAR_DOMAIN,
-                                  language,
-                                  (int *) NULL);
-      if (sym_arr[i1])
-       i1++;
-      else
-       {
-         /* This error message gets printed, but the method
-            still seems to be found.
-            fputs_filtered("(Cannot find method ", gdb_stdout);
-            fprintf_symbol_filtered (gdb_stdout, phys_name,
-                                     language_cplus,
-                                     DMGL_PARAMS | DMGL_ANSI);
-            fputs_filtered(" - possibly inlined.)\n", gdb_stdout);
-         */
-       }
-    }
+static VEC (symbolp) *find_label_symbols (struct linespec_state *self,
+                                         VEC (symbolp) *function_symbols,
+                                         VEC (symbolp) **label_funcs_ret,
+                                         const char *name);
 
-  return i1;
-}
+static void find_linespec_symbols (struct linespec_state *self,
+                                  VEC (symtab_ptr) *file_symtabs,
+                                  const char *name,
+                                  VEC (symbolp) **symbols,
+                                  VEC (bound_minimal_symbol_d) **minsyms);
 
-/* Add the symbols associated to constructors of the class whose type
-   is CLASS_TYPE and which are indexed by by METHOD_COUNTER to the
-   array SYM_ARR.  Return the number of methods added.  */
+static struct line_offset
+     linespec_parse_variable (struct linespec_state *self,
+                             const char *variable);
 
-static int
-add_constructors (int method_counter, struct type *t,
-                 enum language language, struct symbol **sym_arr)
-{
-  int field_counter;
-  int i1 = 0;
-
-  /* For GCC 3.x and stabs, constructors and destructors
-     have names like __base_ctor and __complete_dtor.
-     Check the physname for now if we're looking for a
-     constructor.  */
-  for (field_counter
-        = TYPE_FN_FIELDLIST_LENGTH (t, method_counter) - 1;
-       field_counter >= 0;
-       --field_counter)
-    {
-      struct fn_field *f;
-      const char *phys_name;
-
-      f = TYPE_FN_FIELDLIST1 (t, method_counter);
-
-      /* GCC 3.x will never produce stabs stub methods, so
-        we don't need to handle this case.  */
-      if (TYPE_FN_FIELD_STUB (f, field_counter))
-       continue;
-      phys_name = TYPE_FN_FIELD_PHYSNAME (f, field_counter);
-      if (! is_constructor_name (phys_name))
-       continue;
-
-      /* If this method is actually defined, include it in the
-        list.  */
-      sym_arr[i1] = lookup_symbol_in_language (phys_name,
-                                  NULL, VAR_DOMAIN,
-                                  language,
-                                  (int *) NULL);
-      if (sym_arr[i1])
-       i1++;
-    }
-
-  return i1;
-}
-
-/* Helper function for decode_line_1.
-   Build a canonical line spec in CANONICAL if it is non-NULL and if
-   the SAL has a symtab.
-   If SYMNAME is non-NULL the canonical line spec is `filename:symname'.
-   If SYMNAME is NULL the line number from SAL is used and the canonical
-   line spec is `filename:linenum'.  */
+static int symbol_to_sal (struct symtab_and_line *result,
+                         int funfirstline, struct symbol *sym);
 
-static void
-build_canonical_line_spec (struct symtab_and_line *sal, char *symname,
-                          struct linespec_result *canonical)
-{
-  char **canonical_arr;
-  char *canonical_name;
-  char *filename;
-  struct symtab *s = sal->symtab;
+static void add_matching_symbols_to_info (const char *name,
+                                         struct collect_info *info,
+                                         struct program_space *pspace);
 
-  if (s == (struct symtab *) NULL
-      || s->filename == (char *) NULL
-      || canonical == NULL)
-    return;
+static void add_all_symbol_names_from_pspace (struct collect_info *info,
+                                             struct program_space *pspace,
+                                             VEC (const_char_ptr) *names);
 
-  canonical_arr = (char **) xmalloc (sizeof (char *));
-  canonical->canonical = canonical_arr;
+static VEC (symtab_ptr) *collect_symtabs_from_filename (const char *file);
 
-  filename = s->filename;
-  if (symname != NULL)
-    {
-      canonical_name = xmalloc (strlen (filename) + strlen (symname) + 2);
-      sprintf (canonical_name, "%s:%s", filename, symname);
-    }
-  else
-    {
-      canonical_name = xmalloc (strlen (filename) + 30);
-      sprintf (canonical_name, "%s:%d", filename, sal->line);
-    }
-  canonical_arr[0] = canonical_name;
-}
+static void decode_digits_ordinary (struct linespec_state *self,
+                                   linespec_p ls,
+                                   int line,
+                                   struct symtabs_and_lines *sals,
+                                   struct linetable_entry **best_entry);
 
+static void decode_digits_list_mode (struct linespec_state *self,
+                                    linespec_p ls,
+                                    struct symtabs_and_lines *values,
+                                    struct symtab_and_line val);
 
+static void minsym_found (struct linespec_state *self, struct objfile *objfile,
+                         struct minimal_symbol *msymbol,
+                         struct symtabs_and_lines *result);
 
-/* Find an instance of the character C in the string S that is outside
-   of all parenthesis pairs, single-quoted strings, and double-quoted
-   strings.  Also, ignore the char within a template name, like a ','
-   within foo<int, int>.  */
+static int compare_symbols (const void *a, const void *b);
 
-static char *
-find_toplevel_char (char *s, char c)
-{
-  int quoted = 0;              /* zero if we're not in quotes;
-                                  '"' if we're in a double-quoted string;
-                                  '\'' if we're in a single-quoted string.  */
-  int depth = 0;               /* Number of unclosed parens we've seen.  */
-  char *scan;
+static int compare_msymbols (const void *a, const void *b);
 
-  for (scan = s; *scan; scan++)
-    {
-      if (quoted)
-       {
-         if (*scan == quoted)
-           quoted = 0;
-         else if (*scan == '\\' && *(scan + 1))
-           scan++;
-       }
-      else if (*scan == c && ! quoted && depth == 0)
-       return scan;
-      else if (*scan == '"' || *scan == '\'')
-       quoted = *scan;
-      else if (*scan == '(' || *scan == '<')
-       depth++;
-      else if ((*scan == ')' || *scan == '>') && depth > 0)
-       depth--;
-    }
+static const char *find_toplevel_char (const char *s, char c);
 
-  return 0;
-}
+/* Permitted quote characters for the parser.  This is different from the
+   completer's quote characters to allow backward compatibility with the
+   previous parser.  */
+static const char *const linespec_quote_characters = "\"\'";
 
-/* Determines if the gives string corresponds to an Objective-C method
-   representation, such as -[Foo bar:] or +[Foo bar].  Objective-C symbols
-   are allowed to have spaces and parentheses in them.  */
+/* Lexer functions.  */
 
-static int 
-is_objc_method_format (const char *s)
-{
-  if (s == NULL || *s == '\0')
-    return 0;
-  /* Handle arguments with the format FILENAME:SYMBOL.  */
-  if ((s[0] == ':') && (strchr ("+-", s[1]) != NULL) 
-      && (s[2] == '[') && strchr(s, ']'))
-    return 1;
-  /* Handle arguments that are just SYMBOL.  */
-  else if ((strchr ("+-", s[0]) != NULL) && (s[1] == '[') && strchr(s, ']'))
-    return 1;
-  return 0;
-}
+/* Lex a number from the input in PARSER.  This only supports
+   decimal numbers.
 
-/* Given a list of NELTS symbols in SYM_ARR, return a list of lines to
-   operate on (ask user if necessary).
-   If CANONICAL is non-NULL return a corresponding array of mangled names
-   as canonical line specs there.  */
+   Return true if input is decimal numbers.  Return false if not.  */
 
-static struct symtabs_and_lines
-decode_line_2 (struct symbol *sym_arr[], int nelts, int funfirstline,
-              struct linespec_result *canonical)
+static int
+linespec_lexer_lex_number (linespec_parser *parser, linespec_token *tokenp)
 {
-  struct symtabs_and_lines values, return_values;
-  char *args, *arg1;
-  int i;
-  char *prompt;
-  char *symname;
-  struct cleanup *old_chain;
-  char **canonical_arr = (char **) NULL;
-  const char *select_mode = multiple_symbols_select_mode ();
-
-  if (select_mode == multiple_symbols_cancel)
-    error (_("canceled because the command is ambiguous\n"
-            "See set/show multiple-symbol."));
-  
-  values.sals = (struct symtab_and_line *)
-    alloca (nelts * sizeof (struct symtab_and_line));
-  return_values.sals = (struct symtab_and_line *)
-    xmalloc (nelts * sizeof (struct symtab_and_line));
-  old_chain = make_cleanup (xfree, return_values.sals);
+  tokenp->type = LSTOKEN_NUMBER;
+  LS_TOKEN_STOKEN (*tokenp).length = 0;
+  LS_TOKEN_STOKEN (*tokenp).ptr = PARSER_STREAM (parser);
 
-  if (canonical)
+  /* Keep any sign at the start of the stream.  */
+  if (*PARSER_STREAM (parser) == '+' || *PARSER_STREAM (parser) == '-')
     {
-      canonical_arr = (char **) xmalloc (nelts * sizeof (char *));
-      make_cleanup (xfree, canonical_arr);
-      memset (canonical_arr, 0, nelts * sizeof (char *));
-      canonical->canonical = canonical_arr;
+      ++LS_TOKEN_STOKEN (*tokenp).length;
+      ++(PARSER_STREAM (parser));
     }
 
-  i = 0;
-  while (i < nelts)
+  while (isdigit (*PARSER_STREAM (parser)))
     {
-      init_sal (&return_values.sals[i]);       /* Initialize to zeroes.  */
-      init_sal (&values.sals[i]);
-      if (sym_arr[i] && SYMBOL_CLASS (sym_arr[i]) == LOC_BLOCK)
-       values.sals[i] = find_function_start_sal (sym_arr[i], funfirstline);
-      i++;
+      ++LS_TOKEN_STOKEN (*tokenp).length;
+      ++(PARSER_STREAM (parser));
     }
 
-  /* If select_mode is "all", then do not print the multiple-choice
-     menu and act as if the user had chosen choice "1" (all).  */
-  if (select_mode == multiple_symbols_all
-      || ui_out_is_mi_like_p (interp_ui_out (top_level_interpreter ())))
-    args = "1";
-  else
+  /* If the next character in the input buffer is not a space, comma,
+     quote, or colon, this input does not represent a number.  */
+  if (*PARSER_STREAM (parser) != '\0'
+      && !isspace (*PARSER_STREAM (parser)) && *PARSER_STREAM (parser) != ','
+      && *PARSER_STREAM (parser) != ':'
+      && !strchr (linespec_quote_characters, *PARSER_STREAM (parser)))
     {
-      i = 0;
-      printf_unfiltered (_("[0] cancel\n[1] all\n"));
-      while (i < nelts)
-        {
-          if (sym_arr[i] && SYMBOL_CLASS (sym_arr[i]) == LOC_BLOCK)
-            {
-              if (values.sals[i].symtab)
-                printf_unfiltered ("[%d] %s at %s:%d\n",
-                                   (i + 2),
-                                   SYMBOL_PRINT_NAME (sym_arr[i]),
-                                   values.sals[i].symtab->filename,
-                                   values.sals[i].line);
-              else
-                printf_unfiltered (_("[%d] %s at ?FILE:%d [No symtab? "
-                                    "Probably broken debug info...]\n"),
-                                   (i + 2),
-                                   SYMBOL_PRINT_NAME (sym_arr[i]),
-                                   values.sals[i].line);
-
-            }
-          else
-            printf_unfiltered (_("?HERE\n"));
-          i++;
-        }
-
-      prompt = getenv ("PS2");
-      if (prompt == NULL)
-        {
-          prompt = "> ";
-        }
-      args = command_line_input (prompt, 0, "overload-choice");
+      PARSER_STREAM (parser) = LS_TOKEN_STOKEN (*tokenp).ptr;
+      return 0;
     }
 
-  if (args == 0 || *args == 0)
-    error_no_arg (_("one or more choice numbers"));
-
-  i = 0;
-  while (*args)
-    {
-      int num;
+  return 1;
+}
 
-      arg1 = args;
-      while (*arg1 >= '0' && *arg1 <= '9')
-       arg1++;
-      if (*arg1 && *arg1 != ' ' && *arg1 != '\t')
-       error (_("Arguments must be choice numbers."));
+/* See linespec.h.  */
 
-      num = atoi (args);
+const char *
+linespec_lexer_lex_keyword (const char *p)
+{
+  int i;
 
-      if (num == 0)
-       error (_("canceled"));
-      else if (num == 1)
+  if (p != NULL)
+    {
+      for (i = 0; i < ARRAY_SIZE (linespec_keywords); ++i)
        {
-         if (canonical_arr)
+         int len = strlen (linespec_keywords[i]);
+
+         /* If P begins with one of the keywords and the next
+            character is whitespace, we may have found a keyword.
+            It is only a keyword if it is not followed by another
+            keyword.  */
+         if (strncmp (p, linespec_keywords[i], len) == 0
+             && isspace (p[len]))
            {
-             for (i = 0; i < nelts; i++)
+             int j;
+
+             /* Special case: "if" ALWAYS stops the lexer, since it
+                is not possible to predict what is going to appear in
+                the condition, which can only be parsed after SaLs have
+                been found.  */
+             if (i != IF_KEYWORD_INDEX)
                {
-                 if (canonical_arr[i] == NULL)
+                 p += len;
+                 p = skip_spaces_const (p);
+                 for (j = 0; j < ARRAY_SIZE (linespec_keywords); ++j)
                    {
-                     symname = SYMBOL_LINKAGE_NAME (sym_arr[i]);
-                     canonical_arr[i] = xstrdup (symname);
+                     int nextlen = strlen (linespec_keywords[j]);
+
+                     if (strncmp (p, linespec_keywords[j], nextlen) == 0
+                         && isspace (p[nextlen]))
+                       return NULL;
                    }
                }
-           }
-         memcpy (return_values.sals, values.sals,
-                 (nelts * sizeof (struct symtab_and_line)));
-         return_values.nelts = nelts;
-         discard_cleanups (old_chain);
-         return return_values;
-       }
 
-      if (num >= nelts + 2)
-       {
-         printf_unfiltered (_("No choice number %d.\n"), num);
-       }
-      else
-       {
-         num -= 2;
-         if (values.sals[num].pc)
-           {
-             if (canonical_arr)
-               {
-                 symname = SYMBOL_LINKAGE_NAME (sym_arr[num]);
-                 make_cleanup (xfree, symname);
-                 canonical_arr[i] = xstrdup (symname);
-               }
-             return_values.sals[i++] = values.sals[num];
-             values.sals[num].pc = 0;
-           }
-         else
-           {
-             printf_unfiltered (_("duplicate request for %d ignored.\n"),
-                                num);
+             return linespec_keywords[i];
            }
        }
+    }
+
+  return NULL;
+}
+
+/* Does STRING represent an Ada operator?  If so, return the length
+   of the decoded operator name.  If not, return 0.  */
+
+static int
+is_ada_operator (const char *string)
+{
+  const struct ada_opname_map *mapping;
+
+  for (mapping = ada_opname_table;
+       mapping->encoded != NULL
+        && !startswith (string, mapping->decoded); ++mapping)
+    ;
+
+  return mapping->decoded == NULL ? 0 : strlen (mapping->decoded);
+}
 
-      args = arg1;
-      while (*args == ' ' || *args == '\t')
-       args++;
+/* Find QUOTE_CHAR in STRING, accounting for the ':' terminal.  Return
+   the location of QUOTE_CHAR, or NULL if not found.  */
+
+static const char *
+skip_quote_char (const char *string, char quote_char)
+{
+  const char *p, *last;
+
+  p = last = find_toplevel_char (string, quote_char);
+  while (p && *p != '\0' && *p != ':')
+    {
+      p = find_toplevel_char (p, quote_char);
+      if (p != NULL)
+       last = p++;
     }
-  return_values.nelts = i;
-  discard_cleanups (old_chain);
-  return return_values;
+
+  return last;
 }
 
-/* A helper function for decode_line_1 and friends which skips P
-   past any method overload information at the beginning of P, e.g.,
-   "(const struct foo *)".
+/* Make a writable copy of the string given in TOKEN, trimming
+   any trailing whitespace.  */
 
-   This function assumes that P has already been validated to contain
-   overload information, and it will assert if *P != '('.  */
 static char *
-find_method_overload_end (char *p)
+copy_token_string (linespec_token token)
+{
+  char *str, *s;
+
+  if (token.type == LSTOKEN_KEYWORD)
+    return xstrdup (LS_TOKEN_KEYWORD (token));
+
+  str = savestring (LS_TOKEN_STOKEN (token).ptr,
+                   LS_TOKEN_STOKEN (token).length);
+  s = remove_trailing_whitespace (str, str + LS_TOKEN_STOKEN (token).length);
+  *s = '\0';
+
+  return str;
+}
+
+/* Does P represent the end of a quote-enclosed linespec?  */
+
+static int
+is_closing_quote_enclosed (const char *p)
 {
-  int depth = 0;
+  if (strchr (linespec_quote_characters, *p))
+    ++p;
+  p = skip_spaces ((char *) p);
+  return (*p == '\0' || linespec_lexer_lex_keyword (p));
+}
+
+/* Find the end of the parameter list that starts with *INPUT.
+   This helper function assists with lexing string segments
+   which might contain valid (non-terminating) commas.  */
 
-  gdb_assert (*p == '(');
+static const char *
+find_parameter_list_end (const char *input)
+{
+  char end_char, start_char;
+  int depth;
+  const char *p;
+
+  start_char = *input;
+  if (start_char == '(')
+    end_char = ')';
+  else if (start_char == '<')
+    end_char = '>';
+  else
+    return NULL;
 
+  p = input;
+  depth = 0;
   while (*p)
     {
-      if (*p == '(')
+      if (*p == start_char)
        ++depth;
-      else if (*p == ')')
+      else if (*p == end_char)
        {
          if (--depth == 0)
            {
@@ -683,365 +523,2158 @@ find_method_overload_end (char *p)
   return p;
 }
 
-/* Does P point to a sequence of characters which implies the end
-   of a name?  Terminals include "if" and "thread" clauses. */
 
-static int
-name_end (char *p)
+/* Lex a string from the input in PARSER.  */
+
+static linespec_token
+linespec_lexer_lex_string (linespec_parser *parser)
 {
-  while (isspace (*p))
-    ++p;
-  if (*p == 'i' && p[1] == 'f'
-      && (isspace (p[2]) || p[2] == '\0' || p[2] == '('))
-    return 1;
+  linespec_token token;
+  const char *start = PARSER_STREAM (parser);
 
-  if (strncmp (p, "thread", 6) == 0
-      && (isspace (p[6]) || p[6] == '\0'))
-    return 1;
+  token.type = LSTOKEN_STRING;
 
-  return 0;
-}
+  /* If the input stream starts with a quote character, skip to the next
+     quote character, regardless of the content.  */
+  if (strchr (linespec_quote_characters, *PARSER_STREAM (parser)))
+    {
+      const char *end;
+      char quote_char = *PARSER_STREAM (parser);
 
-/* Keep important information used when looking up a name.  This includes
-   template parameters, overload information, and important keywords.  */
+      /* Special case: Ada operators.  */
+      if (PARSER_STATE (parser)->language->la_language == language_ada
+         && quote_char == '\"')
+       {
+         int len = is_ada_operator (PARSER_STREAM (parser));
 
-static char *
-keep_name_info (char *ptr)
-{
-  char *p = ptr;
-  char *start = ptr;
+         if (len != 0)
+           {
+             /* The input is an Ada operator.  Return the quoted string
+                as-is.  */
+             LS_TOKEN_STOKEN (token).ptr = PARSER_STREAM (parser);
+             LS_TOKEN_STOKEN (token).length = len;
+             PARSER_STREAM (parser) += len;
+             return token;
+           }
 
-  /* Keep any template parameters.  */
-  if (name_end (ptr))
-    return remove_trailing_whitespace (start, ptr);
+         /* The input does not represent an Ada operator -- fall through
+            to normal quoted string handling.  */
+       }
 
-  p = skip_spaces (p);
-  if (*p == '<')
-    ptr = p = find_template_name_end (ptr);
+      /* Skip past the beginning quote.  */
+      ++(PARSER_STREAM (parser));
 
-  if (name_end (ptr))
-    return remove_trailing_whitespace (start, ptr);
+      /* Mark the start of the string.  */
+      LS_TOKEN_STOKEN (token).ptr = PARSER_STREAM (parser);
 
-  /* Keep method overload information.  */
-  if (*p == '(')
-    ptr = p = find_method_overload_end (p);
+      /* Skip to the ending quote.  */
+      end = skip_quote_char (PARSER_STREAM (parser), quote_char);
 
-  if (name_end (ptr))
-    return remove_trailing_whitespace (start, ptr);
+      /* Error if the input did not terminate properly.  */
+      if (end == NULL)
+       error (_("unmatched quote"));
 
-  /* Keep important keywords.  */  
-  while (1)
-    {
-      char *quotes = get_gdb_completer_quote_characters ();
-      p = skip_spaces (p);
-      if (strncmp (p, "const", 5) == 0
-         && (isspace (p[5]) || p[5] == '\0'
-             || strchr (quotes, p[5]) != NULL))
-       ptr = p = p + 5;
-      else if (strncmp (p, "volatile", 8) == 0
-              && (isspace (p[8]) || p[8] == '\0'
-                  || strchr (quotes, p[8]) != NULL))
-       ptr = p = p + 8;
-      else
-       break;
+      /* Skip over the ending quote and mark the length of the string.  */
+      PARSER_STREAM (parser) = (char *) ++end;
+      LS_TOKEN_STOKEN (token).length = PARSER_STREAM (parser) - 2 - start;
     }
+  else
+    {
+      const char *p;
 
-  return remove_trailing_whitespace (start, ptr);
-}
+      /* Otherwise, only identifier characters are permitted.
+        Spaces are the exception.  In general, we keep spaces,
+        but only if the next characters in the input do not resolve
+        to one of the keywords.
 
-\f
-/* The parser of linespec itself.  */
+        This allows users to forgo quoting CV-qualifiers, template arguments,
+        and similar common language constructs.  */
 
-/* Parse a string that specifies a line number.
-   Pass the address of a char * variable; that variable will be
-   advanced over the characters actually parsed.
+      while (1)
+       {
+         if (isspace (*PARSER_STREAM (parser)))
+           {
+             p = skip_spaces_const (PARSER_STREAM (parser));
+             /* When we get here we know we've found something followed by
+                a space (we skip over parens and templates below).
+                So if we find a keyword now, we know it is a keyword and not,
+                say, a function name.  */
+             if (linespec_lexer_lex_keyword (p) != NULL)
+               {
+                 LS_TOKEN_STOKEN (token).ptr = start;
+                 LS_TOKEN_STOKEN (token).length
+                   = PARSER_STREAM (parser) - start;
+                 return token;
+               }
 
-   The string can be:
+             /* Advance past the whitespace.  */
+             PARSER_STREAM (parser) = p;
+           }
 
-   LINENUM -- that line number in current file.  PC returned is 0.
-   FILE:LINENUM -- that line in that file.  PC returned is 0.
-   FUNCTION -- line number of openbrace of that function.
-   PC returned is the start of the function.
-   LABEL -- a label in the current scope
-   VARIABLE -- line number of definition of that variable.
-   PC returned is 0.
-   FILE:FUNCTION -- likewise, but prefer functions in that file.
-   *EXPR -- line in which address EXPR appears.
+         /* If the next character is EOI or (single) ':', the
+            string is complete;  return the token.  */
+         if (*PARSER_STREAM (parser) == 0)
+           {
+             LS_TOKEN_STOKEN (token).ptr = start;
+             LS_TOKEN_STOKEN (token).length = PARSER_STREAM (parser) - start;
+             return token;
+           }
+         else if (PARSER_STREAM (parser)[0] == ':')
+           {
+             /* Do not tokenize the C++ scope operator. */
+             if (PARSER_STREAM (parser)[1] == ':')
+               ++(PARSER_STREAM (parser));
+
+             /* Do not tokenify if the input length so far is one
+                (i.e, a single-letter drive name) and the next character
+                is a directory separator.  This allows Windows-style
+                paths to be recognized as filenames without quoting it.  */
+             else if ((PARSER_STREAM (parser) - start) != 1
+                      || !IS_DIR_SEPARATOR (PARSER_STREAM (parser)[1]))
+               {
+                 LS_TOKEN_STOKEN (token).ptr = start;
+                 LS_TOKEN_STOKEN (token).length
+                   = PARSER_STREAM (parser) - start;
+                 return token;
+               }
+           }
+         /* Special case: permit quote-enclosed linespecs.  */
+         else if (parser->is_quote_enclosed
+                  && strchr (linespec_quote_characters,
+                             *PARSER_STREAM (parser))
+                  && is_closing_quote_enclosed (PARSER_STREAM (parser)))
+           {
+             LS_TOKEN_STOKEN (token).ptr = start;
+             LS_TOKEN_STOKEN (token).length = PARSER_STREAM (parser) - start;
+             return token;
+           }
+         /* Because commas may terminate a linespec and appear in
+            the middle of valid string input, special cases for
+            '<' and '(' are necessary.  */
+         else if (*PARSER_STREAM (parser) == '<'
+                  || *PARSER_STREAM (parser) == '(')
+           {
+             const char *p;
 
-   This may all be followed by an "if EXPR", which we ignore.
+             p = find_parameter_list_end (PARSER_STREAM (parser));
+             if (p != NULL)
+               {
+                 PARSER_STREAM (parser) = p;
+                 continue;
+               }
+           }
+         /* Commas are terminators, but not if they are part of an
+            operator name.  */
+         else if (*PARSER_STREAM (parser) == ',')
+           {
+             if ((PARSER_STATE (parser)->language->la_language
+                  == language_cplus)
+                 && (PARSER_STREAM (parser) - start) > 8
+                 /* strlen ("operator") */)
+               {
+                 char *p = strstr (start, "operator");
 
-   FUNCTION may be an undebuggable function found in minimal symbol table.
+                 if (p != NULL && is_operator_name (p))
+                   {
+                     /* This is an operator name.  Keep going.  */
+                     ++(PARSER_STREAM (parser));
+                     continue;
+                   }
+               }
 
-   If the argument FUNFIRSTLINE is nonzero, we want the first line
-   of real code inside a function when a function is specified, and it is
-   not OK to specify a variable or type to get its line number.
+             /* Comma terminates the string.  */
+             LS_TOKEN_STOKEN (token).ptr = start;
+             LS_TOKEN_STOKEN (token).length = PARSER_STREAM (parser) - start;
+             return token;
+           }
 
-   DEFAULT_SYMTAB specifies the file to use if none is specified.
-   It defaults to current_source_symtab.
-   DEFAULT_LINE specifies the line number to use for relative
-   line numbers (that start with signs).  Defaults to current_source_line.
-   If CANONICAL is non-NULL, store an array of strings containing the canonical
-   line specs there if necessary.  Currently overloaded member functions and
-   line numbers or static functions without a filename yield a canonical
-   line spec.  The array and the line spec strings are allocated on the heap,
-   it is the callers responsibility to free them.
+         /* Advance the stream.  */
+         ++(PARSER_STREAM (parser));
+       }
+    }
 
-   Note that it is possible to return zero for the symtab
-   if no file is validly specified.  Callers must check that.
-   Also, the line number returned may be invalid.  */
+  return token;
+}
 
-/* 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.  */
+/* Lex a single linespec token from PARSER.  */
 
-struct symtabs_and_lines
-decode_line_1 (char **argptr, int funfirstline, struct symtab *default_symtab,
-              int default_line, struct linespec_result *canonical)
-{
-  char *p;
-  char *q;
-  /* If a file name is specified, this is its symtab.  */
-  struct symtab *file_symtab = NULL;
-
-  char *copy;
-  /* This says whether or not something in *ARGPTR is quoted with
-     completer_quotes (i.e. with single quotes).  */
-  int is_quoted;
-  /* Is *ARGPTR is enclosed in double quotes?  */
-  int is_quote_enclosed;
-  int is_objc_method = 0;
-  char *saved_arg = *argptr;
-  /* If IS_QUOTED, the end of the quoted bit.  */
-  char *end_quote = NULL;
-  /* The "first half" of the linespec.  */
-  char *first_half;
-
-  /* If we are parsing `function:label', this holds the symbol for the
-     function.  */
-  struct symbol *function_symbol = NULL;
-  /* If FUNCTION_SYMBOL is not NULL, then this is the exception that
-     was thrown when trying to parse a filename.  */
-  volatile struct gdb_exception file_exception;
-
-  /* Defaults have defaults.  */
-
-  initialize_defaults (&default_symtab, &default_line);
-  
-  /* See if arg is *PC.  */
+static linespec_token
+linespec_lexer_lex_one (linespec_parser *parser)
+{
+  const char *keyword;
 
-  if (**argptr == '*')
-    return decode_indirect (argptr);
+  if (parser->lexer.current.type == LSTOKEN_CONSUMED)
+    {
+      /* Skip any whitespace.  */
+      PARSER_STREAM (parser) = skip_spaces_const (PARSER_STREAM (parser));
 
-  is_quoted = (strchr (get_gdb_completer_quote_characters (),
-                      **argptr) != NULL);
+      /* Check for a keyword, they end the linespec.  */
+      keyword = linespec_lexer_lex_keyword (PARSER_STREAM (parser));
+      if (keyword != NULL)
+       {
+         parser->lexer.current.type = LSTOKEN_KEYWORD;
+         LS_TOKEN_KEYWORD (parser->lexer.current) = keyword;
+         /* We do not advance the stream here intentionally:
+            we would like lexing to stop when a keyword is seen.
 
-  if (is_quoted)
-    end_quote = skip_quoted (*argptr);
+            PARSER_STREAM (parser) +=  strlen (keyword);  */
 
-  /* Check to see if it's a multipart linespec (with colons or
-     periods).  */
+         return parser->lexer.current;
+       }
 
-  /* Locate the end of the first half of the linespec.
-     After the call, for instance, if the argptr string is "foo.c:123"
-     p will point at "123".  If there is only one part, like "foo", p
-     will point to "".  If this is a C++ name, like "A::B::foo", p will
-     point to "::B::foo".  Argptr is not changed by this call.  */
+      /* Handle other tokens.  */
+      switch (*PARSER_STREAM (parser))
+       {
+       case 0:
+         parser->lexer.current.type = LSTOKEN_EOI;
+         break;
 
-  first_half = p = locate_first_half (argptr, &is_quote_enclosed);
+       case '+': case '-':
+       case '0': case '1': case '2': case '3': case '4':
+        case '5': case '6': case '7': case '8': case '9':
+           if (!linespec_lexer_lex_number (parser, &(parser->lexer.current)))
+            parser->lexer.current = linespec_lexer_lex_string (parser);
+          break;
+
+       case ':':
+         /* If we have a scope operator, lex the input as a string.
+            Otherwise, return LSTOKEN_COLON.  */
+         if (PARSER_STREAM (parser)[1] == ':')
+           parser->lexer.current = linespec_lexer_lex_string (parser);
+         else
+           {
+             parser->lexer.current.type = LSTOKEN_COLON;
+             ++(PARSER_STREAM (parser));
+           }
+         break;
+
+       case '\'': case '\"':
+         /* Special case: permit quote-enclosed linespecs.  */
+         if (parser->is_quote_enclosed
+             && is_closing_quote_enclosed (PARSER_STREAM (parser)))
+           {
+             ++(PARSER_STREAM (parser));
+             parser->lexer.current.type = LSTOKEN_EOI;
+           }
+         else
+           parser->lexer.current = linespec_lexer_lex_string (parser);
+         break;
+
+       case ',':
+         parser->lexer.current.type = LSTOKEN_COMMA;
+         LS_TOKEN_STOKEN (parser->lexer.current).ptr
+           = PARSER_STREAM (parser);
+         LS_TOKEN_STOKEN (parser->lexer.current).length = 1;
+         ++(PARSER_STREAM (parser));
+         break;
+
+       default:
+         /* If the input is not a number, it must be a string.
+            [Keywords were already considered above.]  */
+         parser->lexer.current = linespec_lexer_lex_string (parser);
+         break;
+       }
+    }
+
+  return parser->lexer.current;
+}
+
+/* Consume the current token and return the next token in PARSER's
+   input stream.  */
+
+static linespec_token
+linespec_lexer_consume_token (linespec_parser *parser)
+{
+  parser->lexer.current.type = LSTOKEN_CONSUMED;
+  return linespec_lexer_lex_one (parser);
+}
+
+/* Return the next token without consuming the current token.  */
+
+static linespec_token
+linespec_lexer_peek_token (linespec_parser *parser)
+{
+  linespec_token next;
+  const char *saved_stream = PARSER_STREAM (parser);
+  linespec_token saved_token = parser->lexer.current;
+
+  next = linespec_lexer_consume_token (parser);
+  PARSER_STREAM (parser) = saved_stream;
+  parser->lexer.current = saved_token;
+  return next;
+}
+
+/* Helper functions.  */
+
+/* Add SAL to SALS.  */
+
+static void
+add_sal_to_sals_basic (struct symtabs_and_lines *sals,
+                      struct symtab_and_line *sal)
+{
+  ++sals->nelts;
+  sals->sals = xrealloc (sals->sals, sals->nelts * sizeof (sals->sals[0]));
+  sals->sals[sals->nelts - 1] = *sal;
+}
+
+/* Add SAL to SALS, and also update SELF->CANONICAL_NAMES to reflect
+   the new sal, if needed.  If not NULL, SYMNAME is the name of the
+   symbol to use when constructing the new canonical name.
+
+   If LITERAL_CANONICAL is non-zero, SYMNAME will be used as the
+   canonical name for the SAL.  */
+
+static void
+add_sal_to_sals (struct linespec_state *self,
+                struct symtabs_and_lines *sals,
+                struct symtab_and_line *sal,
+                const char *symname, int literal_canonical)
+{
+  add_sal_to_sals_basic (sals, sal);
+
+  if (self->canonical)
+    {
+      struct linespec_canonical_name *canonical;
+
+      self->canonical_names = xrealloc (self->canonical_names,
+                                       (sals->nelts
+                                        * sizeof (*self->canonical_names)));
+      canonical = &self->canonical_names[sals->nelts - 1];
+      if (!literal_canonical && sal->symtab)
+       {
+         const char *fullname = symtab_to_fullname (sal->symtab);
+
+         /* Note that the filter doesn't have to be a valid linespec
+            input.  We only apply the ":LINE" treatment to Ada for
+            the time being.  */
+         if (symname != NULL && sal->line != 0
+             && self->language->la_language == language_ada)
+           canonical->suffix = xstrprintf ("%s:%d", symname, sal->line);
+         else if (symname != NULL)
+           canonical->suffix = xstrdup (symname);
+         else
+           canonical->suffix = xstrprintf ("%d", sal->line);
+         canonical->symtab = sal->symtab;
+       }
+      else
+       {
+         if (symname != NULL)
+           canonical->suffix = xstrdup (symname);
+         else
+           canonical->suffix = xstrdup ("<unknown>");
+         canonical->symtab = NULL;
+       }
+    }
+}
+
+/* A hash function for address_entry.  */
+
+static hashval_t
+hash_address_entry (const void *p)
+{
+  const struct address_entry *aep = p;
+  hashval_t hash;
+
+  hash = iterative_hash_object (aep->pspace, 0);
+  return iterative_hash_object (aep->addr, hash);
+}
+
+/* An equality function for address_entry.  */
+
+static int
+eq_address_entry (const void *a, const void *b)
+{
+  const struct address_entry *aea = a;
+  const struct address_entry *aeb = b;
+
+  return aea->pspace == aeb->pspace && aea->addr == aeb->addr;
+}
+
+/* Check whether the address, represented by PSPACE and ADDR, is
+   already in the set.  If so, return 0.  Otherwise, add it and return
+   1.  */
+
+static int
+maybe_add_address (htab_t set, struct program_space *pspace, CORE_ADDR addr)
+{
+  struct address_entry e, *p;
+  void **slot;
+
+  e.pspace = pspace;
+  e.addr = addr;
+  slot = htab_find_slot (set, &e, INSERT);
+  if (*slot)
+    return 0;
+
+  p = XNEW (struct address_entry);
+  memcpy (p, &e, sizeof (struct address_entry));
+  *slot = p;
+
+  return 1;
+}
+
+/* A callback function and the additional data to call it with.  */
+
+struct symbol_and_data_callback
+{
+  /* The callback to use.  */
+  symbol_found_callback_ftype *callback;
+
+  /* Data to be passed to the callback.  */
+  void *data;
+};
+
+/* A helper for iterate_over_all_matching_symtabs that is used to
+   restrict calls to another callback to symbols representing inline
+   symbols only.  */
+
+static int
+iterate_inline_only (struct symbol *sym, void *d)
+{
+  if (SYMBOL_INLINED (sym))
+    {
+      struct symbol_and_data_callback *cad = d;
+
+      return cad->callback (sym, cad->data);
+    }
+  return 1; /* Continue iterating.  */
+}
+
+/* Some data for the expand_symtabs_matching callback.  */
+
+struct symbol_matcher_data
+{
+  /* The lookup name against which symbol name should be compared.  */
+  const char *lookup_name;
+
+  /* The routine to be used for comparison.  */
+  symbol_name_cmp_ftype symbol_name_cmp;
+};
+
+/* A helper for iterate_over_all_matching_symtabs that is passed as a
+   callback to the expand_symtabs_matching method.  */
+
+static int
+iterate_name_matcher (const char *name, void *d)
+{
+  const struct symbol_matcher_data *data = d;
+
+  if (data->symbol_name_cmp (name, data->lookup_name) == 0)
+    return 1; /* Expand this symbol's symbol table.  */
+  return 0; /* Skip this symbol.  */
+}
+
+/* A helper that walks over all matching symtabs in all objfiles and
+   calls CALLBACK for each symbol matching NAME.  If SEARCH_PSPACE is
+   not NULL, then the search is restricted to just that program
+   space.  If INCLUDE_INLINE is nonzero then symbols representing
+   inlined instances of functions will be included in the result.  */
+
+static void
+iterate_over_all_matching_symtabs (struct linespec_state *state,
+                                  const char *name,
+                                  const domain_enum domain,
+                                  symbol_found_callback_ftype *callback,
+                                  void *data,
+                                  struct program_space *search_pspace,
+                                  int include_inline)
+{
+  struct objfile *objfile;
+  struct program_space *pspace;
+  struct symbol_matcher_data matcher_data;
+
+  matcher_data.lookup_name = name;
+  matcher_data.symbol_name_cmp =
+    state->language->la_get_symbol_name_cmp != NULL
+    ? state->language->la_get_symbol_name_cmp (name)
+    : strcmp_iw;
+
+  ALL_PSPACES (pspace)
+  {
+    if (search_pspace != NULL && search_pspace != pspace)
+      continue;
+    if (pspace->executing_startup)
+      continue;
+
+    set_current_program_space (pspace);
+
+    ALL_OBJFILES (objfile)
+    {
+      struct compunit_symtab *cu;
+
+      if (objfile->sf)
+       objfile->sf->qf->expand_symtabs_matching (objfile, NULL,
+                                                 iterate_name_matcher,
+                                                 NULL, ALL_DOMAIN,
+                                                 &matcher_data);
+
+      ALL_OBJFILE_COMPUNITS (objfile, cu)
+       {
+         struct symtab *symtab = COMPUNIT_FILETABS (cu);
+
+         iterate_over_file_blocks (symtab, name, domain, callback, data);
+
+         if (include_inline)
+           {
+             struct symbol_and_data_callback cad = { callback, data };
+             struct block *block;
+             int i;
+
+             for (i = FIRST_LOCAL_BLOCK;
+                  i < BLOCKVECTOR_NBLOCKS (SYMTAB_BLOCKVECTOR (symtab));
+                  i++)
+               {
+                 block = BLOCKVECTOR_BLOCK (SYMTAB_BLOCKVECTOR (symtab), i);
+                 state->language->la_iterate_over_symbols
+                   (block, name, domain, iterate_inline_only, &cad);
+               }
+           }
+       }
+    }
+  }
+}
+
+/* Returns the block to be used for symbol searches from
+   the current location.  */
+
+static const struct block *
+get_current_search_block (void)
+{
+  const struct block *block;
+  enum language save_language;
+
+  /* get_selected_block can change the current language when there is
+     no selected frame yet.  */
+  save_language = current_language->la_language;
+  block = get_selected_block (0);
+  set_language (save_language);
+
+  return block;
+}
+
+/* Iterate over static and global blocks.  */
+
+static void
+iterate_over_file_blocks (struct symtab *symtab,
+                         const char *name, domain_enum domain,
+                         symbol_found_callback_ftype *callback, void *data)
+{
+  struct block *block;
+
+  for (block = BLOCKVECTOR_BLOCK (SYMTAB_BLOCKVECTOR (symtab), STATIC_BLOCK);
+       block != NULL;
+       block = BLOCK_SUPERBLOCK (block))
+    LA_ITERATE_OVER_SYMBOLS (block, name, domain, callback, data);
+}
+
+/* A helper for find_method.  This finds all methods in type T which
+   match NAME.  It adds matching symbol names to RESULT_NAMES, and
+   adds T's direct superclasses to SUPERCLASSES.  */
+
+static void
+find_methods (struct type *t, const char *name,
+             VEC (const_char_ptr) **result_names,
+             VEC (typep) **superclasses)
+{
+  int ibase;
+  const char *class_name = type_name_no_tag (t);
+
+  /* Ignore this class if it doesn't have a name.  This is ugly, but
+     unless we figure out how to get the physname without the name of
+     the class, then the loop can't do any good.  */
+  if (class_name)
+    {
+      int method_counter;
+
+      t = check_typedef (t);
+
+      /* Loop over each method name.  At this level, all overloads of a name
+         are counted as a single name.  There is an inner loop which loops over
+         each overload.  */
+
+      for (method_counter = TYPE_NFN_FIELDS (t) - 1;
+          method_counter >= 0;
+          --method_counter)
+       {
+         const char *method_name = TYPE_FN_FIELDLIST_NAME (t, method_counter);
+         char dem_opname[64];
+
+         if (startswith (method_name, "__") ||
+             startswith (method_name, "op") ||
+             startswith (method_name, "type"))
+           {
+             if (cplus_demangle_opname (method_name, dem_opname, DMGL_ANSI))
+               method_name = dem_opname;
+             else if (cplus_demangle_opname (method_name, dem_opname, 0))
+               method_name = dem_opname;
+           }
+
+         if (strcmp_iw (method_name, name) == 0)
+           {
+             int field_counter;
+
+             for (field_counter = (TYPE_FN_FIELDLIST_LENGTH (t, method_counter)
+                                   - 1);
+                  field_counter >= 0;
+                  --field_counter)
+               {
+                 struct fn_field *f;
+                 const char *phys_name;
+
+                 f = TYPE_FN_FIELDLIST1 (t, method_counter);
+                 if (TYPE_FN_FIELD_STUB (f, field_counter))
+                   continue;
+                 phys_name = TYPE_FN_FIELD_PHYSNAME (f, field_counter);
+                 VEC_safe_push (const_char_ptr, *result_names, phys_name);
+               }
+           }
+       }
+    }
+
+  for (ibase = 0; ibase < TYPE_N_BASECLASSES (t); ibase++)
+    VEC_safe_push (typep, *superclasses, TYPE_BASECLASS (t, ibase));
+}
+
+/* Find an instance of the character C in the string S that is outside
+   of all parenthesis pairs, single-quoted strings, and double-quoted
+   strings.  Also, ignore the char within a template name, like a ','
+   within foo<int, int>.  */
+
+static const char *
+find_toplevel_char (const char *s, char c)
+{
+  int quoted = 0;              /* zero if we're not in quotes;
+                                  '"' if we're in a double-quoted string;
+                                  '\'' if we're in a single-quoted string.  */
+  int depth = 0;               /* Number of unclosed parens we've seen.  */
+  const char *scan;
+
+  for (scan = s; *scan; scan++)
+    {
+      if (quoted)
+       {
+         if (*scan == quoted)
+           quoted = 0;
+         else if (*scan == '\\' && *(scan + 1))
+           scan++;
+       }
+      else if (*scan == c && ! quoted && depth == 0)
+       return scan;
+      else if (*scan == '"' || *scan == '\'')
+       quoted = *scan;
+      else if (*scan == '(' || *scan == '<')
+       depth++;
+      else if ((*scan == ')' || *scan == '>') && depth > 0)
+       depth--;
+    }
+
+  return 0;
+}
+
+/* The string equivalent of find_toplevel_char.  Returns a pointer
+   to the location of NEEDLE in HAYSTACK, ignoring any occurrences
+   inside "()" and "<>".  Returns NULL if NEEDLE was not found.  */
+
+static const char *
+find_toplevel_string (const char *haystack, const char *needle)
+{
+  const char *s = haystack;
+
+  do
+    {
+      s = find_toplevel_char (s, *needle);
+
+      if (s != NULL)
+       {
+         /* Found first char in HAYSTACK;  check rest of string.  */
+         if (startswith (s, needle))
+           return s;
+
+         /* Didn't find it; loop over HAYSTACK, looking for the next
+            instance of the first character of NEEDLE.  */
+         ++s;
+       }
+    }
+  while (s != NULL && *s != '\0');
+
+  /* NEEDLE was not found in HAYSTACK.  */
+  return NULL;
+}
+
+/* Convert CANONICAL to its string representation using
+   symtab_to_fullname for SYMTAB.  The caller must xfree the result.  */
+
+static char *
+canonical_to_fullform (const struct linespec_canonical_name *canonical)
+{
+  if (canonical->symtab == NULL)
+    return xstrdup (canonical->suffix);
+  else
+    return xstrprintf ("%s:%s", symtab_to_fullname (canonical->symtab),
+                      canonical->suffix);
+}
+
+/* Given FILTERS, a list of canonical names, filter the sals in RESULT
+   and store the result in SELF->CANONICAL.  */
+
+static void
+filter_results (struct linespec_state *self,
+               struct symtabs_and_lines *result,
+               VEC (const_char_ptr) *filters)
+{
+  int i;
+  const char *name;
+
+  for (i = 0; VEC_iterate (const_char_ptr, filters, i, name); ++i)
+    {
+      struct linespec_sals lsal;
+      int j;
+
+      memset (&lsal, 0, sizeof (lsal));
+
+      for (j = 0; j < result->nelts; ++j)
+       {
+         const struct linespec_canonical_name *canonical;
+         char *fullform;
+         struct cleanup *cleanup;
+
+         canonical = &self->canonical_names[j];
+         fullform = canonical_to_fullform (canonical);
+         cleanup = make_cleanup (xfree, fullform);
+
+         if (strcmp (name, fullform) == 0)
+           add_sal_to_sals_basic (&lsal.sals, &result->sals[j]);
+
+         do_cleanups (cleanup);
+       }
+
+      if (lsal.sals.nelts > 0)
+       {
+         lsal.canonical = xstrdup (name);
+         VEC_safe_push (linespec_sals, self->canonical->sals, &lsal);
+       }
+    }
+
+  self->canonical->pre_expanded = 0;
+}
+
+/* Store RESULT into SELF->CANONICAL.  */
+
+static void
+convert_results_to_lsals (struct linespec_state *self,
+                         struct symtabs_and_lines *result)
+{
+  struct linespec_sals lsal;
+
+  lsal.canonical = NULL;
+  lsal.sals = *result;
+  VEC_safe_push (linespec_sals, self->canonical->sals, &lsal);
+}
+
+/* A structure that contains two string representations of a struct
+   linespec_canonical_name:
+     - one where the the symtab's fullname is used;
+     - one where the filename followed the "set filename-display"
+       setting.  */
+
+struct decode_line_2_item
+{
+  /* The form using symtab_to_fullname.
+     It must be xfree'ed after use.  */
+  char *fullform;
+
+  /* The form using symtab_to_filename_for_display.
+     It must be xfree'ed after use.  */
+  char *displayform;
+
+  /* Field is initialized to zero and it is set to one if the user
+     requested breakpoint for this entry.  */
+  unsigned int selected : 1;
+};
+
+/* Helper for qsort to sort decode_line_2_item entries by DISPLAYFORM and
+   secondarily by FULLFORM.  */
+
+static int
+decode_line_2_compare_items (const void *ap, const void *bp)
+{
+  const struct decode_line_2_item *a = ap;
+  const struct decode_line_2_item *b = bp;
+  int retval;
+
+  retval = strcmp (a->displayform, b->displayform);
+  if (retval != 0)
+    return retval;
+
+  return strcmp (a->fullform, b->fullform);
+}
+
+/* Handle multiple results in RESULT depending on SELECT_MODE.  This
+   will either return normally, throw an exception on multiple
+   results, or present a menu to the user.  On return, the SALS vector
+   in SELF->CANONICAL is set up properly.  */
+
+static void
+decode_line_2 (struct linespec_state *self,
+              struct symtabs_and_lines *result,
+              const char *select_mode)
+{
+  char *args, *prompt;
+  int i;
+  struct cleanup *old_chain;
+  VEC (const_char_ptr) *filters = NULL;
+  struct get_number_or_range_state state;
+  struct decode_line_2_item *items;
+  int items_count;
+
+  gdb_assert (select_mode != multiple_symbols_all);
+  gdb_assert (self->canonical != NULL);
+  gdb_assert (result->nelts >= 1);
+
+  old_chain = make_cleanup (VEC_cleanup (const_char_ptr), &filters);
+
+  /* Prepare ITEMS array.  */
+  items_count = result->nelts;
+  items = xmalloc (sizeof (*items) * items_count);
+  make_cleanup (xfree, items);
+  for (i = 0; i < items_count; ++i)
+    {
+      const struct linespec_canonical_name *canonical;
+      struct decode_line_2_item *item;
+
+      canonical = &self->canonical_names[i];
+      gdb_assert (canonical->suffix != NULL);
+      item = &items[i];
+
+      item->fullform = canonical_to_fullform (canonical);
+      make_cleanup (xfree, item->fullform);
+
+      if (canonical->symtab == NULL)
+       item->displayform = canonical->suffix;
+      else
+       {
+         const char *fn_for_display;
+
+         fn_for_display = symtab_to_filename_for_display (canonical->symtab);
+         item->displayform = xstrprintf ("%s:%s", fn_for_display,
+                                         canonical->suffix);
+         make_cleanup (xfree, item->displayform);
+       }
+
+      item->selected = 0;
+    }
+
+  /* Sort the list of method names.  */
+  qsort (items, items_count, sizeof (*items), decode_line_2_compare_items);
+
+  /* Remove entries with the same FULLFORM.  */
+  if (items_count >= 2)
+    {
+      struct decode_line_2_item *dst, *src;
+
+      dst = items;
+      for (src = &items[1]; src < &items[items_count]; src++)
+       if (strcmp (src->fullform, dst->fullform) != 0)
+         *++dst = *src;
+      items_count = dst + 1 - items;
+    }
+
+  if (select_mode == multiple_symbols_cancel && items_count > 1)
+    error (_("canceled because the command is ambiguous\n"
+            "See set/show multiple-symbol."));
+  
+  if (select_mode == multiple_symbols_all || items_count == 1)
+    {
+      do_cleanups (old_chain);
+      convert_results_to_lsals (self, result);
+      return;
+    }
+
+  printf_unfiltered (_("[0] cancel\n[1] all\n"));
+  for (i = 0; i < items_count; i++)
+    printf_unfiltered ("[%d] %s\n", i + 2, items[i].displayform);
+
+  prompt = getenv ("PS2");
+  if (prompt == NULL)
+    {
+      prompt = "> ";
+    }
+  args = command_line_input (prompt, 0, "overload-choice");
+
+  if (args == 0 || *args == 0)
+    error_no_arg (_("one or more choice numbers"));
+
+  init_number_or_range (&state, args);
+  while (!state.finished)
+    {
+      int num;
+
+      num = get_number_or_range (&state);
+
+      if (num == 0)
+       error (_("canceled"));
+      else if (num == 1)
+       {
+         /* We intentionally make this result in a single breakpoint,
+            contrary to what older versions of gdb did.  The
+            rationale is that this lets a user get the
+            multiple_symbols_all behavior even with the 'ask'
+            setting; and he can get separate breakpoints by entering
+            "2-57" at the query.  */
+         do_cleanups (old_chain);
+         convert_results_to_lsals (self, result);
+         return;
+       }
+
+      num -= 2;
+      if (num >= items_count)
+       printf_unfiltered (_("No choice number %d.\n"), num);
+      else
+       {
+         struct decode_line_2_item *item = &items[num];
+
+         if (!item->selected)
+           {
+             VEC_safe_push (const_char_ptr, filters, item->fullform);
+             item->selected = 1;
+           }
+         else
+           {
+             printf_unfiltered (_("duplicate request for %d ignored.\n"),
+                                num + 2);
+           }
+       }
+    }
+
+  filter_results (self, result, filters);
+  do_cleanups (old_chain);
+}
+
+\f
+
+/* The parser of linespec itself.  */
+
+/* Throw an appropriate error when SYMBOL is not found (optionally in
+   FILENAME).  */
+
+static void ATTRIBUTE_NORETURN
+symbol_not_found_error (const char *symbol, const char *filename)
+{
+  if (symbol == NULL)
+    symbol = "";
+
+  if (!have_full_symbols ()
+      && !have_partial_symbols ()
+      && !have_minimal_symbols ())
+    throw_error (NOT_FOUND_ERROR,
+                _("No symbol table is loaded.  Use the \"file\" command."));
+
+  /* If SYMBOL starts with '$', the user attempted to either lookup
+     a function/variable in his code starting with '$' or an internal
+     variable of that name.  Since we do not know which, be concise and
+     explain both possibilities.  */
+  if (*symbol == '$')
+    {
+      if (filename)
+       throw_error (NOT_FOUND_ERROR,
+                    _("Undefined convenience variable or function \"%s\" "
+                      "not defined in \"%s\"."), symbol, filename);
+      else
+       throw_error (NOT_FOUND_ERROR,
+                    _("Undefined convenience variable or function \"%s\" "
+                      "not defined."), symbol);
+    }
+  else
+    {
+      if (filename)
+       throw_error (NOT_FOUND_ERROR,
+                    _("Function \"%s\" not defined in \"%s\"."),
+                    symbol, filename);
+      else
+       throw_error (NOT_FOUND_ERROR,
+                    _("Function \"%s\" not defined."), symbol);
+    }
+}
+
+/* Throw an appropriate error when an unexpected token is encountered 
+   in the input.  */
+
+static void ATTRIBUTE_NORETURN
+unexpected_linespec_error (linespec_parser *parser)
+{
+  linespec_token token;
+  static const char * token_type_strings[]
+    = {"keyword", "colon", "string", "number", "comma", "end of input"};
+
+  /* Get the token that generated the error.  */
+  token = linespec_lexer_lex_one (parser);
+
+  /* Finally, throw the error.  */
+  if (token.type == LSTOKEN_STRING || token.type == LSTOKEN_NUMBER
+      || token.type == LSTOKEN_KEYWORD)
+    {
+      char *string;
+      struct cleanup *cleanup;
+
+      string = copy_token_string (token);
+      cleanup = make_cleanup (xfree, string);
+      throw_error (GENERIC_ERROR,
+                  _("malformed linespec error: unexpected %s, \"%s\""),
+                  token_type_strings[token.type], string);
+    }
+  else
+    throw_error (GENERIC_ERROR,
+                _("malformed linespec error: unexpected %s"),
+                token_type_strings[token.type]);
+}
+
+/* Throw an undefined label error.  */
+
+static void ATTRIBUTE_NORETURN
+undefined_label_error (const char *function, const char *label)
+{
+  if (function != NULL)
+    throw_error (NOT_FOUND_ERROR,
+                _("No label \"%s\" defined in function \"%s\"."),
+                label, function);
+  else
+    throw_error (NOT_FOUND_ERROR,
+                _("No label \"%s\" defined in current function."),
+                label);
+}
+
+/* Throw a source file not found error.  */
+
+static void ATTRIBUTE_NORETURN
+source_file_not_found_error (const char *name)
+{
+  throw_error (NOT_FOUND_ERROR, _("No source file named %s."), name);
+}
+
+/* Parse and return a line offset in STRING.  */
+
+static struct line_offset
+linespec_parse_line_offset (const char *string)
+{
+  struct line_offset line_offset = {0, LINE_OFFSET_NONE};
+
+  if (*string == '+')
+    {
+      line_offset.sign = LINE_OFFSET_PLUS;
+      ++string;
+    }
+  else if (*string == '-')
+    {
+      line_offset.sign = LINE_OFFSET_MINUS;
+      ++string;
+    }
+
+  /* Right now, we only allow base 10 for offsets.  */
+  line_offset.offset = atoi (string);
+  return line_offset;
+}
+
+/* Parse the basic_spec in PARSER's input.  */
+
+static void
+linespec_parse_basic (linespec_parser *parser)
+{
+  char *name;
+  linespec_token token;
+  VEC (symbolp) *symbols, *labels;
+  VEC (bound_minimal_symbol_d) *minimal_symbols;
+  struct cleanup *cleanup;
+
+  /* Get the next token.  */
+  token = linespec_lexer_lex_one (parser);
+
+  /* If it is EOI or KEYWORD, issue an error.  */
+  if (token.type == LSTOKEN_KEYWORD || token.type == LSTOKEN_EOI)
+    unexpected_linespec_error (parser);
+  /* If it is a LSTOKEN_NUMBER, we have an offset.  */
+  else if (token.type == LSTOKEN_NUMBER)
+    {
+      /* Record the line offset and get the next token.  */
+      name = copy_token_string (token);
+      cleanup = make_cleanup (xfree, name);
+      PARSER_EXPLICIT (parser)->line_offset = linespec_parse_line_offset (name);
+      do_cleanups (cleanup);
+
+      /* Get the next token.  */
+      token = linespec_lexer_consume_token (parser);
+
+      /* If the next token is a comma, stop parsing and return.  */
+      if (token.type == LSTOKEN_COMMA)
+       return;
+
+      /* If the next token is anything but EOI or KEYWORD, issue
+        an error.  */
+      if (token.type != LSTOKEN_KEYWORD && token.type != LSTOKEN_EOI)
+       unexpected_linespec_error (parser);
+    }
+
+  if (token.type == LSTOKEN_KEYWORD || token.type == LSTOKEN_EOI)
+    return;
+
+  /* Next token must be LSTOKEN_STRING.  */
+  if (token.type != LSTOKEN_STRING)
+    unexpected_linespec_error (parser);
+
+  /* The current token will contain the name of a function, method,
+     or label.  */
+  name  = copy_token_string (token);
+  cleanup = make_cleanup (xfree, name);
+
+  /* Try looking it up as a function/method.  */
+  find_linespec_symbols (PARSER_STATE (parser),
+                        PARSER_RESULT (parser)->file_symtabs, name,
+                        &symbols, &minimal_symbols);
+
+  if (symbols != NULL || minimal_symbols != NULL)
+    {
+      PARSER_RESULT (parser)->function_symbols = symbols;
+      PARSER_RESULT (parser)->minimal_symbols = minimal_symbols;
+      PARSER_EXPLICIT (parser)->function_name = name;
+      symbols = NULL;
+      discard_cleanups (cleanup);
+    }
+  else
+    {
+      /* NAME was not a function or a method.  So it must be a label
+        name or user specified variable like "break foo.c:$zippo".  */
+      labels = find_label_symbols (PARSER_STATE (parser), NULL,
+                                  &symbols, name);
+      if (labels != NULL)
+       {
+         PARSER_RESULT (parser)->labels.label_symbols = labels;
+         PARSER_RESULT (parser)->labels.function_symbols = symbols;
+         PARSER_EXPLICIT (parser)->label_name = name;
+         symbols = NULL;
+         discard_cleanups (cleanup);
+       }
+      else if (token.type == LSTOKEN_STRING
+              && *LS_TOKEN_STOKEN (token).ptr == '$')
+       {
+         /* User specified a convenience variable or history value.  */
+         PARSER_EXPLICIT (parser)->line_offset
+           = linespec_parse_variable (PARSER_STATE (parser), name);
+
+         if (PARSER_EXPLICIT (parser)->line_offset.sign == LINE_OFFSET_UNKNOWN)
+           {
+             /* The user-specified variable was not valid.  Do not
+                throw an error here.  parse_linespec will do it for us.  */
+             PARSER_EXPLICIT (parser)->function_name = name;
+             discard_cleanups (cleanup);
+             return;
+           }
+
+         /* The convenience variable/history value parsed correctly.
+            NAME is no longer needed.  */
+         do_cleanups (cleanup);
+       }
+      else
+       {
+         /* The name is also not a label.  Abort parsing.  Do not throw
+            an error here.  parse_linespec will do it for us.  */
+
+         /* Save a copy of the name we were trying to lookup.  */
+         PARSER_EXPLICIT (parser)->function_name = name;
+         discard_cleanups (cleanup);
+         return;
+       }
+    }
+
+  /* Get the next token.  */
+  token = linespec_lexer_consume_token (parser);
+
+  if (token.type == LSTOKEN_COLON)
+    {
+      /* User specified a label or a lineno.  */
+      token = linespec_lexer_consume_token (parser);
+
+      if (token.type == LSTOKEN_NUMBER)
+       {
+         /* User specified an offset.  Record the line offset and
+            get the next token.  */
+         name = copy_token_string (token);
+         cleanup = make_cleanup (xfree, name);
+         PARSER_EXPLICIT (parser)->line_offset
+           = linespec_parse_line_offset (name);
+         do_cleanups (cleanup);
+
+         /* Ge the next token.  */
+         token = linespec_lexer_consume_token (parser);
+       }
+      else if (token.type == LSTOKEN_STRING)
+       {
+         /* Grab a copy of the label's name and look it up.  */
+         name = copy_token_string (token);
+         cleanup = make_cleanup (xfree, name);
+         labels = find_label_symbols (PARSER_STATE (parser),
+                                      PARSER_RESULT (parser)->function_symbols,
+                                      &symbols, name);
+
+         if (labels != NULL)
+           {
+             PARSER_RESULT (parser)->labels.label_symbols = labels;
+             PARSER_RESULT (parser)->labels.function_symbols = symbols;
+             PARSER_EXPLICIT (parser)->label_name = name;
+             symbols = NULL;
+             discard_cleanups (cleanup);
+           }
+         else
+           {
+             /* We don't know what it was, but it isn't a label.  */
+             undefined_label_error (PARSER_EXPLICIT (parser)->function_name,
+                                    name);
+           }
+
+         /* Check for a line offset.  */
+         token = linespec_lexer_consume_token (parser);
+         if (token.type == LSTOKEN_COLON)
+           {
+             /* Get the next token.  */
+             token = linespec_lexer_consume_token (parser);
+
+             /* It must be a line offset.  */
+             if (token.type != LSTOKEN_NUMBER)
+               unexpected_linespec_error (parser);
+
+             /* Record the lione offset and get the next token.  */
+             name = copy_token_string (token);
+             cleanup = make_cleanup (xfree, name);
+
+             PARSER_EXPLICIT (parser)->line_offset
+               = linespec_parse_line_offset (name);
+             do_cleanups (cleanup);
+
+             /* Get the next token.  */
+             token = linespec_lexer_consume_token (parser);
+           }
+       }
+      else
+       {
+         /* Trailing ':' in the input. Issue an error.  */
+         unexpected_linespec_error (parser);
+       }
+    }
+}
+
+/* Canonicalize the linespec contained in LS.  The result is saved into
+   STATE->canonical.  This function handles both linespec and explicit
+   locations.  */
+
+static void
+canonicalize_linespec (struct linespec_state *state, const linespec_p ls)
+{
+  struct event_location *canon;
+  struct explicit_location *explicit;
+
+  /* If canonicalization was not requested, no need to do anything.  */
+  if (!state->canonical)
+    return;
+
+  /* Save everything as an explicit location.  */
+  canon = state->canonical->location = new_explicit_location (&ls->explicit);
+  explicit = get_explicit_location (canon);
+
+  if (explicit->label_name != NULL)
+    {
+      state->canonical->special_display = 1;
+
+      if (explicit->function_name == NULL)
+       {
+         struct symbol *s;
+
+         /* No function was specified, so add the symbol name.  */
+         gdb_assert (ls->labels.function_symbols != NULL
+                     && (VEC_length (symbolp, ls->labels.function_symbols)
+                         == 1));
+         s = VEC_index (symbolp, ls->labels.function_symbols, 0);
+         explicit->function_name = xstrdup (SYMBOL_NATURAL_NAME (s));
+       }
+    }
+
+  /* If this location originally came from a linespec, save a string
+     representation of it for display and saving to file.  */
+  if (state->is_linespec)
+    {
+      char *linespec = explicit_location_to_linespec (explicit);
+
+      set_event_location_string (canon, linespec);
+      xfree (linespec);
+    }
+}
+
+/* Given a line offset in LS, construct the relevant SALs.  */
+
+static struct symtabs_and_lines
+create_sals_line_offset (struct linespec_state *self,
+                        linespec_p ls)
+{
+  struct symtabs_and_lines values;
+  struct symtab_and_line val;
+  int use_default = 0;
+
+  init_sal (&val);
+  values.sals = NULL;
+  values.nelts = 0;
+
+  /* This is where we need to make sure we have good defaults.
+     We must guarantee that this section of code is never executed
+     when we are called with just a function name, since
+     set_default_source_symtab_and_line uses
+     select_source_symtab that calls us with such an argument.  */
+
+  if (VEC_length (symtab_ptr, ls->file_symtabs) == 1
+      && VEC_index (symtab_ptr, ls->file_symtabs, 0) == NULL)
+    {
+      const char *fullname;
+
+      set_current_program_space (self->program_space);
+
+      /* Make sure we have at least a default source line.  */
+      set_default_source_symtab_and_line ();
+      initialize_defaults (&self->default_symtab, &self->default_line);
+      fullname = symtab_to_fullname (self->default_symtab);
+      VEC_pop (symtab_ptr, ls->file_symtabs);
+      VEC_free (symtab_ptr, ls->file_symtabs);
+      ls->file_symtabs = collect_symtabs_from_filename (fullname);
+      use_default = 1;
+    }
+
+  val.line = ls->explicit.line_offset.offset;
+  switch (ls->explicit.line_offset.sign)
+    {
+    case LINE_OFFSET_PLUS:
+      if (ls->explicit.line_offset.offset == 0)
+       val.line = 5;
+      if (use_default)
+       val.line = self->default_line + val.line;
+      break;
+
+    case LINE_OFFSET_MINUS:
+      if (ls->explicit.line_offset.offset == 0)
+       val.line = 15;
+      if (use_default)
+       val.line = self->default_line - val.line;
+      else
+       val.line = -val.line;
+      break;
+
+    case LINE_OFFSET_NONE:
+      break;                   /* No need to adjust val.line.  */
+    }
+
+  if (self->list_mode)
+    decode_digits_list_mode (self, ls, &values, val);
+  else
+    {
+      struct linetable_entry *best_entry = NULL;
+      int *filter;
+      const struct block **blocks;
+      struct cleanup *cleanup;
+      struct symtabs_and_lines intermediate_results;
+      int i, j;
+
+      intermediate_results.sals = NULL;
+      intermediate_results.nelts = 0;
+
+      decode_digits_ordinary (self, ls, val.line, &intermediate_results,
+                             &best_entry);
+      if (intermediate_results.nelts == 0 && best_entry != NULL)
+       decode_digits_ordinary (self, ls, best_entry->line,
+                               &intermediate_results, &best_entry);
+
+      cleanup = make_cleanup (xfree, intermediate_results.sals);
+
+      /* For optimized code, the compiler can scatter one source line
+        across disjoint ranges of PC values, even when no duplicate
+        functions or inline functions are involved.  For example,
+        'for (;;)' inside a non-template, non-inline, and non-ctor-or-dtor
+        function can result in two PC ranges.  In this case, we don't
+        want to set a breakpoint on the first PC of each range.  To filter
+        such cases, we use containing blocks -- for each PC found
+        above, we see if there are other PCs that are in the same
+        block.  If yes, the other PCs are filtered out.  */
+
+      filter = XNEWVEC (int, intermediate_results.nelts);
+      make_cleanup (xfree, filter);
+      blocks = XNEWVEC (const struct block *, intermediate_results.nelts);
+      make_cleanup (xfree, blocks);
+
+      for (i = 0; i < intermediate_results.nelts; ++i)
+       {
+         set_current_program_space (intermediate_results.sals[i].pspace);
+
+         filter[i] = 1;
+         blocks[i] = block_for_pc_sect (intermediate_results.sals[i].pc,
+                                        intermediate_results.sals[i].section);
+       }
+
+      for (i = 0; i < intermediate_results.nelts; ++i)
+       {
+         if (blocks[i] != NULL)
+           for (j = i + 1; j < intermediate_results.nelts; ++j)
+             {
+               if (blocks[j] == blocks[i])
+                 {
+                   filter[j] = 0;
+                   break;
+                 }
+             }
+       }
+
+      for (i = 0; i < intermediate_results.nelts; ++i)
+       if (filter[i])
+         {
+           struct symbol *sym = (blocks[i]
+                                 ? block_containing_function (blocks[i])
+                                 : NULL);
+
+           if (self->funfirstline)
+             skip_prologue_sal (&intermediate_results.sals[i]);
+           /* Make sure the line matches the request, not what was
+              found.  */
+           intermediate_results.sals[i].line = val.line;
+           add_sal_to_sals (self, &values, &intermediate_results.sals[i],
+                            sym ? SYMBOL_NATURAL_NAME (sym) : NULL, 0);
+         }
+
+      do_cleanups (cleanup);
+    }
+
+  if (values.nelts == 0)
+    {
+      if (ls->explicit.source_filename)
+       throw_error (NOT_FOUND_ERROR, _("No line %d in file \"%s\"."),
+                    val.line, ls->explicit.source_filename);
+      else
+       throw_error (NOT_FOUND_ERROR, _("No line %d in the current file."),
+                    val.line);
+    }
+
+  return values;
+}
+
+/* Convert the given ADDRESS into SaLs.  */
+
+static struct symtabs_and_lines
+convert_address_location_to_sals (struct linespec_state *self,
+                                 CORE_ADDR address)
+{
+  struct symtab_and_line sal;
+  struct symtabs_and_lines sals = {NULL, 0};
+
+  sal = find_pc_line (address, 0);
+  sal.pc = address;
+  sal.section = find_pc_overlay (address);
+  sal.explicit_pc = 1;
+  add_sal_to_sals (self, &sals, &sal, core_addr_to_string (address), 1);
+
+  return sals;
+}
+
+/* Create and return SALs from the linespec LS.  */
+
+static struct symtabs_and_lines
+convert_linespec_to_sals (struct linespec_state *state, linespec_p ls)
+{
+  struct symtabs_and_lines sals = {NULL, 0};
+
+  if (ls->labels.label_symbols != NULL)
+    {
+      /* We have just a bunch of functions/methods or labels.  */
+      int i;
+      struct symtab_and_line sal;
+      struct symbol *sym;
+
+      for (i = 0; VEC_iterate (symbolp, ls->labels.label_symbols, i, sym); ++i)
+       {
+         struct program_space *pspace = SYMTAB_PSPACE (symbol_symtab (sym));
+
+         if (symbol_to_sal (&sal, state->funfirstline, sym)
+             && maybe_add_address (state->addr_set, pspace, sal.pc))
+           add_sal_to_sals (state, &sals, &sal,
+                            SYMBOL_NATURAL_NAME (sym), 0);
+       }
+    }
+  else if (ls->function_symbols != NULL || ls->minimal_symbols != NULL)
+    {
+      /* We have just a bunch of functions and/or methods.  */
+      int i;
+      struct symtab_and_line sal;
+      struct symbol *sym;
+      bound_minimal_symbol_d *elem;
+      struct program_space *pspace;
+
+      if (ls->function_symbols != NULL)
+       {
+         /* Sort symbols so that symbols with the same program space are next
+            to each other.  */
+         qsort (VEC_address (symbolp, ls->function_symbols),
+                VEC_length (symbolp, ls->function_symbols),
+                sizeof (symbolp), compare_symbols);
+
+         for (i = 0; VEC_iterate (symbolp, ls->function_symbols, i, sym); ++i)
+           {
+             pspace = SYMTAB_PSPACE (symbol_symtab (sym));
+             set_current_program_space (pspace);
+             if (symbol_to_sal (&sal, state->funfirstline, sym)
+                 && maybe_add_address (state->addr_set, pspace, sal.pc))
+               add_sal_to_sals (state, &sals, &sal,
+                                SYMBOL_NATURAL_NAME (sym), 0);
+           }
+       }
+
+      if (ls->minimal_symbols != NULL)
+       {
+         /* Sort minimal symbols by program space, too.  */
+         qsort (VEC_address (bound_minimal_symbol_d, ls->minimal_symbols),
+                VEC_length (bound_minimal_symbol_d, ls->minimal_symbols),
+                sizeof (bound_minimal_symbol_d), compare_msymbols);
+
+         for (i = 0;
+              VEC_iterate (bound_minimal_symbol_d, ls->minimal_symbols,
+                           i, elem);
+              ++i)
+           {
+             pspace = elem->objfile->pspace;
+             set_current_program_space (pspace);
+             minsym_found (state, elem->objfile, elem->minsym, &sals);
+           }
+       }
+    }
+  else if (ls->explicit.line_offset.sign != LINE_OFFSET_UNKNOWN)
+    {
+      /* Only an offset was specified.  */
+       sals = create_sals_line_offset (state, ls);
+
+       /* Make sure we have a filename for canonicalization.  */
+       if (ls->explicit.source_filename == NULL)
+         {
+           const char *fullname = symtab_to_fullname (state->default_symtab);
+
+           /* It may be more appropriate to keep DEFAULT_SYMTAB in its symtab
+              form so that displaying SOURCE_FILENAME can follow the current
+              FILENAME_DISPLAY_STRING setting.  But as it is used only rarely
+              it has been kept for code simplicity only in absolute form.  */
+           ls->explicit.source_filename = xstrdup (fullname);
+         }
+    }
+  else
+    {
+      /* We haven't found any results...  */
+      return sals;
+    }
+
+  canonicalize_linespec (state, ls);
+
+  if (sals.nelts > 0 && state->canonical != NULL)
+    state->canonical->pre_expanded = 1;
+
+  return sals;
+}
+
+/* Convert the explicit location EXPLICIT into SaLs.  */
+
+static struct symtabs_and_lines
+convert_explicit_location_to_sals (struct linespec_state *self,
+                                  linespec_p result,
+                                  const struct explicit_location *explicit)
+{
+  VEC (symbolp) *symbols, *labels;
+  VEC (bound_minimal_symbol_d) *minimal_symbols;
+
+  if (explicit->source_filename != NULL)
+    {
+      TRY
+       {
+         result->file_symtabs
+           = symtabs_from_filename (explicit->source_filename);
+       }
+      CATCH (except, RETURN_MASK_ERROR)
+       {
+         source_file_not_found_error (explicit->source_filename);
+       }
+      END_CATCH
+      result->explicit.source_filename = xstrdup (explicit->source_filename);
+    }
+  else
+    {
+      /* A NULL entry means to use the default symtab.  */
+      VEC_safe_push (symtab_ptr, result->file_symtabs, NULL);
+    }
+
+  if (explicit->function_name != NULL)
+    {
+      find_linespec_symbols (self, result->file_symtabs,
+                            explicit->function_name, &symbols,
+                            &minimal_symbols);
+
+      if (symbols == NULL && minimal_symbols == NULL)
+       symbol_not_found_error (explicit->function_name,
+                               result->explicit.source_filename);
+
+      result->explicit.function_name = xstrdup (explicit->function_name);
+      result->function_symbols = symbols;
+      result->minimal_symbols = minimal_symbols;
+    }
+
+  if (explicit->label_name != NULL)
+    {
+      symbols = NULL;
+      labels = find_label_symbols (self, result->function_symbols,
+                                  &symbols, explicit->label_name);
+
+      if (labels == NULL)
+       undefined_label_error (result->explicit.function_name,
+                              explicit->label_name);
+
+      result->explicit.label_name = xstrdup (explicit->label_name);
+      result->labels.label_symbols = labels;
+      result->labels.function_symbols = symbols;
+    }
+
+  if (explicit->line_offset.sign != LINE_OFFSET_UNKNOWN)
+    result->explicit.line_offset = explicit->line_offset;
+
+   return convert_linespec_to_sals (self, result);
+}
+
+/* Parse a string that specifies a linespec.
+
+   The basic grammar of linespecs:
+
+   linespec -> var_spec | basic_spec
+   var_spec -> '$' (STRING | NUMBER)
+
+   basic_spec -> file_offset_spec | function_spec | label_spec
+   file_offset_spec -> opt_file_spec offset_spec
+   function_spec -> opt_file_spec function_name_spec opt_label_spec
+   label_spec -> label_name_spec
+
+   opt_file_spec -> "" | file_name_spec ':'
+   opt_label_spec -> "" | ':' label_name_spec
+
+   file_name_spec -> STRING
+   function_name_spec -> STRING
+   label_name_spec -> STRING
+   function_name_spec -> STRING
+   offset_spec -> NUMBER
+               -> '+' NUMBER
+              -> '-' NUMBER
+
+   This may all be followed by several keywords such as "if EXPR",
+   which we ignore.
+
+   A comma will terminate parsing.
+
+   The function may be an undebuggable function found in minimal symbol table.
+
+   If the argument FUNFIRSTLINE is nonzero, we want the first line
+   of real code inside a function when a function is specified, and it is
+   not OK to specify a variable or type to get its line number.
+
+   DEFAULT_SYMTAB specifies the file to use if none is specified.
+   It defaults to current_source_symtab.
+   DEFAULT_LINE specifies the line number to use for relative
+   line numbers (that start with signs).  Defaults to current_source_line.
+   If CANONICAL is non-NULL, store an array of strings containing the canonical
+   line specs there if necessary.  Currently overloaded member functions and
+   line numbers or static functions without a filename yield a canonical
+   line spec.  The array and the line spec strings are allocated on the heap,
+   it is the callers responsibility to free them.
+
+   Note that it is possible to return zero for the symtab
+   if no file is validly specified.  Callers must check that.
+   Also, the line number returned may be invalid.  */
+
+/* Parse the linespec in ARG.  */
+
+static struct symtabs_and_lines
+parse_linespec (linespec_parser *parser, const char *arg)
+{
+  linespec_token token;
+  struct symtabs_and_lines values;
+  struct gdb_exception file_exception = exception_none;
+  struct cleanup *cleanup;
+
+  /* A special case to start.  It has become quite popular for
+     IDEs to work around bugs in the previous parser by quoting
+     the entire linespec, so we attempt to deal with this nicely.  */
+  parser->is_quote_enclosed = 0;
+  if (!is_ada_operator (arg)
+      && strchr (linespec_quote_characters, *arg) != NULL)
+    {
+      const char *end;
+
+      end = skip_quote_char (arg + 1, *arg);
+      if (end != NULL && is_closing_quote_enclosed (end))
+       {
+         /* Here's the special case.  Skip ARG past the initial
+            quote.  */
+         ++arg;
+         parser->is_quote_enclosed = 1;
+       }
+    }
+
+  parser->lexer.saved_arg = arg;
+  parser->lexer.stream = arg;
+
+  /* Initialize the default symtab and line offset.  */
+  initialize_defaults (&PARSER_STATE (parser)->default_symtab,
+                      &PARSER_STATE (parser)->default_line);
+
+  /* Objective-C shortcut.  */
+  values = decode_objc (PARSER_STATE (parser), PARSER_RESULT (parser), arg);
+  if (values.sals != NULL)
+    return values;
+
+  /* Start parsing.  */
+
+  /* Get the first token.  */
+  token = linespec_lexer_lex_one (parser);
+
+  /* It must be either LSTOKEN_STRING or LSTOKEN_NUMBER.  */
+  if (token.type == LSTOKEN_STRING && *LS_TOKEN_STOKEN (token).ptr == '$')
+    {
+      char *var;
+
+      /* A NULL entry means to use GLOBAL_DEFAULT_SYMTAB.  */
+      VEC_safe_push (symtab_ptr, PARSER_RESULT (parser)->file_symtabs, NULL);
+
+      /* User specified a convenience variable or history value.  */
+      var = copy_token_string (token);
+      cleanup = make_cleanup (xfree, var);
+      PARSER_EXPLICIT (parser)->line_offset
+       = linespec_parse_variable (PARSER_STATE (parser), var);
+      do_cleanups (cleanup);
+
+      /* If a line_offset wasn't found (VAR is the name of a user
+        variable/function), then skip to normal symbol processing.  */
+      if (PARSER_EXPLICIT (parser)->line_offset.sign != LINE_OFFSET_UNKNOWN)
+       {
+         /* Consume this token.  */
+         linespec_lexer_consume_token (parser);
+
+         goto convert_to_sals;
+       }
+    }
+  else if (token.type != LSTOKEN_STRING && token.type != LSTOKEN_NUMBER)
+    unexpected_linespec_error (parser);
+
+  /* Shortcut: If the next token is not LSTOKEN_COLON, we know that
+     this token cannot represent a filename.  */
+  token = linespec_lexer_peek_token (parser);
+
+  if (token.type == LSTOKEN_COLON)
+    {
+      char *user_filename;
+
+      /* Get the current token again and extract the filename.  */
+      token = linespec_lexer_lex_one (parser);
+      user_filename = copy_token_string (token);
+
+      /* Check if the input is a filename.  */
+      TRY
+       {
+         PARSER_RESULT (parser)->file_symtabs
+           = symtabs_from_filename (user_filename);
+       }
+      CATCH (ex, RETURN_MASK_ERROR)
+       {
+         file_exception = ex;
+       }
+      END_CATCH
+
+      if (file_exception.reason >= 0)
+       {
+         /* Symtabs were found for the file.  Record the filename.  */
+         PARSER_EXPLICIT (parser)->source_filename = user_filename;
+
+         /* Get the next token.  */
+         token = linespec_lexer_consume_token (parser);
+
+         /* This is LSTOKEN_COLON; consume it.  */
+         linespec_lexer_consume_token (parser);
+       }
+      else
+       {
+         /* No symtabs found -- discard user_filename.  */
+         xfree (user_filename);
+
+         /* A NULL entry means to use GLOBAL_DEFAULT_SYMTAB.  */
+         VEC_safe_push (symtab_ptr, PARSER_RESULT (parser)->file_symtabs, NULL);
+       }
+    }
+  /* If the next token is not EOI, KEYWORD, or COMMA, issue an error.  */
+  else if (token.type != LSTOKEN_EOI && token.type != LSTOKEN_KEYWORD
+          && token.type != LSTOKEN_COMMA)
+    {
+      /* TOKEN is the _next_ token, not the one currently in the parser.
+        Consuming the token will give the correct error message.  */
+      linespec_lexer_consume_token (parser);
+      unexpected_linespec_error (parser);
+    }
+  else
+    {
+      /* A NULL entry means to use GLOBAL_DEFAULT_SYMTAB.  */
+      VEC_safe_push (symtab_ptr, PARSER_RESULT (parser)->file_symtabs, NULL);
+    }
+
+  /* Parse the rest of the linespec.  */
+  linespec_parse_basic (parser);
+
+  if (PARSER_RESULT (parser)->function_symbols == NULL
+      && PARSER_RESULT (parser)->labels.label_symbols == NULL
+      && PARSER_EXPLICIT (parser)->line_offset.sign == LINE_OFFSET_UNKNOWN
+      && PARSER_RESULT (parser)->minimal_symbols == NULL)
+    {
+      /* The linespec didn't parse.  Re-throw the file exception if
+        there was one.  */
+      if (file_exception.reason < 0)
+       throw_exception (file_exception);
+
+      /* Otherwise, the symbol is not found.  */
+      symbol_not_found_error (PARSER_EXPLICIT (parser)->function_name,
+                             PARSER_EXPLICIT (parser)->source_filename);
+    }
+
+ convert_to_sals:
+
+  /* Get the last token and record how much of the input was parsed,
+     if necessary.  */
+  token = linespec_lexer_lex_one (parser);
+  if (token.type != LSTOKEN_EOI && token.type != LSTOKEN_KEYWORD)
+    PARSER_STREAM (parser) = LS_TOKEN_STOKEN (token).ptr;
+
+  /* Convert the data in PARSER_RESULT to SALs.  */
+  values = convert_linespec_to_sals (PARSER_STATE (parser),
+                                    PARSER_RESULT (parser));
+
+  return values;
+}
+
+
+/* A constructor for linespec_state.  */
+
+static void
+linespec_state_constructor (struct linespec_state *self,
+                           int flags, const struct language_defn *language,
+                           struct symtab *default_symtab,
+                           int default_line,
+                           struct linespec_result *canonical)
+{
+  memset (self, 0, sizeof (*self));
+  self->language = language;
+  self->funfirstline = (flags & DECODE_LINE_FUNFIRSTLINE) ? 1 : 0;
+  self->list_mode = (flags & DECODE_LINE_LIST_MODE) ? 1 : 0;
+  self->default_symtab = default_symtab;
+  self->default_line = default_line;
+  self->canonical = canonical;
+  self->program_space = current_program_space;
+  self->addr_set = htab_create_alloc (10, hash_address_entry, eq_address_entry,
+                                     xfree, xcalloc, xfree);
+  self->is_linespec = 0;
+}
+
+/* Initialize a new linespec parser.  */
+
+static void
+linespec_parser_new (linespec_parser *parser,
+                    int flags, const struct language_defn *language,
+                    struct symtab *default_symtab,
+                    int default_line,
+                    struct linespec_result *canonical)
+{
+  memset (parser, 0, sizeof (linespec_parser));
+  parser->lexer.current.type = LSTOKEN_CONSUMED;
+  memset (PARSER_RESULT (parser), 0, sizeof (struct linespec));
+  PARSER_EXPLICIT (parser)->line_offset.sign = LINE_OFFSET_UNKNOWN;
+  linespec_state_constructor (PARSER_STATE (parser), flags, language,
+                             default_symtab, default_line, canonical);
+}
+
+/* A destructor for linespec_state.  */
+
+static void
+linespec_state_destructor (struct linespec_state *self)
+{
+  htab_delete (self->addr_set);
+}
+
+/* Delete a linespec parser.  */
+
+static void
+linespec_parser_delete (void *arg)
+{
+  linespec_parser *parser = (linespec_parser *) arg;
+
+  xfree (PARSER_EXPLICIT (parser)->source_filename);
+  xfree (PARSER_EXPLICIT (parser)->label_name);
+  xfree (PARSER_EXPLICIT (parser)->function_name);
+
+  if (PARSER_RESULT (parser)->file_symtabs != NULL)
+    VEC_free (symtab_ptr, PARSER_RESULT (parser)->file_symtabs);
+
+  if (PARSER_RESULT (parser)->function_symbols != NULL)
+    VEC_free (symbolp, PARSER_RESULT (parser)->function_symbols);
+
+  if (PARSER_RESULT (parser)->minimal_symbols != NULL)
+    VEC_free (bound_minimal_symbol_d, PARSER_RESULT (parser)->minimal_symbols);
+
+  if (PARSER_RESULT (parser)->labels.label_symbols != NULL)
+    VEC_free (symbolp, PARSER_RESULT (parser)->labels.label_symbols);
+
+  if (PARSER_RESULT (parser)->labels.function_symbols != NULL)
+    VEC_free (symbolp, PARSER_RESULT (parser)->labels.function_symbols);
+
+  linespec_state_destructor (PARSER_STATE (parser));
+}
+
+/* See description in linespec.h.  */
+
+void
+linespec_lex_to_end (char **stringp)
+{
+  linespec_parser parser;
+  struct cleanup *cleanup;
+  linespec_token token;
+  const char *orig;
+
+  if (stringp == NULL || *stringp == NULL)
+    return;
+
+  linespec_parser_new (&parser, 0, current_language, NULL, 0, NULL);
+  cleanup = make_cleanup (linespec_parser_delete, &parser);
+  parser.lexer.saved_arg = *stringp;
+  PARSER_STREAM (&parser) = orig = *stringp;
+
+  do
+    {
+      /* Stop before any comma tokens;  we need it to keep it
+        as the next token in the string.  */
+      token = linespec_lexer_peek_token (&parser);
+      if (token.type == LSTOKEN_COMMA)
+       break;
+      token = linespec_lexer_consume_token (&parser);
+    }
+  while (token.type != LSTOKEN_EOI && token.type != LSTOKEN_KEYWORD);
+
+  *stringp += PARSER_STREAM (&parser) - orig;
+  do_cleanups (cleanup);
+}
+
+/* A helper function for decode_line_full and decode_line_1 to
+   turn LOCATION into symtabs_and_lines.  */
+
+static struct symtabs_and_lines
+event_location_to_sals (linespec_parser *parser,
+                       const struct event_location *location)
+{
+  struct symtabs_and_lines result = {NULL, 0};
+
+  switch (event_location_type (location))
+    {
+    case LINESPEC_LOCATION:
+      {
+       PARSER_STATE (parser)->is_linespec = 1;
+       TRY
+         {
+           result = parse_linespec (parser, get_linespec_location (location));
+         }
+       CATCH (except, RETURN_MASK_ERROR)
+         {
+           throw_exception (except);
+         }
+       END_CATCH
+      }
+      break;
 
-  /* Check if this is an Objective-C method (anything that starts with
-     a '+' or '-' and a '[').  */
-  if (is_objc_method_format (p))
-    is_objc_method = 1;
+    case ADDRESS_LOCATION:
+      result
+       = convert_address_location_to_sals (PARSER_STATE (parser),
+                                           get_address_location (location));
+      break;
 
-  /* Check if the symbol could be an Objective-C selector.  */
+    case EXPLICIT_LOCATION:
+      {
+       const struct explicit_location *explicit;
 
-  {
-    struct symtabs_and_lines values;
+       explicit = get_explicit_location_const (location);
+       result = convert_explicit_location_to_sals (PARSER_STATE (parser),
+                                                   PARSER_RESULT (parser),
+                                                   explicit);
+      }
+      break;
 
-    values = decode_objc (argptr, funfirstline, NULL,
-                         canonical, saved_arg);
-    if (values.sals != NULL)
-      return values;
-  }
+    case PROBE_LOCATION:
+      /* Probes are handled by their own decoders.  */
+      gdb_assert_not_reached ("attempt to decode probe location");
+      break;
 
-  /* Does it look like there actually were two parts?  */
+    default:
+      gdb_assert_not_reached ("unhandled event location type");
+    }
 
-  if (p[0] == ':' || p[0] == '.')
-    {
-      /* Is it a C++ or Java compound data structure?
-        The check on p[1] == ':' is capturing the case of "::",
-        since p[0]==':' was checked above.
-        Note that the call to decode_compound does everything
-        for us, including the lookup on the symbol table, so we
-        can return now.  */
-       
-      if (p[0] == '.' || p[1] == ':')
-       {
-         struct symtabs_and_lines values;
-
-         if (is_quote_enclosed)
-           ++saved_arg;
-         values = decode_compound (argptr, funfirstline, canonical,
-                                   saved_arg, p);
-         if (is_quoted && **argptr == '\'')
-           *argptr = *argptr + 1;
-         return values;
-       }
+  return result;
+}
 
-      /* No, the first part is a filename; set file_symtab to be that file's
-        symtab.  Also, move argptr past the filename.  */
+/* See linespec.h.  */
 
-      TRY_CATCH (file_exception, RETURN_MASK_ERROR)
-       {
-         file_symtab = symtab_from_filename (argptr, p, is_quote_enclosed);
-       }
-      /* If that failed, maybe we have `function:label'.  */
-      if (file_exception.reason < 0)
+void
+decode_line_full (const struct event_location *location, int flags,
+                 struct symtab *default_symtab,
+                 int default_line, struct linespec_result *canonical,
+                 const char *select_mode,
+                 const char *filter)
+{
+  struct symtabs_and_lines result;
+  struct cleanup *cleanups;
+  VEC (const_char_ptr) *filters = NULL;
+  linespec_parser parser;
+  struct linespec_state *state;
+
+  gdb_assert (canonical != NULL);
+  /* The filter only makes sense for 'all'.  */
+  gdb_assert (filter == NULL || select_mode == multiple_symbols_all);
+  gdb_assert (select_mode == NULL
+             || select_mode == multiple_symbols_all
+             || select_mode == multiple_symbols_ask
+             || select_mode == multiple_symbols_cancel);
+  gdb_assert ((flags & DECODE_LINE_LIST_MODE) == 0);
+
+  linespec_parser_new (&parser, flags, current_language, default_symtab,
+                      default_line, canonical);
+  cleanups = make_cleanup (linespec_parser_delete, &parser);
+  save_current_program_space ();
+
+  result = event_location_to_sals (&parser, location);
+  state = PARSER_STATE (&parser);
+
+  gdb_assert (result.nelts == 1 || canonical->pre_expanded);
+  canonical->pre_expanded = 1;
+
+  /* Arrange for allocated canonical names to be freed.  */
+  if (result.nelts > 0)
+    {
+      int i;
+
+      make_cleanup (xfree, state->canonical_names);
+      for (i = 0; i < result.nelts; ++i)
        {
-         function_symbol = find_function_symbol (argptr, p, is_quote_enclosed);
-         /* If we did not find a function, re-throw the original
-            exception.  */
-         if (!function_symbol)
-           throw_exception (file_exception);
+         gdb_assert (state->canonical_names[i].suffix != NULL);
+         make_cleanup (xfree, state->canonical_names[i].suffix);
        }
+    }
+
+  if (select_mode == NULL)
+    {
+      if (ui_out_is_mi_like_p (interp_ui_out (top_level_interpreter ())))
+       select_mode = multiple_symbols_all;
+      else
+       select_mode = multiple_symbols_select_mode ();
+    }
 
-      /* Check for single quotes on the non-filename part.  */
-      if (!is_quoted)
+  if (select_mode == multiple_symbols_all)
+    {
+      if (filter != NULL)
        {
-         is_quoted = (**argptr
-                      && strchr (get_gdb_completer_quote_characters (),
-                                 **argptr) != NULL);
-         if (is_quoted)
-           end_quote = skip_quoted (*argptr);
+         make_cleanup (VEC_cleanup (const_char_ptr), &filters);
+         VEC_safe_push (const_char_ptr, filters, filter);
+         filter_results (state, &result, filters);
        }
+      else
+       convert_results_to_lsals (state, &result);
     }
+  else
+    decode_line_2 (state, &result, select_mode);
 
-  /* file_symtab is specified file's symtab, or 0 if no file specified.
-     If we are parsing `function:symbol', then FUNCTION_SYMBOL is the
-     function before the `:'.
-     arg no longer contains the file name.  */
+  do_cleanups (cleanups);
+}
 
-  /* If the filename was quoted, we must re-check the quotation.  */
+/* See linespec.h.  */
 
-  if (end_quote == first_half && *end_quote!= '\0')
-    {
-      is_quoted = (**argptr
-                  && strchr (get_gdb_completer_quote_characters (),
-                             **argptr) != NULL);
-      if (is_quoted)
-       end_quote = skip_quoted (*argptr);
-    }
+struct symtabs_and_lines
+decode_line_1 (const struct event_location *location, int flags,
+              struct symtab *default_symtab,
+              int default_line)
+{
+  struct symtabs_and_lines result;
+  linespec_parser parser;
+  struct cleanup *cleanups;
 
-  /* Check whether arg is all digits (and sign).  */
+  linespec_parser_new (&parser, flags, current_language, default_symtab,
+                      default_line, NULL);
+  cleanups = make_cleanup (linespec_parser_delete, &parser);
+  save_current_program_space ();
 
-  q = *argptr;
-  if (*q == '-' || *q == '+')
-    q++;
-  while (*q >= '0' && *q <= '9')
-    q++;
+  result = event_location_to_sals (&parser, location);
 
-  if (q != *argptr && (*q == 0 || *q == ' ' || *q == '\t' || *q == ',')
-      && function_symbol == NULL)
-    /* We found a token consisting of all digits -- at least one digit.  */
-    return decode_all_digits (argptr, default_symtab, default_line,
-                             canonical, file_symtab, q);
+  do_cleanups (cleanups);
+  return result;
+}
 
-  /* Arg token is not digits => try it as a variable name
-     Find the next token (everything up to end or next whitespace).  */
+/* See linespec.h.  */
 
-  if (**argptr == '$')         /* May be a convenience variable.  */
-    /* One or two $ chars possible.  */
-    p = skip_quoted (*argptr + (((*argptr)[1] == '$') ? 2 : 1));
-  else if (is_quoted)
-    {
-      p = end_quote;
-      if (p[-1] != '\'')
-       error (_("Unmatched single quote."));
-    }
-  else if (is_objc_method)
-    {
-      /* allow word separators in method names for Obj-C.  */
-      p = skip_quoted_chars (*argptr, NULL, "");
-    }
-  else
-    {
-      p = skip_quoted (*argptr);
-    }
+struct symtabs_and_lines
+decode_line_with_current_source (char *string, int flags)
+{
+  struct symtabs_and_lines sals;
+  struct symtab_and_line cursal;
+  struct event_location *location;
+  struct cleanup *cleanup;
 
-  /* Keep any important naming information.  */
-  p = keep_name_info (p);
+  if (string == 0)
+    error (_("Empty line specification."));
 
-  copy = (char *) alloca (p - *argptr + 1);
-  memcpy (copy, *argptr, p - *argptr);
-  copy[p - *argptr] = '\0';
-  if (p != *argptr
-      && copy[0]
-      && copy[0] == copy[p - *argptr - 1]
-      && strchr (get_gdb_completer_quote_characters (), copy[0]) != NULL)
-    {
-      copy[p - *argptr - 1] = '\0';
-      copy++;
-    }
-  else if (is_quoted)
-    copy[p - *argptr - 1] = '\0';
-  while (*p == ' ' || *p == '\t')
-    p++;
-  *argptr = p;
+  /* We use whatever is set as the current source line.  We do not try
+     and get a default source symtab+line or it will recursively call us!  */
+  cursal = get_current_source_symtab_and_line ();
 
-  /* If it starts with $: may be a legitimate variable or routine name
-     (e.g. HP-UX millicode routines such as $$dyncall), or it may
-     be history value, or it may be a convenience variable.  */
+  location = string_to_event_location (&string, current_language);
+  cleanup = make_cleanup_delete_event_location (location);
+  sals = decode_line_1 (location, flags,
+                       cursal.symtab, cursal.line);
 
-  if (*copy == '$' && function_symbol == NULL)
-    return decode_dollar (copy, funfirstline, default_symtab,
-                         canonical, file_symtab);
+  if (*string)
+    error (_("Junk at end of line specification: %s"), string);
 
-  /* Try the token as a label, but only if no file was specified,
-     because we can only really find labels in the current scope.  */
+  do_cleanups (cleanup);
+  return sals;
+}
 
-  if (!file_symtab)
-    {
-      struct symtabs_and_lines label_result;
-      if (decode_label (function_symbol, copy, canonical, &label_result))
-       return label_result;
-    }
+/* See linespec.h.  */
+
+struct symtabs_and_lines
+decode_line_with_last_displayed (char *string, int flags)
+{
+  struct symtabs_and_lines sals;
+  struct event_location *location;
+  struct cleanup *cleanup;
 
-  if (function_symbol)
-    throw_exception (file_exception);
+  if (string == 0)
+    error (_("Empty line specification."));
 
-  /* Look up that token as a variable.
-     If file specified, use that file's per-file block to start with.  */
+  location = string_to_event_location (&string, current_language);
+  cleanup = make_cleanup_delete_event_location (location);
+  if (last_displayed_sal_is_valid ())
+    sals = decode_line_1 (location, flags,
+                         get_last_displayed_symtab (),
+                         get_last_displayed_line ());
+  else
+    sals = decode_line_1 (location, flags, (struct symtab *) NULL, 0);
+
+  if (*string)
+    error (_("Junk at end of line specification: %s"), string);
 
-  return decode_variable (copy, funfirstline, canonical, file_symtab);
+  do_cleanups (cleanup);
+  return sals;
 }
 
 \f
 
-/* Now, more helper functions for decode_line_1.  Some conventions
-   that these functions follow:
-
-   Decode_line_1 typically passes along some of its arguments or local
-   variables to the subfunctions.  It passes the variables by
-   reference if they are modified by the subfunction, and by value
-   otherwise.
-
-   Some of the functions have side effects that don't arise from
-   variables that are passed by reference.  In particular, if a
-   function is passed ARGPTR as an argument, it modifies what ARGPTR
-   points to; typically, it advances *ARGPTR past whatever substring
-   it has just looked at.  (If it doesn't modify *ARGPTR, then the
-   function gets passed *ARGPTR instead, which is then called ARG.)
-   Also, functions that return a struct symtabs_and_lines may modify
-   CANONICAL, as in the description of decode_line_1.
-
-   If a function returns a struct symtabs_and_lines, then that struct
-   will immediately make its way up the call chain to be returned by
-   decode_line_1.  In particular, all of the functions decode_XXX
-   calculate the appropriate struct symtabs_and_lines, under the
-   assumption that their argument is of the form XXX.  */
-
 /* First, some functions to initialize stuff at the beggining of the
    function.  */
 
@@ -1063,1177 +2696,1210 @@ initialize_defaults (struct symtab **default_symtab, int *default_line)
 
 \f
 
-/* Decode arg of the form *PC.  */
+/* Evaluate the expression pointed to by EXP_PTR into a CORE_ADDR,
+   advancing EXP_PTR past any parsed text.  */
+
+CORE_ADDR
+linespec_expression_to_pc (const char **exp_ptr)
+{
+  if (current_program_space->executing_startup)
+    /* The error message doesn't really matter, because this case
+       should only hit during breakpoint reset.  */
+    throw_error (NOT_FOUND_ERROR, _("cannot evaluate expressions while "
+                                   "program space is in startup"));
+
+  (*exp_ptr)++;
+  return value_as_address (parse_to_comma_and_eval (exp_ptr));
+}
+
+\f
+
+/* Here's where we recognise an Objective-C Selector.  An Objective C
+   selector may be implemented by more than one class, therefore it
+   may represent more than one method/function.  This gives us a
+   situation somewhat analogous to C++ overloading.  If there's more
+   than one method that could represent the selector, then use some of
+   the existing C++ code to let the user choose one.  */
 
 static struct symtabs_and_lines
-decode_indirect (char **argptr)
+decode_objc (struct linespec_state *self, linespec_p ls, const char *arg)
 {
+  struct collect_info info;
+  VEC (const_char_ptr) *symbol_names = NULL;
   struct symtabs_and_lines values;
-  CORE_ADDR pc;
-  
-  (*argptr)++;
-  pc = value_as_address (parse_to_comma_and_eval (argptr));
+  const char *new_argptr;
+  struct cleanup *cleanup = make_cleanup (VEC_cleanup (const_char_ptr),
+                                         &symbol_names);
+
+  info.state = self;
+  info.file_symtabs = NULL;
+  VEC_safe_push (symtab_ptr, info.file_symtabs, NULL);
+  make_cleanup (VEC_cleanup (symtab_ptr), &info.file_symtabs);
+  info.result.symbols = NULL;
+  info.result.minimal_symbols = NULL;
+  values.nelts = 0;
+  values.sals = NULL;
+
+  new_argptr = find_imps (arg, &symbol_names);
+  if (VEC_empty (const_char_ptr, symbol_names))
+    {
+      do_cleanups (cleanup);
+      return values;
+    }
 
-  values.sals = (struct symtab_and_line *)
-    xmalloc (sizeof (struct symtab_and_line));
+  add_all_symbol_names_from_pspace (&info, NULL, symbol_names);
 
-  values.nelts = 1;
-  values.sals[0] = find_pc_line (pc, 0);
-  values.sals[0].pc = pc;
-  values.sals[0].section = find_pc_overlay (pc);
-  values.sals[0].explicit_pc = 1;
+  if (!VEC_empty (symbolp, info.result.symbols)
+      || !VEC_empty (bound_minimal_symbol_d, info.result.minimal_symbols))
+    {
+      char *saved_arg;
+
+      saved_arg = alloca (new_argptr - arg + 1);
+      memcpy (saved_arg, arg, new_argptr - arg);
+      saved_arg[new_argptr - arg] = '\0';
+
+      ls->explicit.function_name = xstrdup (saved_arg);
+      ls->function_symbols = info.result.symbols;
+      ls->minimal_symbols = info.result.minimal_symbols;
+      values = convert_linespec_to_sals (self, ls);
+
+      if (self->canonical)
+       {
+         char *str;
+
+         self->canonical->pre_expanded = 1;
+
+         if (ls->explicit.source_filename)
+           {
+             str = xstrprintf ("%s:%s",
+                               ls->explicit.source_filename, saved_arg);
+           }
+         else
+           str = xstrdup (saved_arg);
+
+         make_cleanup (xfree, str);
+         self->canonical->location = new_linespec_location (&str);
+       }
+    }
+
+  do_cleanups (cleanup);
 
   return values;
 }
 
-\f
+/* An instance of this type is used when collecting prefix symbols for
+   decode_compound.  */
 
-/* Locate the first half of the linespec, ending in a colon, period,
-   or whitespace.  (More or less.)  Also, check to see if *ARGPTR is
-   enclosed in double quotes; if so, set is_quote_enclosed, advance
-   ARGPTR past that and zero out the trailing double quote.
-   If ARGPTR is just a simple name like "main", p will point to ""
-   at the end.  */
+struct decode_compound_collector
+{
+  /* The result vector.  */
+  VEC (symbolp) *symbols;
 
-static char *
-locate_first_half (char **argptr, int *is_quote_enclosed)
+  /* A hash table of all symbols we found.  We use this to avoid
+     adding any symbol more than once.  */
+  htab_t unique_syms;
+};
+
+/* A callback for iterate_over_symbols that is used by
+   lookup_prefix_sym to collect type symbols.  */
+
+static int
+collect_one_symbol (struct symbol *sym, void *d)
 {
-  char *ii;
-  char *p, *p1;
-  int has_comma;
+  struct decode_compound_collector *collector = d;
+  void **slot;
+  struct type *t;
 
-  /* Maybe we were called with a line range FILENAME:LINENUM,FILENAME:LINENUM
-     and we must isolate the first half.  Outer layers will call again later
-     for the second half.
+  if (SYMBOL_CLASS (sym) != LOC_TYPEDEF)
+    return 1; /* Continue iterating.  */
 
-     Don't count commas that appear in argument lists of overloaded
-     functions, or in quoted strings.  It's stupid to go to this much
-     trouble when the rest of the function is such an obvious roach hotel.  */
-  ii = find_toplevel_char (*argptr, ',');
-  has_comma = (ii != 0);
+  t = SYMBOL_TYPE (sym);
+  t = check_typedef (t);
+  if (TYPE_CODE (t) != TYPE_CODE_STRUCT
+      && TYPE_CODE (t) != TYPE_CODE_UNION
+      && TYPE_CODE (t) != TYPE_CODE_NAMESPACE)
+    return 1; /* Continue iterating.  */
 
-  /* Temporarily zap out second half to not confuse the code below.
-     This is undone below.  Do not change ii!!  */
-  if (has_comma)
+  slot = htab_find_slot (collector->unique_syms, sym, INSERT);
+  if (!*slot)
     {
-      *ii = '\0';
+      *slot = sym;
+      VEC_safe_push (symbolp, collector->symbols, sym);
     }
 
-  /* Maybe arg is FILE : LINENUM or FILE : FUNCTION.  May also be
-     CLASS::MEMBER, or NAMESPACE::NAME.  Look for ':', but ignore
-     inside of <>.  */
+  return 1; /* Continue iterating.  */
+}
 
-  p = *argptr;
-  if (p[0] == '"')
-    {
-      *is_quote_enclosed = 1;
-      (*argptr)++;
-      p++;
-    }
-  else
-    {
-      *is_quote_enclosed = 0;
-      if (strchr (get_gdb_completer_quote_characters (), *p))
-       {
-         ++(*argptr);
-         ++p;
-       }
-    }
-  for (; *p; p++)
-    {
-      if (p[0] == '<')
-       {
-         char *temp_end = find_template_name_end (p);
+/* Return any symbols corresponding to CLASS_NAME in FILE_SYMTABS.  */
 
-         if (!temp_end)
-           error (_("malformed template specification in command"));
-         p = temp_end;
-       }
+static VEC (symbolp) *
+lookup_prefix_sym (struct linespec_state *state, VEC (symtab_ptr) *file_symtabs,
+                  const char *class_name)
+{
+  int ix;
+  struct symtab *elt;
+  struct decode_compound_collector collector;
+  struct cleanup *outer;
+  struct cleanup *cleanup;
+
+  collector.symbols = NULL;
+  outer = make_cleanup (VEC_cleanup (symbolp), &collector.symbols);
 
-      if (p[0] == '(')
-       p = find_method_overload_end (p);
+  collector.unique_syms = htab_create_alloc (1, htab_hash_pointer,
+                                            htab_eq_pointer, NULL,
+                                            xcalloc, xfree);
+  cleanup = make_cleanup_htab_delete (collector.unique_syms);
 
-      /* Check for a colon and a plus or minus and a [ (which
-         indicates an Objective-C method).  */
-      if (is_objc_method_format (p))
+  for (ix = 0; VEC_iterate (symtab_ptr, file_symtabs, ix, elt); ++ix)
+    {
+      if (elt == NULL)
        {
-         break;
+         iterate_over_all_matching_symtabs (state, class_name, STRUCT_DOMAIN,
+                                            collect_one_symbol, &collector,
+                                            NULL, 0);
+         iterate_over_all_matching_symtabs (state, class_name, VAR_DOMAIN,
+                                            collect_one_symbol, &collector,
+                                            NULL, 0);
        }
-      /* Check for the end of the first half of the linespec.  End of
-         line, a tab, a double colon or the last single colon, or a
-         space.  But if enclosed in double quotes we do not break on
-         enclosed spaces.  */
-      if (!*p
-         || p[0] == '\t'
-         || ((p[0] == ':')
-             && ((p[1] == ':') || (strchr (p + 1, ':') == NULL)))
-         || ((p[0] == ' ') && !*is_quote_enclosed))
-       break;
-      if (p[0] == '.' && strchr (p, ':') == NULL)
+      else
        {
-         /* Java qualified method.  Find the *last* '.', since the
-            others are package qualifiers.  Stop at any open parenthesis
-            which might provide overload information.  */
-         for (p1 = p; *p1 && *p1 != '('; p1++)
-           {
-             if (*p1 == '.')
-               p = p1;
-           }
-         break;
+         /* Program spaces that are executing startup should have
+            been filtered out earlier.  */
+         gdb_assert (!SYMTAB_PSPACE (elt)->executing_startup);
+         set_current_program_space (SYMTAB_PSPACE (elt));
+         iterate_over_file_blocks (elt, class_name, STRUCT_DOMAIN,
+                                   collect_one_symbol, &collector);
+         iterate_over_file_blocks (elt, class_name, VAR_DOMAIN,
+                                   collect_one_symbol, &collector);
        }
     }
-  while (p[0] == ' ' || p[0] == '\t')
-    p++;
 
-  /* If the closing double quote was left at the end, remove it.  */
-  if (*is_quote_enclosed)
-    {
-      char *closing_quote = strchr (p - 1, '"');
+  do_cleanups (cleanup);
+  discard_cleanups (outer);
+  return collector.symbols;
+}
 
-      if (closing_quote && closing_quote[1] == '\0')
-       *closing_quote = '\0';
-    }
+/* A qsort comparison function for symbols.  The resulting order does
+   not actually matter; we just need to be able to sort them so that
+   symbols with the same program space end up next to each other.  */
+
+static int
+compare_symbols (const void *a, const void *b)
+{
+  struct symbol * const *sa = a;
+  struct symbol * const *sb = b;
+  uintptr_t uia, uib;
 
-  /* Now that we've safely parsed the first half, put back ',' so
-     outer layers can see it.  */
-  if (has_comma)
-    *ii = ',';
+  uia = (uintptr_t) SYMTAB_PSPACE (symbol_symtab (*sa));
+  uib = (uintptr_t) SYMTAB_PSPACE (symbol_symtab (*sb));
 
-  return p;
+  if (uia < uib)
+    return -1;
+  if (uia > uib)
+    return 1;
+
+  uia = (uintptr_t) *sa;
+  uib = (uintptr_t) *sb;
+
+  if (uia < uib)
+    return -1;
+  if (uia > uib)
+    return 1;
+
+  return 0;
 }
 
-\f
+/* Like compare_symbols but for minimal symbols.  */
 
-/* Here's where we recognise an Objective-C Selector.  An Objective C
-   selector may be implemented by more than one class, therefore it
-   may represent more than one method/function.  This gives us a
-   situation somewhat analogous to C++ overloading.  If there's more
-   than one method that could represent the selector, then use some of
-   the existing C++ code to let the user choose one.  */
+static int
+compare_msymbols (const void *a, const void *b)
+{
+  const struct bound_minimal_symbol *sa = a;
+  const struct bound_minimal_symbol *sb = b;
+  uintptr_t uia, uib;
 
-struct symtabs_and_lines
-decode_objc (char **argptr, int funfirstline, struct symtab *file_symtab,
-            struct linespec_result *canonical, char *saved_arg)
+  uia = (uintptr_t) sa->objfile->pspace;
+  uib = (uintptr_t) sa->objfile->pspace;
+
+  if (uia < uib)
+    return -1;
+  if (uia > uib)
+    return 1;
+
+  uia = (uintptr_t) sa->minsym;
+  uib = (uintptr_t) sb->minsym;
+
+  if (uia < uib)
+    return -1;
+  if (uia > uib)
+    return 1;
+
+  return 0;
+}
+
+/* Look for all the matching instances of each symbol in NAMES.  Only
+   instances from PSPACE are considered; other program spaces are
+   handled by our caller.  If PSPACE is NULL, then all program spaces
+   are considered.  Results are stored into INFO.  */
+
+static void
+add_all_symbol_names_from_pspace (struct collect_info *info,
+                                 struct program_space *pspace,
+                                 VEC (const_char_ptr) *names)
 {
-  struct symtabs_and_lines values;
-  struct symbol **sym_arr = NULL;
-  struct symbol *sym = NULL;
-  struct block *block = NULL;
-  unsigned i1 = 0;
-  unsigned i2 = 0;
+  int ix;
+  const char *iter;
 
-  values.sals = NULL;
-  values.nelts = 0;
+  for (ix = 0; VEC_iterate (const_char_ptr, names, ix, iter); ++ix)
+    add_matching_symbols_to_info (iter, info, pspace);
+}
 
-  if (file_symtab != NULL)
-    block = BLOCKVECTOR_BLOCK (BLOCKVECTOR (file_symtab), STATIC_BLOCK);
-  else
+static void
+find_superclass_methods (VEC (typep) *superclasses,
+                        const char *name,
+                        VEC (const_char_ptr) **result_names)
+{
+  int old_len = VEC_length (const_char_ptr, *result_names);
+  VEC (typep) *iter_classes;
+  struct cleanup *cleanup = make_cleanup (null_cleanup, NULL);
+
+  iter_classes = superclasses;
+  while (1)
     {
-      enum language save_language;
+      VEC (typep) *new_supers = NULL;
+      int ix;
+      struct type *t;
 
-      /* get_selected_block can change the current language when there is
-        no selected frame yet.  */
-      save_language = current_language->la_language;
-      block = get_selected_block (0);
-      set_language (save_language);
-    }
+      make_cleanup (VEC_cleanup (typep), &new_supers);
+      for (ix = 0; VEC_iterate (typep, iter_classes, ix, t); ++ix)
+       find_methods (t, name, result_names, &new_supers);
 
-  find_imps (file_symtab, block, *argptr, NULL, &i1, &i2); 
-    
-  if (i1 > 0)
-    {
-      sym_arr = (struct symbol **)
-       alloca ((i1 + 1) * sizeof (struct symbol *));
-      sym_arr[i1] = NULL;
+      if (VEC_length (const_char_ptr, *result_names) != old_len
+         || VEC_empty (typep, new_supers))
+       break;
 
-      *argptr = find_imps (file_symtab, block, *argptr, sym_arr, &i1, &i2);
+      iter_classes = new_supers;
     }
 
-  /* i1 now represents the TOTAL number of matches found.
-     i2 represents how many HIGH-LEVEL (struct symbol) matches,
-     which will come first in the sym_arr array.  Any low-level
-     (minimal_symbol) matches will follow those.  */
-      
-  if (i1 == 1)
+  do_cleanups (cleanup);
+}
+
+/* This finds the method METHOD_NAME in the class CLASS_NAME whose type is
+   given by one of the symbols in SYM_CLASSES.  Matches are returned
+   in SYMBOLS (for debug symbols) and MINSYMS (for minimal symbols).  */
+
+static void
+find_method (struct linespec_state *self, VEC (symtab_ptr) *file_symtabs,
+            const char *class_name, const char *method_name,
+            VEC (symbolp) *sym_classes, VEC (symbolp) **symbols,
+            VEC (bound_minimal_symbol_d) **minsyms)
+{
+  struct symbol *sym;
+  struct cleanup *cleanup = make_cleanup (null_cleanup, NULL);
+  int ix;
+  int last_result_len;
+  VEC (typep) *superclass_vec;
+  VEC (const_char_ptr) *result_names;
+  struct collect_info info;
+
+  /* Sort symbols so that symbols with the same program space are next
+     to each other.  */
+  qsort (VEC_address (symbolp, sym_classes),
+        VEC_length (symbolp, sym_classes),
+        sizeof (symbolp),
+        compare_symbols);
+
+  info.state = self;
+  info.file_symtabs = file_symtabs;
+  info.result.symbols = NULL;
+  info.result.minimal_symbols = NULL;
+
+  /* Iterate over all the types, looking for the names of existing
+     methods matching METHOD_NAME.  If we cannot find a direct method in a
+     given program space, then we consider inherited methods; this is
+     not ideal (ideal would be to respect C++ hiding rules), but it
+     seems good enough and is what GDB has historically done.  We only
+     need to collect the names because later we find all symbols with
+     those names.  This loop is written in a somewhat funny way
+     because we collect data across the program space before deciding
+     what to do.  */
+  superclass_vec = NULL;
+  make_cleanup (VEC_cleanup (typep), &superclass_vec);
+  result_names = NULL;
+  make_cleanup (VEC_cleanup (const_char_ptr), &result_names);
+  last_result_len = 0;
+  for (ix = 0; VEC_iterate (symbolp, sym_classes, ix, sym); ++ix)
     {
-      if (i2 > 0)
-       {
-         /* Already a struct symbol.  */
-         sym = sym_arr[0];
-       }
-      else
-       {
-         sym = find_pc_function (SYMBOL_VALUE_ADDRESS (sym_arr[0]));
-         if ((sym != NULL) && strcmp (SYMBOL_LINKAGE_NAME (sym_arr[0]),
-                                      SYMBOL_LINKAGE_NAME (sym)) != 0)
-           {
-             warning (_("debugging symbol \"%s\" does "
-                        "not match selector; ignoring"),
-                      SYMBOL_LINKAGE_NAME (sym));
-             sym = NULL;
-           }
-       }
-             
-      values.sals = (struct symtab_and_line *)
-       xmalloc (sizeof (struct symtab_and_line));
-      values.nelts = 1;
-             
-      if (sym && SYMBOL_CLASS (sym) == LOC_BLOCK)
-       {
-         /* Canonicalize this, so it remains resolved for dylib loads.  */
-         values.sals[0] = find_function_start_sal (sym, funfirstline);
-         build_canonical_line_spec (values.sals,
-                                    SYMBOL_NATURAL_NAME (sym), canonical);
-       }
-      else
+      struct type *t;
+      struct program_space *pspace;
+
+      /* Program spaces that are executing startup should have
+        been filtered out earlier.  */
+      pspace = SYMTAB_PSPACE (symbol_symtab (sym));
+      gdb_assert (!pspace->executing_startup);
+      set_current_program_space (pspace);
+      t = check_typedef (SYMBOL_TYPE (sym));
+      find_methods (t, method_name, &result_names, &superclass_vec);
+
+      /* Handle all items from a single program space at once; and be
+        sure not to miss the last batch.  */
+      if (ix == VEC_length (symbolp, sym_classes) - 1
+         || (pspace
+             != SYMTAB_PSPACE (symbol_symtab (VEC_index (symbolp, sym_classes,
+                                                         ix + 1)))))
        {
-         /* The only match was a non-debuggable symbol, which might point
-            to a function descriptor; resolve it to the actual code address
-            instead.  */
-         struct minimal_symbol *msymbol = (struct minimal_symbol *)sym_arr[0];
-         struct objfile *objfile = msymbol_objfile (msymbol);
-         struct gdbarch *gdbarch = get_objfile_arch (objfile);
-         CORE_ADDR pc = SYMBOL_VALUE_ADDRESS (msymbol);
-
-         pc = gdbarch_convert_from_func_ptr_addr (gdbarch, pc,
-                                                  &current_target);
-
-         init_sal (&values.sals[0]);
-         values.sals[0].pc = pc;
+         /* If we did not find a direct implementation anywhere in
+            this program space, consider superclasses.  */
+         if (VEC_length (const_char_ptr, result_names) == last_result_len)
+           find_superclass_methods (superclass_vec, method_name,
+                                    &result_names);
+
+         /* We have a list of candidate symbol names, so now we
+            iterate over the symbol tables looking for all
+            matches in this pspace.  */
+         add_all_symbol_names_from_pspace (&info, pspace, result_names);
+
+         VEC_truncate (typep, superclass_vec, 0);
+         last_result_len = VEC_length (const_char_ptr, result_names);
        }
-      return values;
     }
 
-  if (i1 > 1)
+  if (!VEC_empty (symbolp, info.result.symbols)
+      || !VEC_empty (bound_minimal_symbol_d, info.result.minimal_symbols))
     {
-      /* More than one match.  The user must choose one or more.  */
-      return decode_line_2 (sym_arr, i2, funfirstline, canonical);
+      *symbols = info.result.symbols;
+      *minsyms = info.result.minimal_symbols;
+      do_cleanups (cleanup);
+      return;
     }
 
-  return values;
+  /* Throw an NOT_FOUND_ERROR.  This will be caught by the caller
+     and other attempts to locate the symbol will be made.  */
+  throw_error (NOT_FOUND_ERROR, _("see caller, this text doesn't matter"));
 }
 
-/* This handles C++ and Java compound data structures.  P should point
-   at the first component separator, i.e. double-colon or period.  As
-   an example, on entrance to this function we could have ARGPTR
-   pointing to "AAA::inA::fun" and P pointing to "::inA::fun".  */
+\f
 
-static struct symtabs_and_lines
-decode_compound (char **argptr, int funfirstline,
-                struct linespec_result *canonical,
-                char *the_real_saved_arg, char *p)
+/* This object is used when collecting all matching symtabs.  */
+
+struct symtab_collector
 {
-  struct symtabs_and_lines values;
-  char *p2;
-  char *saved_arg2 = *argptr;
-  char *temp_end;
-  struct symbol *sym;
-  char *copy;
-  struct symbol *sym_class;
-  struct type *t;
-  char *saved_java_argptr = NULL;
-  char *saved_arg;
+  /* The result vector of symtabs.  */
+  VEC (symtab_ptr) *symtabs;
 
-  /* If the user specified any completer quote characters in the input,
-     strip them.  They are superfluous.  */
-  saved_arg = alloca (strlen (the_real_saved_arg) + 1);
-  {
-    char *dst = saved_arg;
-    char *src = the_real_saved_arg;
-    char *quotes = get_gdb_completer_quote_characters ();
-    while (*src != '\0')
-      {
-       if (strchr (quotes, *src) == NULL)
-         *dst++ = *src;
-       ++src;
-      }
-    *dst = '\0';
-  }
+  /* This is used to ensure the symtabs are unique.  */
+  htab_t symtab_table;
+};
 
-  /* First check for "global" namespace specification, of the form
-     "::foo".  If found, skip over the colons and jump to normal
-     symbol processing.  I.e. the whole line specification starts with
-     "::" (note the condition that *argptr == p).  */
-  if (p[0] == ':' 
-      && ((*argptr == p) || (p[-1] == ' ') || (p[-1] == '\t')))
-    saved_arg2 += 2;
+/* Callback for iterate_over_symtabs.  */
 
-  /* Given our example "AAA::inA::fun", we have two cases to consider:
+static int
+add_symtabs_to_list (struct symtab *symtab, void *d)
+{
+  struct symtab_collector *data = d;
+  void **slot;
 
-     1) AAA::inA is the name of a class.  In that case, presumably it
-        has a method called "fun"; we then look up that method using
-        find_method.
+  slot = htab_find_slot (data->symtab_table, symtab, INSERT);
+  if (!*slot)
+    {
+      *slot = symtab;
+      VEC_safe_push (symtab_ptr, data->symtabs, symtab);
+    }
 
-     2) AAA::inA isn't the name of a class.  In that case, either the
-        user made a typo, AAA::inA is the name of a namespace, or it is
-        the name of a minimal symbol.
-        We just look up AAA::inA::fun with lookup_symbol.  If that fails,
-        try lookup_minimal_symbol.
+  return 0;
+}
 
-     Thus, our first task is to find everything before the last set of
-     double-colons and figure out if it's the name of a class.  So we
-     first loop through all of the double-colons.  */
+/* Given a file name, return a VEC of all matching symtabs.  */
 
-  p2 = p;              /* Save for restart.  */
+static VEC (symtab_ptr) *
+collect_symtabs_from_filename (const char *file)
+{
+  struct symtab_collector collector;
+  struct cleanup *cleanups;
+  struct program_space *pspace;
 
-  /* This is very messy.  Following the example above we have now the
-     following pointers:
-     p -> "::inA::fun"
-     argptr -> "AAA::inA::fun
-     saved_arg -> "AAA::inA::fun
-     saved_arg2 -> "AAA::inA::fun
-     p2 -> "::inA::fun".  */
+  collector.symtabs = NULL;
+  collector.symtab_table = htab_create (1, htab_hash_pointer, htab_eq_pointer,
+                                       NULL);
+  cleanups = make_cleanup_htab_delete (collector.symtab_table);
 
-  /* In the loop below, with these strings, we'll make 2 passes, each
-     is marked in comments.  */
+  /* Find that file's data.  */
+  ALL_PSPACES (pspace)
+  {
+    if (pspace->executing_startup)
+      continue;
 
-  while (1)
-    {
-      static char *break_characters = " \t(";
+    set_current_program_space (pspace);
+    iterate_over_symtabs (file, add_symtabs_to_list, &collector);
+  }
 
-      /* Move pointer up to next possible class/namespace token.  */
+  do_cleanups (cleanups);
+  return collector.symtabs;
+}
 
-      p = p2 + 1;      /* Restart with old value +1.  */
+/* Return all the symtabs associated to the FILENAME.  */
 
-      /* PASS1: at this point p2->"::inA::fun", so p->":inA::fun",
-        i.e. if there is a double-colon, p will now point to the
-        second colon.  */
-      /* PASS2: p2->"::fun", p->":fun" */
+static VEC (symtab_ptr) *
+symtabs_from_filename (const char *filename)
+{
+  VEC (symtab_ptr) *result;
+  
+  result = collect_symtabs_from_filename (filename);
 
-      /* Move pointer ahead to next double-colon.  */
-      while (*p
-            && strchr (break_characters, *p) == NULL
-            && strchr (get_gdb_completer_quote_characters (), *p) == NULL)
-       {
-         if (current_language->la_language == language_cplus)
-           p += cp_validate_operator (p);
+  if (VEC_empty (symtab_ptr, result))
+    {
+      if (!have_full_symbols () && !have_partial_symbols ())
+       throw_error (NOT_FOUND_ERROR,
+                    _("No symbol table is loaded.  "
+                      "Use the \"file\" command."));
+      source_file_not_found_error (filename);
+    }
 
-         if (p[0] == '<')
-           {
-             temp_end = find_template_name_end (p);
-             if (!temp_end)
-               error (_("malformed template specification in command"));
-             p = temp_end;
-           }
-         /* Note that, since, at the start of this loop, p would be
-            pointing to the second colon in a double-colon, we only
-            satisfy the condition below if there is another
-            double-colon to the right (after).  I.e. there is another
-            component that can be a class or a namespace.  I.e, if at
-            the beginning of this loop (PASS1), we had
-            p->":inA::fun", we'll trigger this when p has been
-            advanced to point to "::fun".  */
-         /* PASS2: we will not trigger this.  */
-         else if ((p[0] == ':') && (p[1] == ':'))
-           break;      /* Found double-colon.  */
-         else
-           {
-             /* PASS2: We'll keep getting here, until P points to one of the
-                break characters, at which point we exit this loop.  */
-             if (*p)
-               {
-                 if (p[1] == '('
-                     && strncmp (&p[1], CP_ANONYMOUS_NAMESPACE_STR,
-                                 CP_ANONYMOUS_NAMESPACE_LEN) == 0)
-                   p += CP_ANONYMOUS_NAMESPACE_LEN;
-                 else if (strchr (break_characters, *p) == NULL)
-                   ++p;
-               }
-           }
-       }
+  return result;
+}
 
-      if (*p != ':')
-       break;          /* Out of the while (1).  This would happen
-                          for instance if we have looked up
-                          unsuccessfully all the components of the
-                          string, and p->""(PASS2).  */
-
-      /* We get here if p points to one of the break characters or "" (i.e.,
-        string ended).  */
-      /* Save restart for next time around.  */
-      p2 = p;
-      /* Restore argptr as it was on entry to this function.  */
-      *argptr = saved_arg2;
-      /* PASS1: at this point p->"::fun" argptr->"AAA::inA::fun",
-        p2->"::fun".  */
-
-      /* All ready for next pass through the loop.  */
-    }                  /* while (1) */
-
-
-  /* Start of lookup in the symbol tables.  */
-
-  /* Lookup in the symbol table the substring between argptr and
-     p.  Note, this call changes the value of argptr.  */
-  /* Before the call, argptr->"AAA::inA::fun",
-     p->"", p2->"::fun".  After the call: argptr->"fun", p, p2
-     unchanged.  */
-  sym_class = lookup_prefix_sym (argptr, p2);
-
-  /* If sym_class has been found, and if "AAA::inA" is a class, then
-     we're in case 1 above.  So we look up "fun" as a method of that
-     class.  */
-  if (sym_class &&
-      (t = check_typedef (SYMBOL_TYPE (sym_class)),
-       (TYPE_CODE (t) == TYPE_CODE_STRUCT
-       || TYPE_CODE (t) == TYPE_CODE_UNION)))
-    {
-      /* Arg token is not digits => try it as a function name.
-        Find the next token (everything up to end or next
-        blank).  */
-      if (**argptr
-         && strchr (get_gdb_completer_quote_characters (),
-                    **argptr) != NULL)
-       {
-         p = skip_quoted (*argptr);
-         *argptr = *argptr + 1;
-       }
-      else
-       {
-         /* At this point argptr->"fun".  */
-         char *a;
-
-         p = *argptr;
-         while (*p && *p != ' ' && *p != '\t' && *p != ',' && *p != ':'
-                && *p != '(')
-           p++;
-         /* At this point p->"".  String ended.  */
-         /* Nope, C++ operators could have spaces in them
-            ("foo::operator <" or "foo::operator delete []").
-            I apologize, this is a bit hacky...  */
-         if (current_language->la_language == language_cplus
-             && *p == ' ' && p - 8 - *argptr + 1 > 0)
-           {
-             /* The above loop has already swallowed "operator".  */
-             p += cp_validate_operator (p - 8) - 8;
-           }
+/* Look up a function symbol named NAME in symtabs FILE_SYMTABS.  Matching
+   debug symbols are returned in SYMBOLS.  Matching minimal symbols are
+   returned in MINSYMS.  */
 
-         /* Keep any important naming information.  */
-         p = keep_name_info (p);
+static void
+find_function_symbols (struct linespec_state *state,
+                      VEC (symtab_ptr) *file_symtabs, const char *name,
+                      VEC (symbolp) **symbols,
+                      VEC (bound_minimal_symbol_d) **minsyms)
+{
+  struct collect_info info;
+  VEC (const_char_ptr) *symbol_names = NULL;
+  struct cleanup *cleanup = make_cleanup (VEC_cleanup (const_char_ptr),
+                                         &symbol_names);
+
+  info.state = state;
+  info.result.symbols = NULL;
+  info.result.minimal_symbols = NULL;
+  info.file_symtabs = file_symtabs;
+
+  /* Try NAME as an Objective-C selector.  */
+  find_imps (name, &symbol_names);
+  if (!VEC_empty (const_char_ptr, symbol_names))
+    add_all_symbol_names_from_pspace (&info, NULL, symbol_names);
+  else
+    add_matching_symbols_to_info (name, &info, NULL);
 
-         /* Java may append typenames,  so assume that if there is
-            anything else left in *argptr, it must be a typename.  */
-         if (*p && current_language->la_language == language_java)
-           {
-             struct type *type;
-
-             p2 = p;
-             while (*p2)
-               ++p2;
-             copy = (char *) alloca (p2 - p + 1);
-             memcpy (copy, p, p2 - p);
-             copy[p2 - p] = '\0';
-             type = lookup_typename (current_language, get_current_arch (),
-                                     copy, NULL, 1);
-             if (type != NULL)
-               {
-                 /* Save the location of this just in case this
-                    method/type combination isn't actually defined.
-                    It will be checked later.  */
-                 saved_java_argptr = p;
-                 p = p2;
-               }
-           }
-       }
+  do_cleanups (cleanup);
 
-      /* Allocate our own copy of the substring between argptr and
-        p.  */
-      copy = (char *) alloca (p - *argptr + 1);
-      memcpy (copy, *argptr, p - *argptr);
-      copy[p - *argptr] = '\0';
-      if (p != *argptr
-         && copy[p - *argptr - 1]
-         && strchr (get_gdb_completer_quote_characters (),
-                    copy[p - *argptr - 1]) != NULL)
-       copy[p - *argptr - 1] = '\0';
-
-      /* At this point copy->"fun", p->"".  */
-
-      /* No line number may be specified.  */
-      while (*p == ' ' || *p == '\t')
-       p++;
-      *argptr = p;
-      /* At this point arptr->"".  */
-
-      /* Look for copy as a method of sym_class.  */
-      /* At this point copy->"fun", sym_class is "AAA:inA",
-        saved_arg->"AAA::inA::fun".  This concludes the scanning of
-        the string for possible components matches.  If we find it
-        here, we return.  If not, and we are at the and of the string,
-        we'll lookup the whole string in the symbol tables.  */
-
-      values = find_method (funfirstline, canonical, saved_arg,
-                           copy, t, sym_class);
-      if (saved_java_argptr != NULL && values.nelts == 1)
-       {
-         /* The user specified a specific return type for a java method.
-            Double-check that it really is the one the user specified.
-            [This is a necessary evil because strcmp_iw_ordered stops
-            comparisons too prematurely.]  */
-         sym = find_pc_sect_function (values.sals[0].pc,
-                                      values.sals[0].section);
-         /* We just found a SAL, we had better be able to go backwards!  */
-         gdb_assert (sym != NULL);
-         if (strcmp_iw (SYMBOL_LINKAGE_NAME (sym), saved_arg) != 0)
-           {
-             xfree (values.sals);
-             error (_("the class `%s' does not have "
-                      "any method instance named %s"),
-                    SYMBOL_PRINT_NAME (sym_class), copy);
-           }
-       }
-      return values;
-    } /* End if symbol found.  */
+  if (VEC_empty (symbolp, info.result.symbols))
+    {
+      VEC_free (symbolp, info.result.symbols);
+      *symbols = NULL;
+    }
+  else
+    *symbols = info.result.symbols;
 
+  if (VEC_empty (bound_minimal_symbol_d, info.result.minimal_symbols))
+    {
+      VEC_free (bound_minimal_symbol_d, info.result.minimal_symbols);
+      *minsyms = NULL;
+    }
+  else
+    *minsyms = info.result.minimal_symbols;
+}
 
-  /* We couldn't find a class, so we're in case 2 above.  We check the
-     entire name as a symbol instead.  */
+/* Find all symbols named NAME in FILE_SYMTABS, returning debug symbols
+   in SYMBOLS and minimal symbols in MINSYMS.  */
 
-  p = keep_name_info (p);
+static void
+find_linespec_symbols (struct linespec_state *state,
+                      VEC (symtab_ptr) *file_symtabs,
+                      const char *name,
+                      VEC (symbolp) **symbols,
+                      VEC (bound_minimal_symbol_d) **minsyms)
+{
+  struct cleanup *cleanup;
+  char *canon;
+  const char *lookup_name;
 
-  copy = (char *) alloca (p - saved_arg2 + 1);
-  memcpy (copy, saved_arg2, p - saved_arg2);
-  /* Note: if is_quoted should be true, we snuff out quote here
-     anyway.  */
-  copy[p - saved_arg2] = '\000';
-  /* Set argptr to skip over the name.  */
-  *argptr = (*p == '\'') ? p + 1 : p;
+  cleanup = demangle_for_lookup (name, state->language->la_language,
+                                &lookup_name);
+  if (state->language->la_language == language_ada)
+    {
+      /* In Ada, the symbol lookups are performed using the encoded
+         name rather than the demangled name.  */
+      lookup_name = ada_name_for_lookup (name);
+      make_cleanup (xfree, (void *) lookup_name);
+    }
 
-  /* Look up entire name.  */
-  sym = lookup_symbol (copy, get_selected_block (0), VAR_DOMAIN, 0);
-  if (sym)
-    return symbol_found (funfirstline, canonical, copy, sym, NULL, NULL);
-  else
+  canon = cp_canonicalize_string_no_typedefs (lookup_name);
+  if (canon != NULL)
     {
-      struct minimal_symbol *msym;
+      lookup_name = canon;
+      make_cleanup (xfree, canon);
+    }
 
-      /* Couldn't find any interpretation as classes/namespaces.  As a last
-        resort, try the minimal symbol tables.  */
-      msym = lookup_minimal_symbol (copy, NULL, NULL);
-      if (msym != NULL)
-       return minsym_found (funfirstline, msym);
-    }    
+  /* It's important to not call expand_symtabs_matching unnecessarily
+     as it can really slow things down (by unnecessarily expanding
+     potentially 1000s of symtabs, which when debugging some apps can
+     cost 100s of seconds).  Avoid this to some extent by *first* calling
+     find_function_symbols, and only if that doesn't find anything
+     *then* call find_method.  This handles two important cases:
+     1) break (anonymous namespace)::foo
+     2) break class::method where method is in class (and not a baseclass)  */
+
+  find_function_symbols (state, file_symtabs, lookup_name,
+                        symbols, minsyms);
+
+  /* If we were unable to locate a symbol of the same name, try dividing
+     the name into class and method names and searching the class and its
+     baseclasses.  */
+  if (VEC_empty (symbolp, *symbols)
+      && VEC_empty (bound_minimal_symbol_d, *minsyms))
+    {
+      char *klass, *method;
+      const char *last, *p, *scope_op;
+      VEC (symbolp) *classes;
+
+      /* See if we can find a scope operator and break this symbol
+        name into namespaces${SCOPE_OPERATOR}class_name and method_name.  */
+      scope_op = "::";
+      p = find_toplevel_string (lookup_name, scope_op);
+      if (p == NULL)
+       {
+         /* No C++ scope operator.  Try Java.  */
+         scope_op = ".";
+         p = find_toplevel_string (lookup_name, scope_op);
+       }
 
-  /* Couldn't find a minimal symbol, either, so give up.  */
-  cplusplus_error (the_real_saved_arg,
-                  "Can't find member of namespace, "
-                  "class, struct, or union named \"%s\"\n",
-                  copy);
-}
+      last = NULL;
+      while (p != NULL)
+       {
+         last = p;
+         p = find_toplevel_string (p + strlen (scope_op), scope_op);
+       }
 
-/* Next come some helper functions for decode_compound.  */
+      /* If no scope operator was found, there is nothing more we can do;
+        we already attempted to lookup the entire name as a symbol
+        and failed.  */
+      if (last == NULL)
+       {
+         do_cleanups (cleanup);
+         return;
+       }
 
-/* Return the symbol corresponding to the substring of *ARGPTR ending
-   at P, allowing whitespace.  Also, advance *ARGPTR past the symbol
-   name in question, the compound object separator ("::" or "."), and
-   whitespace.  Note that *ARGPTR is changed whether or not the
-   lookup_symbol call finds anything (i.e we return NULL).  As an
-   example, say ARGPTR is "AAA::inA::fun" and P is "::inA::fun".  */
+      /* LOOKUP_NAME points to the class name.
+        LAST points to the method name.  */
+      klass = xmalloc ((last - lookup_name + 1) * sizeof (char));
+      make_cleanup (xfree, klass);
+      strncpy (klass, lookup_name, last - lookup_name);
+      klass[last - lookup_name] = '\0';
 
-static struct symbol *
-lookup_prefix_sym (char **argptr, char *p)
-{
-  char *p1;
-  char *copy;
-  struct symbol *sym;
+      /* Skip past the scope operator.  */
+      last += strlen (scope_op);
+      method = xmalloc ((strlen (last) + 1) * sizeof (char));
+      make_cleanup (xfree, method);
+      strcpy (method, last);
 
-  /* Extract the class name.  */
-  p1 = p;
-  while (p != *argptr && p[-1] == ' ')
-    --p;
-  copy = (char *) alloca (p - *argptr + 1);
-  memcpy (copy, *argptr, p - *argptr);
-  copy[p - *argptr] = 0;
-
-  /* Discard the class name from the argptr.  */
-  p = p1 + (p1[0] == ':' ? 2 : 1);
-  while (*p == ' ' || *p == '\t')
-    p++;
-  *argptr = p;
-
-  /* At this point p1->"::inA::fun", p->"inA::fun" copy->"AAA",
-     argptr->"inA::fun".  */
-
-  sym = lookup_symbol (copy, get_selected_block (0), STRUCT_DOMAIN, 0);
-  if (sym == NULL)
-    {
-      /* Typedefs are in VAR_DOMAIN so the above symbol lookup will
-        fail when the user attempts to lookup a method of a class
-        via a typedef'd name (NOT via the class's name, which is already
-        handled in symbol_matches_domain).  So try the lookup again
-        using VAR_DOMAIN (where typedefs live) and double-check that we
-        found a struct/class type.  */
-      struct symbol *s = lookup_symbol (copy, 0, VAR_DOMAIN, 0);
+      /* Find a list of classes named KLASS.  */
+      classes = lookup_prefix_sym (state, file_symtabs, klass);
+      make_cleanup (VEC_cleanup (symbolp), &classes);
 
-      if (s != NULL)
+      if (!VEC_empty (symbolp, classes))
        {
-         struct type *t = SYMBOL_TYPE (s);
+         /* Now locate a list of suitable methods named METHOD.  */
+         TRY
+           {
+             find_method (state, file_symtabs, klass, method, classes,
+                          symbols, minsyms);
+           }
 
-         CHECK_TYPEDEF (t);
-         if (TYPE_CODE (t) == TYPE_CODE_STRUCT)
-           return s;
+         /* If successful, we're done.  If NOT_FOUND_ERROR
+            was not thrown, rethrow the exception that we did get.  */
+         CATCH (except, RETURN_MASK_ERROR)
+           {
+             if (except.error != NOT_FOUND_ERROR)
+               throw_exception (except);
+           }
+         END_CATCH
        }
     }
 
-  return sym;
+  do_cleanups (cleanup);
 }
 
-/* This finds the method COPY in the class whose type is T and whose
-   symbol is SYM_CLASS.  */
+/* Return all labels named NAME in FUNCTION_SYMBOLS.  Return the
+   actual function symbol in which the label was found in LABEL_FUNC_RET.  */
 
-static struct symtabs_and_lines
-find_method (int funfirstline, struct linespec_result *canonical,
-            char *saved_arg,
-            char *copy, struct type *t, struct symbol *sym_class)
+static VEC (symbolp) *
+find_label_symbols (struct linespec_state *self,
+                   VEC (symbolp) *function_symbols,
+                   VEC (symbolp) **label_funcs_ret, const char *name)
 {
-  struct symtabs_and_lines values;
-  struct symbol *sym = NULL;
-  int i1;      /*  Counter for the symbol array.  */
-  struct symbol **sym_arr =  alloca (total_number_of_methods (t)
-                                    * sizeof (struct symbol *));
+  int ix;
+  const struct block *block;
+  struct symbol *sym;
+  struct symbol *fn_sym;
+  VEC (symbolp) *result = NULL;
 
-  /* Find all methods with a matching name, and put them in
-     sym_arr.  */
+  if (function_symbols == NULL)
+    {
+      set_current_program_space (self->program_space);
+      block = get_current_search_block ();
 
-  i1 = find_methods (t, copy, SYMBOL_LANGUAGE (sym_class), sym_arr);
+      for (;
+          block && !BLOCK_FUNCTION (block);
+          block = BLOCK_SUPERBLOCK (block))
+       ;
+      if (!block)
+       return NULL;
+      fn_sym = BLOCK_FUNCTION (block);
 
-  if (i1 == 1)
-    {
-      /* There is exactly one field with that name.  */
-      sym = sym_arr[0];
+      sym = lookup_symbol (name, block, LABEL_DOMAIN, 0).symbol;
 
-      if (sym && SYMBOL_CLASS (sym) == LOC_BLOCK)
-       {
-         values.sals = (struct symtab_and_line *)
-           xmalloc (sizeof (struct symtab_and_line));
-         values.nelts = 1;
-         values.sals[0] = find_function_start_sal (sym,
-                                                   funfirstline);
-       }
-      else
+      if (sym != NULL)
        {
-         values.sals = NULL;
-         values.nelts = 0;
+         VEC_safe_push (symbolp, result, sym);
+         VEC_safe_push (symbolp, *label_funcs_ret, fn_sym);
        }
-      return values;
     }
-  if (i1 > 0)
+  else
     {
-      /* If we were given a specific overload instance, use that
-        (or error if no matches were found).  Otherwise ask the user
-        which one to use.  */
-      if (strchr (copy, '('))
+      for (ix = 0;
+          VEC_iterate (symbolp, function_symbols, ix, fn_sym); ++ix)
        {
-         int i;
-         char *name;
-         char *canon;
-         struct cleanup *cleanup;
-
-         /* Construct the proper search name based on SYM_CLASS and COPY.
-            SAVED_ARG may contain a valid name, but that name might not be
-            what is actually stored in the symbol table.  For example,
-            if SAVED_ARG (and SYM_CLASS) were found via an import
-            ("using namespace" in C++), then the physname of
-            SYM_CLASS ("A::myclass") may not be the same as SAVED_ARG
-            ("myclass").  */
-         name = xmalloc (strlen (SYMBOL_NATURAL_NAME (sym_class))
-                         + 2 /* "::" */ + strlen (copy) + 1);
-         strcpy (name, SYMBOL_NATURAL_NAME (sym_class));
-         strcat (name, "::");
-         strcat (name, copy);
-         canon = cp_canonicalize_string (name);
-         if (canon != NULL)
-           {
-             xfree (name);
-             name = canon;
-           }
-         cleanup = make_cleanup (xfree, name);
+         set_current_program_space (SYMTAB_PSPACE (symbol_symtab (fn_sym)));
+         block = SYMBOL_BLOCK_VALUE (fn_sym);
+         sym = lookup_symbol (name, block, LABEL_DOMAIN, 0).symbol;
 
-         for (i = 0; i < i1; ++i)
+         if (sym != NULL)
            {
-             if (strcmp_iw (name, SYMBOL_LINKAGE_NAME (sym_arr[i])) == 0)
-               {
-                 values.sals = (struct symtab_and_line *)
-                   xmalloc (sizeof (struct symtab_and_line));
-                 values.nelts = 1;
-                 values.sals[0] = find_function_start_sal (sym_arr[i],
-                                                           funfirstline);
-                 do_cleanups (cleanup);
-                 return values;
-               }
+             VEC_safe_push (symbolp, result, sym);
+             VEC_safe_push (symbolp, *label_funcs_ret, fn_sym);
            }
-
-         error (_("the class `%s' does not have "
-                  "any method instance named %s"),
-                SYMBOL_PRINT_NAME (sym_class), copy);
        }
-
-      return decode_line_2 (sym_arr, i1, funfirstline, canonical);
-    }
-  else
-    {
-      if (copy[0] == '~')
-       cplusplus_error (saved_arg,
-                        "the class `%s' does not have destructor defined\n",
-                        SYMBOL_PRINT_NAME (sym_class));
-      else
-       cplusplus_error (saved_arg,
-                        "the class %s does not have any method named %s\n",
-                        SYMBOL_PRINT_NAME (sym_class), copy);
     }
+
+  return result;
 }
 
 \f
 
-/* Return the symtab associated to the filename given by the substring
-   of *ARGPTR ending at P, and advance ARGPTR past that filename.  */
+/* A helper for create_sals_line_offset that handles the 'list_mode' case.  */
 
-static struct symtab *
-symtab_from_filename (char **argptr, char *p, int is_quote_enclosed)
+static void
+decode_digits_list_mode (struct linespec_state *self,
+                        linespec_p ls,
+                        struct symtabs_and_lines *values,
+                        struct symtab_and_line val)
 {
-  char *p1;
-  char *copy;
-  struct symtab *file_symtab;
-  
-  p1 = p;
-  while (p != *argptr && p[-1] == ' ')
-    --p;
-  if ((*p == '"') && is_quote_enclosed)
-    --p;
-  copy = (char *) alloca (p - *argptr + 1);
-  memcpy (copy, *argptr, p - *argptr);
-  /* It may have the ending quote right after the file name.  */
-  if ((is_quote_enclosed && copy[p - *argptr - 1] == '"')
-      || copy[p - *argptr - 1] == '\'')
-    copy[p - *argptr - 1] = 0;
-  else
-    copy[p - *argptr] = 0;
-
-  /* Find that file's data.  */
-  file_symtab = lookup_symtab (copy);
-  if (file_symtab == 0)
-    {
-      if (!have_full_symbols () && !have_partial_symbols ())
-       throw_error (NOT_FOUND_ERROR,
-                    _("No symbol table is loaded.  "
-                      "Use the \"file\" command."));
-      throw_error (NOT_FOUND_ERROR, _("No source file named %s."), copy);
-    }
-
-  /* Discard the file name from the arg.  */
-  p = p1 + 1;
-  while (*p == ' ' || *p == '\t')
-    p++;
-  *argptr = p;
-
-  return file_symtab;
-}
-
-/* Look up a function symbol in *ARGPTR.  If found, advance *ARGPTR
-   and return the symbol.  If not found, return NULL.  */
+  int ix;
+  struct symtab *elt;
 
-static struct symbol *
-find_function_symbol (char **argptr, char *p, int is_quote_enclosed)
-{
-  char *p1;
-  char *copy;
-  struct symbol *function_symbol;
+  gdb_assert (self->list_mode);
 
-  p1 = p;
-  while (p != *argptr && p[-1] == ' ')
-    --p;
-  if ((*p == '"') && is_quote_enclosed)
-    --p;
-  copy = (char *) alloca (p - *argptr + 1);
-  memcpy (copy, *argptr, p - *argptr);
-  /* It may have the ending quote right after the file name.  */
-  if ((is_quote_enclosed && copy[p - *argptr - 1] == '"')
-      || copy[p - *argptr - 1] == '\'')
-    copy[p - *argptr - 1] = 0;
-  else
-    copy[p - *argptr] = 0;
+  for (ix = 0; VEC_iterate (symtab_ptr, ls->file_symtabs, ix, elt);
+       ++ix)
+    {
+      /* The logic above should ensure this.  */
+      gdb_assert (elt != NULL);
 
-  function_symbol = lookup_symbol (copy, get_selected_block (0),
-                                  VAR_DOMAIN, 0);
-  if (!function_symbol || SYMBOL_CLASS (function_symbol) != LOC_BLOCK)
-    return NULL;
+      set_current_program_space (SYMTAB_PSPACE (elt));
 
-  /* Discard the file name from the arg.  */
-  p = p1 + 1;
-  while (*p == ' ' || *p == '\t')
-    p++;
-  *argptr = p;
+      /* Simplistic search just for the list command.  */
+      val.symtab = find_line_symtab (elt, val.line, NULL, NULL);
+      if (val.symtab == NULL)
+       val.symtab = elt;
+      val.pspace = SYMTAB_PSPACE (elt);
+      val.pc = 0;
+      val.explicit_line = 1;
 
-  return function_symbol;
+      add_sal_to_sals (self, values, &val, NULL, 0);
+    }
 }
 
-\f
-
-/* This decodes a line where the argument is all digits (possibly
-   preceded by a sign).  Q should point to the end of those digits;
-   the other arguments are as usual.  */
-
-static struct symtabs_and_lines
-decode_all_digits (char **argptr, struct symtab *default_symtab,
-                  int default_line, struct linespec_result *canonical,
-                  struct symtab *file_symtab, char *q)
+/* A helper for create_sals_line_offset that iterates over the symtabs,
+   adding lines to the VEC.  */
 
+static void
+decode_digits_ordinary (struct linespec_state *self,
+                       linespec_p ls,
+                       int line,
+                       struct symtabs_and_lines *sals,
+                       struct linetable_entry **best_entry)
 {
-  struct symtabs_and_lines values;
-  struct symtab_and_line val;
+  int ix;
+  struct symtab *elt;
 
-  enum sign
+  for (ix = 0; VEC_iterate (symtab_ptr, ls->file_symtabs, ix, elt); ++ix)
     {
-      none, plus, minus
-    }
-  sign = none;
-
-  /* We might need a canonical line spec if no file was specified.  */
-  int need_canonical = (file_symtab == NULL) ? 1 : 0;
+      int i;
+      VEC (CORE_ADDR) *pcs;
+      CORE_ADDR pc;
 
-  init_sal (&val);
+      /* The logic above should ensure this.  */
+      gdb_assert (elt != NULL);
 
-  val.pspace = current_program_space;
+      set_current_program_space (SYMTAB_PSPACE (elt));
 
-  /* This is where we need to make sure that we have good defaults.
-     We must guarantee that this section of code is never executed
-     when we are called with just a function name, since
-     set_default_source_symtab_and_line uses
-     select_source_symtab that calls us with such an argument.  */
+      pcs = find_pcs_for_symtab_line (elt, line, best_entry);
+      for (i = 0; VEC_iterate (CORE_ADDR, pcs, i, pc); ++i)
+       {
+         struct symtab_and_line sal;
+
+         init_sal (&sal);
+         sal.pspace = SYMTAB_PSPACE (elt);
+         sal.symtab = elt;
+         sal.line = line;
+         sal.pc = pc;
+         add_sal_to_sals_basic (sals, &sal);
+       }
 
-  if (file_symtab == 0 && default_symtab == 0)
-    {
-      /* Make sure we have at least a default source file.  */
-      set_default_source_symtab_and_line ();
-      initialize_defaults (&default_symtab, &default_line);
+      VEC_free (CORE_ADDR, pcs);
     }
-
-  if (**argptr == '+')
-    sign = plus, (*argptr)++;
-  else if (**argptr == '-')
-    sign = minus, (*argptr)++;
-  val.line = atoi (*argptr);
-  switch (sign)
-    {
-    case plus:
-      if (q == *argptr)
-       val.line = 5;
-      if (file_symtab == 0)
-       val.line = default_line + val.line;
-      break;
-    case minus:
-      if (q == *argptr)
-       val.line = 15;
-      if (file_symtab == 0)
-       val.line = default_line - val.line;
-      else
-       val.line = 1;
-      break;
-    case none:
-      break;           /* No need to adjust val.line.  */
-    }
-
-  while (*q == ' ' || *q == '\t')
-    q++;
-  *argptr = q;
-  if (file_symtab == 0)
-    file_symtab = default_symtab;
-
-  /* It is possible that this source file has more than one symtab, 
-     and that the new line number specification has moved us from the
-     default (in file_symtab) to a new one.  */
-  val.symtab = find_line_symtab (file_symtab, val.line, NULL, NULL);
-  if (val.symtab == 0)
-    val.symtab = file_symtab;
-
-  val.pspace = SYMTAB_PSPACE (val.symtab);
-  val.pc = 0;
-  values.sals = (struct symtab_and_line *)
-    xmalloc (sizeof (struct symtab_and_line));
-  values.sals[0] = val;
-  values.nelts = 1;
-  if (need_canonical)
-    build_canonical_line_spec (values.sals, NULL, canonical);
-  values.sals[0].explicit_line = 1;
-  return values;
 }
 
 \f
 
-/* Decode a linespec starting with a dollar sign.  */
+/* Return the line offset represented by VARIABLE.  */
 
-static struct symtabs_and_lines
-decode_dollar (char *copy, int funfirstline, struct symtab *default_symtab,
-              struct linespec_result *canonical, struct symtab *file_symtab)
+static struct line_offset
+linespec_parse_variable (struct linespec_state *self, const char *variable)
 {
-  LONGEST valx;
   int index = 0;
-  int need_canonical = 0;
-  struct symtabs_and_lines values;
-  struct symtab_and_line val;
-  char *p;
-  struct symbol *sym;
-  struct minimal_symbol *msymbol;
+  const char *p;
+  struct line_offset offset = {0, LINE_OFFSET_NONE};
 
-  p = (copy[1] == '$') ? copy + 2 : copy + 1;
+  p = (variable[1] == '$') ? variable + 2 : variable + 1;
+  if (*p == '$')
+    ++p;
   while (*p >= '0' && *p <= '9')
-    p++;
+    ++p;
   if (!*p)             /* Reached end of token without hitting non-digit.  */
     {
       /* We have a value history reference.  */
       struct value *val_history;
 
-      sscanf ((copy[1] == '$') ? copy + 2 : copy + 1, "%d", &index);
-      val_history = access_value_history ((copy[1] == '$') ? -index : index);
+      sscanf ((variable[1] == '$') ? variable + 2 : variable + 1, "%d", &index);
+      val_history
+       = access_value_history ((variable[1] == '$') ? -index : index);
       if (TYPE_CODE (value_type (val_history)) != TYPE_CODE_INT)
        error (_("History values used in line "
                 "specs must have integer values."));
-      valx = value_as_long (val_history);
+      offset.offset = value_as_long (val_history);
     }
   else
     {
       /* Not all digits -- may be user variable/function or a
         convenience variable.  */
-
-      /* Look up entire name as a symbol first.  */
-      sym = lookup_symbol (copy, 0, VAR_DOMAIN, 0);
-      file_symtab = (struct symtab *) NULL;
-      need_canonical = 1;
-      /* Symbol was found --> jump to normal symbol processing.  */
-      if (sym)
-       return symbol_found (funfirstline, canonical, copy, sym, NULL, NULL);
-
-      /* If symbol was not found, look in minimal symbol tables.  */
-      msymbol = lookup_minimal_symbol (copy, NULL, NULL);
-      /* Min symbol was found --> jump to minsym processing.  */
-      if (msymbol)
-       return minsym_found (funfirstline, msymbol);
-
-      /* Not a user variable or function -- must be convenience variable.  */
-      if (!get_internalvar_integer (lookup_internalvar (copy + 1), &valx))
-       error (_("Convenience variables used in line "
-                "specs must have integer values."));
+      LONGEST valx;
+      struct internalvar *ivar;
+
+      /* Try it as a convenience variable.  If it is not a convenience
+        variable, return and allow normal symbol lookup to occur.  */
+      ivar = lookup_only_internalvar (variable + 1);
+      if (ivar == NULL)
+       /* No internal variable with that name.  Mark the offset
+          as unknown to allow the name to be looked up as a symbol.  */
+       offset.sign = LINE_OFFSET_UNKNOWN;
+      else
+       {
+         /* We found a valid variable name.  If it is not an integer,
+            throw an error.  */
+         if (!get_internalvar_integer (ivar, &valx))
+           error (_("Convenience variables used in line "
+                    "specs must have integer values."));
+         else
+           offset.offset = valx;
+       }
     }
 
-  init_sal (&val);
-
-  /* Either history value or convenience value from above, in valx.  */
-  val.symtab = file_symtab ? file_symtab : default_symtab;
-  val.line = valx;
-  val.pc = 0;
-  val.pspace = current_program_space;
+  return offset;
+}
+\f
 
-  values.sals = (struct symtab_and_line *) xmalloc (sizeof val);
-  values.sals[0] = val;
-  values.nelts = 1;
+/* A callback used to possibly add a symbol to the results.  */
 
-  if (need_canonical)
-    build_canonical_line_spec (values.sals, NULL, canonical);
+static int
+collect_symbols (struct symbol *sym, void *data)
+{
+  struct collect_info *info = data;
 
-  return values;
+  /* In list mode, add all matching symbols, regardless of class.
+     This allows the user to type "list a_global_variable".  */
+  if (SYMBOL_CLASS (sym) == LOC_BLOCK || info->state->list_mode)
+    VEC_safe_push (symbolp, info->result.symbols, sym);
+  return 1; /* Continue iterating.  */
 }
 
-\f
-
-/* A helper for decode_line_1 that tries to find a label.  The label
-   is searched for in the current block.
-   FUNCTION_SYMBOL is the enclosing function; or NULL if none
-   specified.
-   COPY is the name of the label to find.
-   CANONICAL is the same as the "canonical" argument to decode_line_1.
-   RESULT is a pointer to a symtabs_and_lines structure which will be
-   filled in on success.
-   This function returns 1 if a label was found, 0 otherwise.  */
+/* We've found a minimal symbol MSYMBOL in OBJFILE to associate with our
+   linespec; return the SAL in RESULT.  This function should return SALs
+   matching those from find_function_start_sal, otherwise false
+   multiple-locations breakpoints could be placed.  */
 
-static int
-decode_label (struct symbol *function_symbol, char *copy,
-             struct linespec_result *canonical,
+static void
+minsym_found (struct linespec_state *self, struct objfile *objfile,
+             struct minimal_symbol *msymbol,
              struct symtabs_and_lines *result)
 {
-  struct symbol *sym;
-  struct block *block;
+  struct gdbarch *gdbarch = get_objfile_arch (objfile);
+  CORE_ADDR pc;
+  struct symtab_and_line sal;
 
-  if (function_symbol)
-    block = SYMBOL_BLOCK_VALUE (function_symbol);
-  else
+  sal = find_pc_sect_line (MSYMBOL_VALUE_ADDRESS (objfile, msymbol),
+                          (struct obj_section *) 0, 0);
+  sal.section = MSYMBOL_OBJ_SECTION (objfile, msymbol);
+
+  /* The minimal symbol might point to a function descriptor;
+     resolve it to the actual code address instead.  */
+  pc = gdbarch_convert_from_func_ptr_addr (gdbarch, sal.pc, &current_target);
+  if (pc != sal.pc)
+    sal = find_pc_sect_line (pc, NULL, 0);
+
+  if (self->funfirstline)
     {
-      block = get_selected_block (0);
-      for (;
-          block && !BLOCK_FUNCTION (block);
-          block = BLOCK_SUPERBLOCK (block))
-       ;
-      if (!block)
-       return 0;
-      function_symbol = BLOCK_FUNCTION (block);
+      if (sal.symtab != NULL
+         && (COMPUNIT_LOCATIONS_VALID (SYMTAB_COMPUNIT (sal.symtab))
+             || SYMTAB_LANGUAGE (sal.symtab) == language_asm))
+       {
+         /* If gdbarch_convert_from_func_ptr_addr does not apply then
+            sal.SECTION, sal.LINE&co. will stay correct from above.
+            If gdbarch_convert_from_func_ptr_addr applies then
+            sal.SECTION is cleared from above and sal.LINE&co. will
+            stay correct from the last find_pc_sect_line above.  */
+         sal.pc = MSYMBOL_VALUE_ADDRESS (objfile, msymbol);
+         sal.pc = gdbarch_convert_from_func_ptr_addr (gdbarch, sal.pc,
+                                                      &current_target);
+       }
+      else
+       skip_prologue_sal (&sal);
     }
 
-  sym = lookup_symbol (copy, block, LABEL_DOMAIN, 0);
+  if (maybe_add_address (self->addr_set, objfile->pspace, sal.pc))
+    add_sal_to_sals (self, result, &sal, MSYMBOL_NATURAL_NAME (msymbol), 0);
+}
+
+/* A helper struct to pass some data through
+   iterate_over_minimal_symbols.  */
+
+struct collect_minsyms
+{
+  /* The objfile we're examining.  */
+  struct objfile *objfile;
+
+  /* Only search the given symtab, or NULL to search for all symbols.  */
+  struct symtab *symtab;
 
-  if (sym != NULL)
-    *result = symbol_found (0, canonical, copy, sym, NULL, function_symbol);
+  /* The funfirstline setting from the initial call.  */
+  int funfirstline;
 
-  return sym != NULL;
+  /* The list_mode setting from the initial call.  */
+  int list_mode;
+
+  /* The resulting symbols.  */
+  VEC (bound_minimal_symbol_d) *msyms;
+};
+
+/* A helper function to classify a minimal_symbol_type according to
+   priority.  */
+
+static int
+classify_mtype (enum minimal_symbol_type t)
+{
+  switch (t)
+    {
+    case mst_file_text:
+    case mst_file_data:
+    case mst_file_bss:
+      /* Intermediate priority.  */
+      return 1;
+
+    case mst_solib_trampoline:
+      /* Lowest priority.  */
+      return 2;
+
+    default:
+      /* Highest priority.  */
+      return 0;
+    }
 }
 
-/* Decode a linespec that's a variable.  If FILE_SYMTAB is non-NULL,
-   look in that symtab's static variables first.  */ 
+/* Callback for qsort that sorts symbols by priority.  */
 
-static struct symtabs_and_lines
-decode_variable (char *copy, int funfirstline,
-                struct linespec_result *canonical,
-                struct symtab *file_symtab)
+static int
+compare_msyms (const void *a, const void *b)
 {
-  struct symbol *sym;
-  struct minimal_symbol *msymbol;
+  const bound_minimal_symbol_d *moa = a;
+  const bound_minimal_symbol_d *mob = b;
+  enum minimal_symbol_type ta = MSYMBOL_TYPE (moa->minsym);
+  enum minimal_symbol_type tb = MSYMBOL_TYPE (mob->minsym);
+
+  return classify_mtype (ta) - classify_mtype (tb);
+}
 
-  sym = lookup_symbol (copy,
-                      (file_symtab
-                       ? BLOCKVECTOR_BLOCK (BLOCKVECTOR (file_symtab),
-                                            STATIC_BLOCK)
-                       : get_selected_block (0)),
-                      VAR_DOMAIN, 0);
+/* Callback for iterate_over_minimal_symbols that adds the symbol to
+   the result.  */
 
-  if (sym != NULL)
-    return symbol_found (funfirstline, canonical, copy, sym, file_symtab, NULL);
+static void
+add_minsym (struct minimal_symbol *minsym, void *d)
+{
+  struct collect_minsyms *info = d;
+  bound_minimal_symbol_d mo;
 
-  msymbol = lookup_minimal_symbol (copy, NULL, NULL);
+  mo.minsym = minsym;
+  mo.objfile = info->objfile;
 
-  if (msymbol != NULL)
-    return minsym_found (funfirstline, msymbol);
+  if (info->symtab != NULL)
+    {
+      CORE_ADDR pc;
+      struct symtab_and_line sal;
+      struct gdbarch *gdbarch = get_objfile_arch (info->objfile);
+
+      sal = find_pc_sect_line (MSYMBOL_VALUE_ADDRESS (info->objfile, minsym),
+                              NULL, 0);
+      sal.section = MSYMBOL_OBJ_SECTION (info->objfile, minsym);
+      pc
+       = gdbarch_convert_from_func_ptr_addr (gdbarch, sal.pc, &current_target);
+      if (pc != sal.pc)
+       sal = find_pc_sect_line (pc, NULL, 0);
+
+      if (info->symtab != sal.symtab)
+       return;
+    }
 
-  if (!have_full_symbols ()
-      && !have_partial_symbols ()
-      && !have_minimal_symbols ())
-    throw_error (NOT_FOUND_ERROR,
-                _("No symbol table is loaded.  Use the \"file\" command."));
-  throw_error (NOT_FOUND_ERROR, _("Function \"%s\" not defined."), copy);
+  /* Exclude data symbols when looking for breakpoint locations.   */
+  if (!info->list_mode)
+    switch (minsym->type)
+      {
+       case mst_slot_got_plt:
+       case mst_data:
+       case mst_bss:
+       case mst_abs:
+       case mst_file_data:
+       case mst_file_bss:
+         {
+           /* Make sure this minsym is not a function descriptor
+              before we decide to discard it.  */
+           struct gdbarch *gdbarch = get_objfile_arch (info->objfile);
+           CORE_ADDR addr = gdbarch_convert_from_func_ptr_addr
+                              (gdbarch, BMSYMBOL_VALUE_ADDRESS (mo),
+                               &current_target);
+
+           if (addr == BMSYMBOL_VALUE_ADDRESS (mo))
+             return;
+         }
+      }
+
+  VEC_safe_push (bound_minimal_symbol_d, info->msyms, &mo);
 }
 
+/* Search for minimal symbols called NAME.  If SEARCH_PSPACE
+   is not NULL, the search is restricted to just that program
+   space.
 
-\f
+   If SYMTAB is NULL, search all objfiles, otherwise
+   restrict results to the given SYMTAB.  */
 
-/* Now come some functions that are called from multiple places within
-   decode_line_1.  */
+static void
+search_minsyms_for_name (struct collect_info *info, const char *name,
+                        struct program_space *search_pspace,
+                        struct symtab *symtab)
+{
+  struct collect_minsyms local;
+  struct cleanup *cleanup;
 
-/* We've found a symbol SYM to associate with our linespec; build a
-   corresponding struct symtabs_and_lines.  */
+  memset (&local, 0, sizeof (local));
+  local.funfirstline = info->state->funfirstline;
+  local.list_mode = info->state->list_mode;
+  local.symtab = symtab;
 
-static struct symtabs_and_lines
-symbol_found (int funfirstline, struct linespec_result *canonical, char *copy,
-             struct symbol *sym, struct symtab *file_symtab,
-             struct symbol *function_symbol)
-{
-  struct symtabs_and_lines values;
-  
-  if (SYMBOL_CLASS (sym) == LOC_BLOCK)
+  cleanup = make_cleanup (VEC_cleanup (bound_minimal_symbol_d), &local.msyms);
+
+  if (symtab == NULL)
     {
-      /* Arg is the name of a function.  */
-      values.sals = (struct symtab_and_line *)
-       xmalloc (sizeof (struct symtab_and_line));
-      values.sals[0] = find_function_start_sal (sym, funfirstline);
-      values.nelts = 1;
+      struct program_space *pspace;
 
-      /* Don't use the SYMBOL_LINE; if used at all it points to
-        the line containing the parameters or thereabouts, not
-        the first line of code.  */
+      ALL_PSPACES (pspace)
+      {
+       struct objfile *objfile;
+
+       if (search_pspace != NULL && search_pspace != pspace)
+         continue;
+       if (pspace->executing_startup)
+         continue;
 
-      /* We might need a canonical line spec if it is a static
-        function.  */
-      if (file_symtab == 0)
+       set_current_program_space (pspace);
+
+       ALL_OBJFILES (objfile)
+       {
+         local.objfile = objfile;
+         iterate_over_minimal_symbols (objfile, name, add_minsym, &local);
+       }
+      }
+    }
+  else
+    {
+      if (search_pspace == NULL || SYMTAB_PSPACE (symtab) == search_pspace)
        {
-         struct blockvector *bv = BLOCKVECTOR (SYMBOL_SYMTAB (sym));
-         struct block *b = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK);
+         set_current_program_space (SYMTAB_PSPACE (symtab));
+         local.objfile = SYMTAB_OBJFILE(symtab);
+         iterate_over_minimal_symbols (local.objfile, name, add_minsym,
+                                       &local);
+       }
+    }
+
+    if (!VEC_empty (bound_minimal_symbol_d, local.msyms))
+      {
+       int classification;
+       int ix;
+       bound_minimal_symbol_d *item;
+
+       qsort (VEC_address (bound_minimal_symbol_d, local.msyms),
+              VEC_length (bound_minimal_symbol_d, local.msyms),
+              sizeof (bound_minimal_symbol_d),
+              compare_msyms);
+
+       /* Now the minsyms are in classification order.  So, we walk
+          over them and process just the minsyms with the same
+          classification as the very first minsym in the list.  */
+       item = VEC_index (bound_minimal_symbol_d, local.msyms, 0);
+       classification = classify_mtype (MSYMBOL_TYPE (item->minsym));
+
+       for (ix = 0;
+            VEC_iterate (bound_minimal_symbol_d, local.msyms, ix, item);
+            ++ix)
+         {
+           if (classify_mtype (MSYMBOL_TYPE (item->minsym)) != classification)
+             break;
+
+           VEC_safe_push (bound_minimal_symbol_d,
+                          info->result.minimal_symbols, item);
+         }
+      }
 
-         if (lookup_block_symbol (b, copy, VAR_DOMAIN) != NULL)
-           build_canonical_line_spec (values.sals, copy, canonical);
+    do_cleanups (cleanup);
+}
+
+/* A helper function to add all symbols matching NAME to INFO.  If
+   PSPACE is not NULL, the search is restricted to just that program
+   space.  */
+
+static void
+add_matching_symbols_to_info (const char *name,
+                             struct collect_info *info,
+                             struct program_space *pspace)
+{
+  int ix;
+  struct symtab *elt;
+
+  for (ix = 0; VEC_iterate (symtab_ptr, info->file_symtabs, ix, elt); ++ix)
+    {
+      if (elt == NULL)
+       {
+         iterate_over_all_matching_symtabs (info->state, name, VAR_DOMAIN,
+                                            collect_symbols, info,
+                                            pspace, 1);
+         search_minsyms_for_name (info, name, pspace, NULL);
        }
-      return values;
+      else if (pspace == NULL || pspace == SYMTAB_PSPACE (elt))
+       {
+         int prev_len = VEC_length (symbolp, info->result.symbols);
+
+         /* Program spaces that are executing startup should have
+            been filtered out earlier.  */
+         gdb_assert (!SYMTAB_PSPACE (elt)->executing_startup);
+         set_current_program_space (SYMTAB_PSPACE (elt));
+         iterate_over_file_blocks (elt, name, VAR_DOMAIN,
+                                   collect_symbols, info);
+
+         /* If no new symbols were found in this iteration and this symtab
+            is in assembler, we might actually be looking for a label for
+            which we don't have debug info.  Check for a minimal symbol in
+            this case.  */
+         if (prev_len == VEC_length (symbolp, info->result.symbols)
+             && elt->language == language_asm)
+           search_minsyms_for_name (info, name, pspace, elt);
+       }
+    }
+}
+
+\f
+
+/* Now come some functions that are called from multiple places within
+   decode_line_1.  */
+
+static int
+symbol_to_sal (struct symtab_and_line *result,
+              int funfirstline, struct symbol *sym)
+{
+  if (SYMBOL_CLASS (sym) == LOC_BLOCK)
+    {
+      *result = find_function_start_sal (sym, funfirstline);
+      return 1;
     }
   else
     {
       if (SYMBOL_CLASS (sym) == LOC_LABEL && SYMBOL_VALUE_ADDRESS (sym) != 0)
        {
-         /* We know its line number.  */
-         values.sals = (struct symtab_and_line *)
-           xmalloc (sizeof (struct symtab_and_line));
-         values.nelts = 1;
-         init_sal (&values.sals[0]);
-         values.sals[0].symtab = SYMBOL_SYMTAB (sym);
-         values.sals[0].line = SYMBOL_LINE (sym);
-         values.sals[0].pc = SYMBOL_VALUE_ADDRESS (sym);
-         values.sals[0].pspace = SYMTAB_PSPACE (SYMBOL_SYMTAB (sym));
-         values.sals[0].explicit_pc = 1;
-
-         if (canonical)
-           {
-             canonical->special_display = 1;
-             canonical->canonical = xmalloc (sizeof (char *));
-             canonical->canonical[0]
-               = xstrprintf ("%s:%s",
-                             SYMBOL_NATURAL_NAME (function_symbol),
-                             SYMBOL_NATURAL_NAME (sym));
-           }
-
-         return values;
+         init_sal (result);
+         result->symtab = symbol_symtab (sym);
+         result->line = SYMBOL_LINE (sym);
+         result->pc = SYMBOL_VALUE_ADDRESS (sym);
+         result->pspace = SYMTAB_PSPACE (result->symtab);
+         result->explicit_pc = 1;
+         return 1;
        }
       else if (funfirstline)
-       error (_("\"%s\" is not a function"), copy);
+       {
+         /* Nothing.  */
+       }
       else if (SYMBOL_LINE (sym) != 0)
        {
          /* We know its line number.  */
-         values.sals = (struct symtab_and_line *)
-           xmalloc (sizeof (struct symtab_and_line));
-         values.nelts = 1;
-         memset (&values.sals[0], 0, sizeof (values.sals[0]));
-         values.sals[0].symtab = SYMBOL_SYMTAB (sym);
-         values.sals[0].line = SYMBOL_LINE (sym);
-         values.sals[0].pspace = SYMTAB_PSPACE (SYMBOL_SYMTAB (sym));
-         return values;
+         init_sal (result);
+         result->symtab = symbol_symtab (sym);
+         result->line = SYMBOL_LINE (sym);
+         result->pspace = SYMTAB_PSPACE (result->symtab);
+         return 1;
        }
-      else
-       /* This can happen if it is compiled with a compiler which doesn't
-          put out line numbers for variables.  */
-       /* FIXME: Shouldn't we just set .line and .symtab to zero
-          and return?  For example, "info line foo" could print
-          the address.  */
-       error (_("Line number not known for symbol \"%s\""), copy);
     }
+
+  return 0;
 }
 
-/* We've found a minimal symbol MSYMBOL to associate with our
-   linespec; build a corresponding struct symtabs_and_lines.  */
+/* See the comment in linespec.h.  */
 
-static struct symtabs_and_lines
-minsym_found (int funfirstline, struct minimal_symbol *msymbol)
+void
+init_linespec_result (struct linespec_result *lr)
 {
-  struct objfile *objfile = msymbol_objfile (msymbol);
-  struct gdbarch *gdbarch = get_objfile_arch (objfile);
-  struct symtabs_and_lines values;
-  CORE_ADDR pc;
+  memset (lr, 0, sizeof (*lr));
+}
 
-  values.sals = (struct symtab_and_line *)
-    xmalloc (sizeof (struct symtab_and_line));
-  values.sals[0] = find_pc_sect_line (SYMBOL_VALUE_ADDRESS (msymbol),
-                                     (struct obj_section *) 0, 0);
-  values.sals[0].section = SYMBOL_OBJ_SECTION (msymbol);
+/* See the comment in linespec.h.  */
 
-  /* The minimal symbol might point to a function descriptor;
-     resolve it to the actual code address instead.  */
-  pc = gdbarch_convert_from_func_ptr_addr (gdbarch,
-                                           values.sals[0].pc,
-                                           &current_target);
-  if (pc != values.sals[0].pc)
-    values.sals[0] = find_pc_sect_line (pc, NULL, 0);
+void
+destroy_linespec_result (struct linespec_result *ls)
+{
+  int i;
+  struct linespec_sals *lsal;
+
+  delete_event_location (ls->location);
+  for (i = 0; VEC_iterate (linespec_sals, ls->sals, i, lsal); ++i)
+    {
+      xfree (lsal->canonical);
+      xfree (lsal->sals.sals);
+    }
+  VEC_free (linespec_sals, ls->sals);
+}
 
-  if (funfirstline)
-    skip_prologue_sal (&values.sals[0]);
+/* Cleanup function for a linespec_result.  */
 
-  values.nelts = 1;
-  return values;
+static void
+cleanup_linespec_result (void *a)
+{
+  destroy_linespec_result (a);
 }
 
-void
-init_linespec_result (struct linespec_result *lr)
+/* See the comment in linespec.h.  */
+
+struct cleanup *
+make_cleanup_destroy_linespec_result (struct linespec_result *ls)
 {
-  memset (lr, 0, sizeof (*lr));
+  return make_cleanup (cleanup_linespec_result, ls);
 }
This page took 0.170548 seconds and 4 git commands to generate.