Replace free() with xfree().
[deliverable/binutils-gdb.git] / gdb / source.c
index 3869e1a169a96249f5844e22e9ed74f99542c51e..3d639b9e1a9bec853321d75c02115cf908b38dbf 100644 (file)
@@ -1,79 +1,90 @@
 /* List lines of source files for GDB, the GNU debugger.
-   Copyright (C) 1986, 1987, 1988, 1989, 1991 Free Software Foundation, Inc.
+   Copyright 1986-1989, 1991-1999 Free Software Foundation, Inc.
 
-This file is part of GDB.
+   This file is part of GDB.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.  */
 
 #include "defs.h"
 #include "symtab.h"
 #include "expression.h"
 #include "language.h"
 #include "command.h"
+#include "source.h"
 #include "gdbcmd.h"
 #include "frame.h"
+#include "value.h"
 
-#ifdef USG
 #include <sys/types.h>
-#endif
-
-#include <string.h>
-#include <sys/param.h>
-#include <sys/stat.h>
+#include "gdb_string.h"
+#include "gdb_stat.h"
 #include <fcntl.h>
 #include "gdbcore.h"
-#include "regex.h"
+#include "gdb_regex.h"
 #include "symfile.h"
 #include "objfiles.h"
+#include "annotate.h"
+#include "gdbtypes.h"
+#include "linespec.h"
+#include "completer.h"
+#ifdef UI_OUT
+#include "ui-out.h"
+#endif
 
-/* Prototypes for local functions. */
+#ifdef CRLF_SOURCE_FILES
 
-static int
-open_source_file PARAMS ((struct symtab *));
+/* Define CRLF_SOURCE_FILES in an xm-*.h file if source files on the
+   host use \r\n rather than just \n.  Defining CRLF_SOURCE_FILES is
+   much faster than defining LSEEK_NOT_LINEAR.  */
 
-static int
-get_filename_and_charpos PARAMS ((struct symtab *, char **));
+#ifndef O_BINARY
+#define O_BINARY 0
+#endif
 
-static void
-reverse_search_command PARAMS ((char *, int));
+#define OPEN_MODE (O_RDONLY | O_BINARY)
+#define FDOPEN_MODE FOPEN_RB
 
-static void
-forward_search_command PARAMS ((char *, int));
+#else /* ! defined (CRLF_SOURCE_FILES) */
 
-static void
-line_info PARAMS ((char *, int));
+#define OPEN_MODE O_RDONLY
+#define FDOPEN_MODE FOPEN_RT
 
-static void
-list_command PARAMS ((char *, int));
+#endif /* ! defined (CRLF_SOURCE_FILES) */
 
-static void
-ambiguous_line_spec PARAMS ((struct symtabs_and_lines *));
+/* Prototypes for exported functions. */
 
-static void
-source_info PARAMS ((char *, int));
+void _initialize_source (void);
 
-static void
-show_directories PARAMS ((char *, int));
+/* Prototypes for local functions. */
 
-static void
-find_source_lines PARAMS ((struct symtab *, int));
+static int get_filename_and_charpos (struct symtab *, char **);
+
+static void reverse_search_command (char *, int);
+
+static void forward_search_command (char *, int);
+
+static void line_info (char *, int);
+
+static void list_command (char *, int);
+
+static void ambiguous_line_spec (struct symtabs_and_lines *);
 
-/* If we use this declaration, it breaks because of fucking ANSI "const" stuff
-   on some systems.  We just have to not declare it at all, have it default
-   to int, and possibly botch on a few systems.  Thanks, ANSIholes... */
-/* extern char *strstr(); */
+static void source_info (char *, int);
+
+static void show_directories (char *, int);
 
 /* Path of directories to search for source files.
    Same format as the PATH environment variable's value.  */
@@ -105,25 +116,30 @@ static int last_line_listed;
 
 static int first_line_listed;
 
+/* Saves the name of the last source file visited and a possible error code.
+   Used to prevent repeating annoying "No such file or directories" msgs */
+
+static struct symtab *last_source_visited = NULL;
+static int last_source_error = 0;
 \f
-/* Set the source file default for the "list" command, specifying a
-   symtab.  Sigh.  Behavior specification: If it is called with a
-   non-zero argument, that is the symtab to select.  If it is not,
-   first lookup "main"; if it exists, use the symtab and line it
-   defines.  If not, take the last symtab in the symtab lists (if it
-   exists) or the last symtab in the psymtab lists (if *it* exists).  If
-   none of this works, report an error.   */
+
+/* Set the source file default for the "list" command to be S.
+
+   If S is NULL, and we don't have a default, find one.  This
+   should only be called when the user actually tries to use the
+   default, since we produce an error if we can't find a reasonable
+   default.  Also, since this can cause symbols to be read, doing it
+   before we need to would make things slower than necessary.  */
 
 void
-select_source_symtab (s)
-     register struct symtab *s;
+select_source_symtab (register struct symtab *s)
 {
   struct symtabs_and_lines sals;
   struct symtab_and_line sal;
   struct partial_symtab *ps;
   struct partial_symtab *cs_pst = 0;
   struct objfile *ofp;
-  
+
   if (s)
     {
       current_source_symtab = s;
@@ -131,30 +147,33 @@ select_source_symtab (s)
       return;
     }
 
+  if (current_source_symtab)
+    return;
+
   /* Make the default place to list be the function `main'
      if one exists.  */
   if (lookup_symbol ("main", 0, VAR_NAMESPACE, 0, NULL))
     {
       sals = decode_line_spec ("main", 1);
       sal = sals.sals[0];
-      free (sals.sals);
+      xfree (sals.sals);
       current_source_symtab = sal.symtab;
       current_source_line = max (sal.line - (lines_to_list - 1), 1);
       if (current_source_symtab)
-        return;
+       return;
     }
-  
+
   /* All right; find the last file in the symtab list (ignoring .h's).  */
 
   current_source_line = 1;
 
-  for (ofp = object_files; ofp != NULL; ofp = ofp -> next)
+  for (ofp = object_files; ofp != NULL; ofp = ofp->next)
     {
-      for (s = ofp -> symtabs; s; s = s->next)
+      for (s = ofp->symtabs; s; s = s->next)
        {
-         char *name = s -> filename;
+         char *name = s->filename;
          int len = strlen (name);
-         if (! (len > 2 && (strcmp (&name[len - 2], ".h") == 0)))
+         if (!(len > 2 && (STREQ (&name[len - 2], ".h"))))
            {
              current_source_symtab = s;
            }
@@ -165,13 +184,13 @@ select_source_symtab (s)
 
   /* Howabout the partial symbol tables? */
 
-  for (ofp = object_files; ofp != NULL; ofp = ofp -> next)
+  for (ofp = object_files; ofp != NULL; ofp = ofp->next)
     {
-      for (ps = ofp -> psymtabs; ps != NULL; ps = ps -> next)
+      for (ps = ofp->psymtabs; ps != NULL; ps = ps->next)
        {
-         char *name = ps -> filename;
+         char *name = ps->filename;
          int len = strlen (name);
-         if (! (len > 2 && (strcmp (&name[len - 2], ".h") == 0)))
+         if (!(len > 2 && (STREQ (&name[len - 2], ".h"))))
            {
              cs_pst = ps;
            }
@@ -179,16 +198,15 @@ select_source_symtab (s)
     }
   if (cs_pst)
     {
-      if (cs_pst -> readin)
+      if (cs_pst->readin)
        {
-         fatal ("Internal: select_source_symtab: readin pst found and no symtabs.");
+         internal_error ("select_source_symtab: readin pst found and no symtabs.");
        }
       else
        {
          current_source_symtab = PSYMTAB_TO_SYMTAB (cs_pst);
        }
     }
-
   if (current_source_symtab)
     return;
 
@@ -196,81 +214,80 @@ select_source_symtab (s)
 }
 \f
 static void
-show_directories (ignore, from_tty)
-     char *ignore;
-     int from_tty;
+show_directories (char *ignore, int from_tty)
 {
   puts_filtered ("Source directories searched: ");
   puts_filtered (source_path);
   puts_filtered ("\n");
 }
 
-/* Forget what we learned about line positions in source files,
-   and which directories contain them;
-   must check again now since files may be found in
-   a different directory now.  */
+/* Forget what we learned about line positions in source files, and
+   which directories contain them; must check again now since files
+   may be found in a different directory now.  */
 
 void
-forget_cached_source_info ()
+forget_cached_source_info (void)
 {
   register struct symtab *s;
   register struct objfile *objfile;
 
-  for (objfile = object_files; objfile != NULL; objfile = objfile -> next)
+  for (objfile = object_files; objfile != NULL; objfile = objfile->next)
     {
-      for (s = objfile -> symtabs; s != NULL; s = s -> next)
+      for (s = objfile->symtabs; s != NULL; s = s->next)
        {
-         if (s -> line_charpos != NULL)
+         if (s->line_charpos != NULL)
            {
-             mfree (objfile -> md, s -> line_charpos);
-             s -> line_charpos = NULL;
+             mfree (objfile->md, s->line_charpos);
+             s->line_charpos = NULL;
            }
-         if (s -> fullname != NULL)
+         if (s->fullname != NULL)
            {
-             mfree (objfile -> md, s -> fullname);
-             s -> fullname = NULL;
+             mfree (objfile->md, s->fullname);
+             s->fullname = NULL;
            }
        }
     }
 }
 
 void
-init_source_path ()
+init_source_path (void)
 {
-  source_path = savestring ("$cdir:$cwd", /* strlen of it */ 10);
+  char buf[20];
+
+  sprintf (buf, "$cdir%c$cwd", DIRNAME_SEPARATOR);
+  source_path = strsave (buf);
   forget_cached_source_info ();
 }
 
 /* Add zero or more directories to the front of the source path.  */
+
 void
-directory_command (dirname, from_tty)
-     char *dirname;
-     int from_tty;
+directory_command (char *dirname, int from_tty)
 {
   dont_repeat ();
   /* FIXME, this goes to "delete dir"... */
   if (dirname == 0)
     {
-      if (query ("Reinitialize source path to empty? ", ""))
+      if (from_tty && query ("Reinitialize source path to empty? "))
        {
-         free (source_path);
+         xfree (source_path);
          init_source_path ();
        }
     }
   else
-    mod_path (dirname, &source_path);
+    {
+      mod_path (dirname, &source_path);
+      last_source_visited = NULL;
+    }
   if (from_tty)
-    show_directories ((char *)0, from_tty);
+    show_directories ((char *) 0, from_tty);
   forget_cached_source_info ();
 }
 
 /* Add zero or more directories to the front of an arbitrary path.  */
 
 void
-mod_path (dirname, which_path)
-     char *dirname;
-     char **which_path;
+mod_path (char *dirname, char **which_path)
 {
   char *old = *which_path;
   int prefix = 0;
@@ -279,7 +296,7 @@ mod_path (dirname, which_path)
     return;
 
   dirname = strsave (dirname);
-  make_cleanup (free, dirname);
+  make_cleanup (xfree, dirname);
 
   do
     {
@@ -288,32 +305,40 @@ mod_path (dirname, which_path)
       struct stat st;
 
       {
-       char *colon = strchr (name, ':');
+       char *separator = strchr (name, DIRNAME_SEPARATOR);
        char *space = strchr (name, ' ');
        char *tab = strchr (name, '\t');
-       if (colon == 0 && space == 0 && tab ==  0)
+
+       if (separator == 0 && space == 0 && tab == 0)
          p = dirname = name + strlen (name);
        else
          {
            p = 0;
-           if (colon != 0 && (p == 0 || colon < p))
-             p = colon;
+           if (separator != 0 && (p == 0 || separator < p))
+             p = separator;
            if (space != 0 && (p == 0 || space < p))
              p = space;
            if (tab != 0 && (p == 0 || tab < p))
              p = tab;
            dirname = p + 1;
-           while (*dirname == ':' || *dirname == ' ' || *dirname == '\t')
+           while (*dirname == DIRNAME_SEPARATOR
+                  || *dirname == ' '
+                  || *dirname == '\t')
              ++dirname;
          }
       }
 
-      if (p[-1] == '/')
+      if (!(SLASH_P (*name) && p <= name + 1)  /* "/" */
+#if defined(_WIN32) || defined(__MSDOS__)
+      /* On MS-DOS and MS-Windows, h:\ is different from h: */
+         && !(!SLASH_P (*name) && ROOTED_P (name) && p <= name + 3)    /* d:/ */
+#endif
+         && SLASH_P (p[-1]))
        /* Sigh. "foo/" => "foo" */
        --p;
       *p = '\0';
 
-      while (p[-1] == '.')
+      while (p > name && p[-1] == '.')
        {
          if (p - name == 1)
            {
@@ -321,7 +346,7 @@ mod_path (dirname, which_path)
              name = current_directory;
              goto append;
            }
-         else if (p[-2] == '/')
+         else if (p > name + 1 && SLASH_P (p[-2]))
            {
              if (p - name == 2)
                {
@@ -343,19 +368,37 @@ mod_path (dirname, which_path)
 
       if (name[0] == '~')
        name = tilde_expand (name);
-      else if (name[0] != '/' && name[0] != '$')
-       name = concat (current_directory, "/", name, NULL);
+#if defined(_WIN32) || defined(__MSDOS__)
+      else if (ROOTED_P (name) && p == name + 2)       /* "d:" => "d:." */
+       name = concat (name, ".", NULL);
+#endif
+      else if (!ROOTED_P (name) && name[0] != '$')
+       name = concat (current_directory, SLASH_STRING, name, NULL);
       else
        name = savestring (name, p - name);
-      make_cleanup (free, name);
+      make_cleanup (xfree, name);
 
       /* Unless it's a variable, check existence.  */
-      if (name[0] != '$') {
-       if (stat (name, &st) < 0)
-         perror_with_name (name);
-       if ((st.st_mode & S_IFMT) != S_IFDIR)
-         error ("%s is not a directory.", name);
-      }
+      if (name[0] != '$')
+       {
+         /* These are warnings, not errors, since we don't want a
+            non-existent directory in a .gdbinit file to stop processing
+            of the .gdbinit file.
+
+            Whether they get added to the path is more debatable.  Current
+            answer is yes, in case the user wants to go make the directory
+            or whatever.  If the directory continues to not exist/not be
+            a directory/etc, then having them in the path should be
+            harmless.  */
+         if (stat (name, &st) < 0)
+           {
+             int save_errno = errno;
+             fprintf_unfiltered (gdb_stderr, "Warning: ");
+             print_sys_errmsg (name, save_errno);
+           }
+         else if ((st.st_mode & S_IFMT) != S_IFDIR)
+           warning ("%s is not a directory.", name);
+       }
 
     append:
       {
@@ -364,17 +407,25 @@ mod_path (dirname, which_path)
        p = *which_path;
        while (1)
          {
+           /* FIXME: strncmp loses in interesting ways on MS-DOS and
+              MS-Windows because of case-insensitivity and two different
+              but functionally identical slash characters.  We need a
+              special filesystem-dependent file-name comparison function.
+
+              Actually, even on Unix I would use realpath() or its work-
+              alike before comparing.  Then all the code above which
+              removes excess slashes and dots could simply go away.  */
            if (!strncmp (p, name, len)
-               && (p[len] == '\0' || p[len] == ':'))
+               && (p[len] == '\0' || p[len] == DIRNAME_SEPARATOR))
              {
                /* Found it in the search path, remove old copy */
                if (p > *which_path)
-                 p--;                  /* Back over leading colon */
+                 p--;          /* Back over leading separator */
                if (prefix > p - *which_path)
                  goto skip_dup;        /* Same dir twice in one cmd */
-               strcpy (p, &p[len+1]);  /* Copy from next \0 or  : */
+               strcpy (p, &p[len + 1]);        /* Copy from next \0 or  : */
              }
-           p = strchr (p, ':');
+           p = strchr (p, DIRNAME_SEPARATOR);
            if (p != 0)
              ++p;
            else
@@ -382,43 +433,47 @@ mod_path (dirname, which_path)
          }
        if (p == 0)
          {
-           /* If we have already tacked on a name(s) in this command,                     be sure they stay on the front as we tack on some more.  */
+           char tinybuf[2];
+
+           tinybuf[0] = DIRNAME_SEPARATOR;
+           tinybuf[1] = '\0';
+
+           /* If we have already tacked on a name(s) in this command,                     be sure they stay on the front as we tack on some more.  */
            if (prefix)
              {
                char *temp, c;
 
                c = old[prefix];
                old[prefix] = '\0';
-               temp = concat (old, ":", name, NULL);
+               temp = concat (old, tinybuf, name, NULL);
                old[prefix] = c;
                *which_path = concat (temp, "", &old[prefix], NULL);
                prefix = strlen (temp);
-               free (temp);
+               xfree (temp);
              }
            else
              {
-               *which_path = concat (name, (old[0]? ":" : old), old, NULL);
+               *which_path = concat (name, (old[0] ? tinybuf : old), old, NULL);
                prefix = strlen (name);
              }
-           free (old);
+           xfree (old);
            old = *which_path;
          }
       }
-  skip_dup: ;
-    } while (*dirname != '\0');
+    skip_dup:;
+    }
+  while (*dirname != '\0');
 }
 
 
 static void
-source_info (ignore, from_tty)
-     char *ignore;
-     int from_tty;
+source_info (char *ignore, int from_tty)
 {
   register struct symtab *s = current_source_symtab;
 
   if (!s)
     {
-      printf_filtered("No current source file.\n");
+      printf_filtered ("No current source file.\n");
       return;
     }
   printf_filtered ("Current source file is %s\n", s->filename);
@@ -427,17 +482,24 @@ source_info (ignore, from_tty)
   if (s->fullname)
     printf_filtered ("Located in %s\n", s->fullname);
   if (s->nlines)
-    printf_filtered ("Contains %d lines\n", s->nlines);
+    printf_filtered ("Contains %d line%s.\n", s->nlines,
+                    s->nlines == 1 ? "" : "s");
 
-  printf_filtered("Source language %s.\n", language_str (s->language));
+  printf_filtered ("Source language is %s.\n", language_str (s->language));
+  printf_filtered ("Compiled with %s debugging format.\n", s->debugformat);
 }
+\f
 
 
-\f
-/* Open a file named STRING, searching path PATH (dir names sep by colons)
+/* Open a file named STRING, searching path PATH (dir names sep by some char)
    using mode MODE and protection bits PROT in the calls to open.
+
    If TRY_CWD_FIRST, try to open ./STRING before searching PATH.
-   (ie pretend the first element of PATH is ".")
+   (ie pretend the first element of PATH is ".").  This also indicates
+   that a slash in STRING disables searching of the path (this is
+   so that "exec-file ./foo" or "symbol-file ./foo" insures that you
+   get that particular version of foo or an error message).
+
    If FILENAMED_OPENED is non-null, set it to a newly allocated string naming
    the actual file opened (this string will always start with a "/".  We
    have to take special pains to avoid doubling the "/" between the directory
@@ -448,15 +510,10 @@ source_info (ignore, from_tty)
    Otherwise, return -1, with errno set for the last name we tried to open.  */
 
 /*  >>>> This should only allow files of certain types,
-    >>>>  eg executable, non-directory */
+   >>>>  eg executable, non-directory */
 int
-openp (path, try_cwd_first, string, mode, prot, filename_opened)
-     char *path;
-     int try_cwd_first;
-     char *string;
-     int mode;
-     int prot;
-     char **filename_opened;
+openp (char *path, int try_cwd_first, char *string, int mode, int prot,
+       char **filename_opened)
 {
   register int fd;
   register char *filename;
@@ -467,83 +524,144 @@ openp (path, try_cwd_first, string, mode, prot, filename_opened)
   if (!path)
     path = ".";
 
-  /* ./foo => foo */
-  while (string[0] == '.' && string[1] == '/')
-    string += 2;
+#ifdef _WIN32
+  mode |= O_BINARY;
+#endif
 
-  if (try_cwd_first || string[0] == '/')
+  if (try_cwd_first || ROOTED_P (string))
     {
+      int i;
       filename = string;
       fd = open (filename, mode, prot);
-      if (fd >= 0 || string[0] == '/')
+      if (fd >= 0)
        goto done;
+      for (i = 0; string[i]; i++)
+       if (SLASH_P (string[i]))
+         goto done;
     }
 
+  /* ./foo => foo */
+  while (string[0] == '.' && SLASH_P (string[1]))
+    string += 2;
+
   alloclen = strlen (path) + strlen (string) + 2;
   filename = (char *) alloca (alloclen);
   fd = -1;
   for (p = path; p; p = p1 ? p1 + 1 : 0)
     {
-      p1 = (char *) strchr (p, ':');
+      p1 = (char *) strchr (p, DIRNAME_SEPARATOR);
       if (p1)
        len = p1 - p;
       else
        len = strlen (p);
 
       if (len == 4 && p[0] == '$' && p[1] == 'c'
-                  && p[2] == 'w' && p[3] == 'd') {
-       /* Name is $cwd -- insert current directory name instead.  */
-       int newlen;
-
-       /* First, realloc the filename buffer if too short. */
-       len = strlen (current_directory);
-       newlen = len + strlen (string) + 2;
-       if (newlen > alloclen) {
-         alloclen = newlen;
-         filename = (char *) alloca (alloclen);
+         && p[2] == 'w' && p[3] == 'd')
+       {
+         /* Name is $cwd -- insert current directory name instead.  */
+         int newlen;
+
+         /* First, realloc the filename buffer if too short. */
+         len = strlen (current_directory);
+         newlen = len + strlen (string) + 2;
+         if (newlen > alloclen)
+           {
+             alloclen = newlen;
+             filename = (char *) alloca (alloclen);
+           }
+         strcpy (filename, current_directory);
+       }
+      else
+       {
+         /* Normal file name in path -- just use it.  */
+         strncpy (filename, p, len);
+         filename[len] = 0;
        }
-       strcpy (filename, current_directory);
-      } else {
-       /* Normal file name in path -- just use it.  */
-       strncpy (filename, p, len);
-       filename[len] = 0;
-      }
 
       /* Remove trailing slashes */
-      while (len > 0 && filename[len-1] == '/')
-       filename[--len] = 0;
+      while (len > 0 && SLASH_P (filename[len - 1]))
+       filename[--len] = 0;
 
-      strcat (filename+len, "/");
+      strcat (filename + len, SLASH_STRING);
       strcat (filename, string);
 
-      fd = open (filename, mode, prot);
-      if (fd >= 0) break;
+      fd = open (filename, mode);
+      if (fd >= 0)
+       break;
     }
 
- done:
+done:
   if (filename_opened)
-    if (fd < 0)
-      *filename_opened = (char *) 0;
-    else if (filename[0] == '/')
-      *filename_opened = savestring (filename, strlen (filename));
-    else
+    {
+      if (fd < 0)
+       *filename_opened = (char *) 0;
+      else if (ROOTED_P (filename))
+       *filename_opened = savestring (filename, strlen (filename));
+      else
+       {
+         /* Beware the // my son, the Emacs barfs, the botch that catch... */
+
+         *filename_opened = concat (current_directory,
+                SLASH_P (current_directory[strlen (current_directory) - 1])
+                                    ? "" : SLASH_STRING,
+                                    filename, NULL);
+       }
+    }
+#ifdef MPW
+  /* This is a debugging hack that can go away when all combinations
+     of Mac and Unix names are handled reasonably.  */
+  {
+    extern int debug_openp;
+
+    if (debug_openp)
       {
-       /* Beware the // my son, the Emacs barfs, the botch that catch... */
-          
-       *filename_opened = concat (current_directory, 
-          '/' == current_directory[strlen(current_directory)-1]? "": "/",
-                                  filename, NULL);
+       printf ("openp on %s, path %s mode %d prot %d\n  returned %d",
+               string, path, mode, prot, fd);
+       if (*filename_opened)
+         printf (" (filename is %s)", *filename_opened);
+       printf ("\n");
       }
+  }
+#endif /* MPW */
 
   return fd;
 }
 
-/* Open a source file given a symtab S.  Returns a file descriptor
-   or negative number for error.  */
 
-static int
-open_source_file (s)
-     struct symtab *s;
+/* This is essentially a convenience, for clients that want the behaviour
+   of openp, using source_path, but that really don't want the file to be
+   opened but want instead just to know what the full pathname is (as
+   qualified against source_path).
+
+   The current working directory is searched first.
+
+   If the file was found, this function returns 1, and FULL_PATHNAME is
+   set to the fully-qualified pathname.
+
+   Else, this functions returns 0, and FULL_PATHNAME is set to NULL.
+ */
+int
+source_full_path_of (char *filename, char **full_pathname)
+{
+  int fd;
+
+  fd = openp (source_path, 1, filename, O_RDONLY, 0, full_pathname);
+  if (fd < 0)
+    {
+      *full_pathname = NULL;
+      return 0;
+    }
+
+  close (fd);
+  return 1;
+}
+
+
+/* Open a source file given a symtab S.  Returns a file descriptor or
+   negative number for error.  */
+
+int
+open_source_file (struct symtab *s)
 {
   char *path = source_path;
   char *p;
@@ -551,11 +669,11 @@ open_source_file (s)
   char *fullname;
 
   /* Quick way out if we already know its full name */
-  if (s->fullname) 
+  if (s->fullname)
     {
-      result = open (s->fullname, O_RDONLY);
+      result = open (s->fullname, OPEN_MODE);
       if (result >= 0)
-        return result;
+       return result;
       /* Didn't work -- free old one, try again. */
       mfree (s->objfile->md, s->fullname);
       s->fullname = NULL;
@@ -566,142 +684,200 @@ open_source_file (s)
       /* Replace a path entry of  $cdir  with the compilation directory name */
 #define        cdir_len        5
       /* We cast strstr's result in case an ANSIhole has made it const,
-        which produces a "required warning" when assigned to a nonconst. */
-      p = (char *)strstr (source_path, "$cdir");
-      if (p && (p == path || p[-1] == ':')
-           && (p[cdir_len] == ':' || p[cdir_len] == '\0')) {
-       int len;
-
-       path = (char *)
-              alloca (strlen (source_path) + 1 + strlen (s->dirname) + 1);
-       len = p - source_path;
-       strncpy (path, source_path, len);               /* Before $cdir */
-       strcpy (path + len, s->dirname);                /* new stuff */
-       strcat (path + len, source_path + len + cdir_len); /* After $cdir */
-      }
+         which produces a "required warning" when assigned to a nonconst. */
+      p = (char *) strstr (source_path, "$cdir");
+      if (p && (p == path || p[-1] == DIRNAME_SEPARATOR)
+         && (p[cdir_len] == DIRNAME_SEPARATOR || p[cdir_len] == '\0'))
+       {
+         int len;
+
+         path = (char *)
+           alloca (strlen (source_path) + 1 + strlen (s->dirname) + 1);
+         len = p - source_path;
+         strncpy (path, source_path, len);     /* Before $cdir */
+         strcpy (path + len, s->dirname);      /* new stuff */
+         strcat (path + len, source_path + len + cdir_len);    /* After $cdir */
+       }
     }
 
-  result = openp (path, 0, s->filename, O_RDONLY, 0, &s->fullname);
+  result = openp (path, 0, s->filename, OPEN_MODE, 0, &s->fullname);
   if (result < 0)
     {
       /* Didn't work.  Try using just the basename. */
       p = basename (s->filename);
       if (p != s->filename)
-       result = openp(path, 0, p, O_RDONLY,0, &s->fullname);
+       result = openp (path, 0, p, OPEN_MODE, 0, &s->fullname);
+    }
+#ifdef MPW
+  if (result < 0)
+    {
+      /* Didn't work.  Try using just the MPW basename. */
+      p = (char *) mpw_basename (s->filename);
+      if (p != s->filename)
+       result = openp (path, 0, p, OPEN_MODE, 0, &s->fullname);
     }
+  if (result < 0)
+    {
+      /* Didn't work.  Try using the mixed Unix/MPW basename. */
+      p = (char *) mpw_mixed_basename (s->filename);
+      if (p != s->filename)
+       result = openp (path, 0, p, OPEN_MODE, 0, &s->fullname);
+    }
+#endif /* MPW */
+
   if (result >= 0)
     {
-      fullname = s -> fullname;
-      s -> fullname = mstrsave (s -> objfile -> md, s -> fullname);
-      free (fullname);
+      fullname = s->fullname;
+      s->fullname = mstrsave (s->objfile->md, s->fullname);
+      xfree (fullname);
     }
   return result;
 }
 
+/* Return the path to the source file associated with symtab.  Returns NULL
+   if no symtab.  */
+
+char *
+symtab_to_filename (struct symtab *s)
+{
+  int fd;
+
+  if (!s)
+    return NULL;
+
+  /* If we've seen the file before, just return fullname. */
+
+  if (s->fullname)
+    return s->fullname;
+
+  /* Try opening the file to setup fullname */
+
+  fd = open_source_file (s);
+  if (fd < 0)
+    return s->filename;                /* File not found.  Just use short name */
+
+  /* Found the file.  Cleanup and return the full name */
+
+  close (fd);
+  return s->fullname;
+}
 \f
+
 /* Create and initialize the table S->line_charpos that records
    the positions of the lines in the source file, which is assumed
    to be open on descriptor DESC.
    All set S->nlines to the number of such lines.  */
 
-static void
-find_source_lines (s, desc)
-     struct symtab *s;
-     int desc;
+void
+find_source_lines (struct symtab *s, int desc)
 {
   struct stat st;
-  char c;
   register char *data, *p, *end;
   int nlines = 0;
   int lines_allocated = 1000;
   int *line_charpos;
-  long exec_mtime;
+  long mtime = 0;
   int size;
 
-  line_charpos = (int *) xmmalloc (s -> objfile -> md,
+  line_charpos = (int *) xmmalloc (s->objfile->md,
                                   lines_allocated * sizeof (int));
   if (fstat (desc, &st) < 0)
-   perror_with_name (s->filename);
+    perror_with_name (s->filename);
 
-  if (exec_bfd) {
-    exec_mtime = bfd_get_mtime(exec_bfd);
-    if (exec_mtime && exec_mtime < st.st_mtime)
-     printf_filtered ("Source file is more recent than executable.\n");
-  }
+  if (s && s->objfile && s->objfile->obfd)
+    mtime = bfd_get_mtime (s->objfile->obfd);
+  else if (exec_bfd)
+    mtime = bfd_get_mtime (exec_bfd);
+
+  if (mtime && mtime < st.st_mtime)
+    {
+      if (tui_version)
+       printf_filtered ("\n");
+      warning ("Source file is more recent than executable.\n");
+    }
 
 #ifdef LSEEK_NOT_LINEAR
-  /* Have to read it byte by byte to find out where the chars live */
-
-   line_charpos[0] = tell(desc);
-   nlines = 1;
-   while (myread(desc, &c, 1)>0) 
-   {
-     if (c == '\n') 
-     {
-       if (nlines == lines_allocated) 
-       {
-        lines_allocated *= 2;
-        line_charpos =
-         (int *) xmrealloc (s -> objfile -> md, (char *) line_charpos,
-                            sizeof (int) * lines_allocated);
-       }
-       line_charpos[nlines++] = tell(desc);
-     }
-   }
+  {
+    char c;
 
-#else
-  /* st_size might be a large type, but we only support source files whose 
-     size fits in an int.  FIXME. */
-  size = (int) st.st_size;
+    /* Have to read it byte by byte to find out where the chars live */
 
-#ifdef BROKEN_LARGE_ALLOCA
-  data = (char *) xmalloc (size);
-  make_cleanup (free, data);
-#else
-  data = (char *) alloca (size);
-#endif
-  if (myread (desc, data, size) < 0)
-   perror_with_name (s->filename);
-  end = data + size;
-  p = data;
-  line_charpos[0] = 0;
-  nlines = 1;
-  while (p != end)
+    line_charpos[0] = lseek (desc, 0, SEEK_CUR);
+    nlines = 1;
+    while (myread (desc, &c, 1) > 0)
+      {
+       if (c == '\n')
+         {
+           if (nlines == lines_allocated)
+             {
+               lines_allocated *= 2;
+               line_charpos =
+                 (int *) xmrealloc (s->objfile->md, (char *) line_charpos,
+                                    sizeof (int) * lines_allocated);
+             }
+           line_charpos[nlines++] = lseek (desc, 0, SEEK_CUR);
+         }
+      }
+  }
+#else /* lseek linear.  */
   {
-    if (*p++ == '\n'
-       /* A newline at the end does not start a new line.  */
-       && p != end)
-    {
-      if (nlines == lines_allocated)
+    struct cleanup *old_cleanups;
+
+    /* st_size might be a large type, but we only support source files whose 
+       size fits in an int.  */
+    size = (int) st.st_size;
+
+    /* Use malloc, not alloca, because this may be pretty large, and we may
+       run into various kinds of limits on stack size.  */
+    data = (char *) xmalloc (size);
+    old_cleanups = make_cleanup (xfree, data);
+
+    /* Reassign `size' to result of read for systems where \r\n -> \n.  */
+    size = myread (desc, data, size);
+    if (size < 0)
+      perror_with_name (s->filename);
+    end = data + size;
+    p = data;
+    line_charpos[0] = 0;
+    nlines = 1;
+    while (p != end)
       {
-       lines_allocated *= 2;
-       line_charpos =
-        (int *) xmrealloc (s -> objfile -> md, (char *) line_charpos,
-                           sizeof (int) * lines_allocated);
+       if (*p++ == '\n'
+       /* A newline at the end does not start a new line.  */
+           && p != end)
+         {
+           if (nlines == lines_allocated)
+             {
+               lines_allocated *= 2;
+               line_charpos =
+                 (int *) xmrealloc (s->objfile->md, (char *) line_charpos,
+                                    sizeof (int) * lines_allocated);
+             }
+           line_charpos[nlines++] = p - data;
+         }
       }
-      line_charpos[nlines++] = p - data;
-    }
+    do_cleanups (old_cleanups);
   }
-#endif
+#endif /* lseek linear.  */
   s->nlines = nlines;
   s->line_charpos =
-   (int *) xmrealloc (s -> objfile -> md, (char *) line_charpos,
-                     nlines * sizeof (int));
+    (int *) xmrealloc (s->objfile->md, (char *) line_charpos,
+                      nlines * sizeof (int));
 
 }
 
 /* Return the character position of a line LINE in symtab S.
    Return 0 if anything is invalid.  */
 
-#if 0  /* Currently unused */
+#if 0                          /* Currently unused */
 
 int
-source_line_charpos (s, line)
-     struct symtab *s;
-     int line;
+source_line_charpos (struct symtab *s, int line)
 {
-  if (!s) return 0;
-  if (!s->line_charpos || line <= 0) return 0;
+  if (!s)
+    return 0;
+  if (!s->line_charpos || line <= 0)
+    return 0;
   if (line > s->nlines)
     line = s->nlines;
   return s->line_charpos[line - 1];
@@ -710,17 +886,16 @@ source_line_charpos (s, line)
 /* Return the line number of character position POS in symtab S.  */
 
 int
-source_charpos_line (s, chr)
-    register struct symtab *s;
-    register int chr;
+source_charpos_line (register struct symtab *s, register int chr)
 {
   register int line = 0;
   register int *lnp;
-    
-  if (s == 0 || s->line_charpos == 0) return 0;
+
+  if (s == 0 || s->line_charpos == 0)
+    return 0;
   lnp = s->line_charpos;
   /* Files are usually short, so sequential search is Ok */
-  while (line < s->nlines  && *lnp <= chr)
+  while (line < s->nlines && *lnp <= chr)
     {
       line++;
       lnp++;
@@ -730,32 +905,32 @@ source_charpos_line (s, chr)
   return line;
 }
 
-#endif /* 0 */
-
+#endif /* 0 */
 \f
+
 /* Get full pathname and line number positions for a symtab.
    Return nonzero if line numbers may have changed.
    Set *FULLNAME to actual name of the file as found by `openp',
    or to 0 if the file is not found.  */
 
 static int
-get_filename_and_charpos (s, fullname)
-     struct symtab *s;
-     char **fullname;
+get_filename_and_charpos (struct symtab *s, char **fullname)
 {
   register int desc, linenums_changed = 0;
-  
+
   desc = open_source_file (s);
   if (desc < 0)
     {
       if (fullname)
        *fullname = NULL;
       return 0;
-    }  
+    }
   if (fullname)
     *fullname = s->fullname;
-  if (s->line_charpos == 0) linenums_changed = 1;
-  if (linenums_changed) find_source_lines (s, desc);
+  if (s->line_charpos == 0)
+    linenums_changed = 1;
+  if (linenums_changed)
+    find_source_lines (s, desc);
   close (desc);
   return linenums_changed;
 }
@@ -770,19 +945,19 @@ get_filename_and_charpos (s, fullname)
    Return 1 if successful, 0 if could not find the file.  */
 
 int
-identify_source_line (s, line, mid_statement)
-     struct symtab *s;
-     int line;
-     int mid_statement;
+identify_source_line (struct symtab *s, int line, int mid_statement,
+                     CORE_ADDR pc)
 {
   if (s->line_charpos == 0)
-    get_filename_and_charpos (s, (char **)NULL);
+    get_filename_and_charpos (s, (char **) NULL);
   if (s->fullname == 0)
     return 0;
-  printf ("\032\032%s:%d:%d:%s:0x%x\n", s->fullname,
-         line, s->line_charpos[line - 1],
-         mid_statement ? "middle" : "beg",
-         get_frame_pc (get_current_frame()));
+  if (line > s->nlines)
+    /* Don't index off the end of the line_charpos array.  */
+    return 0;
+  annotate_source (s->fullname, line, s->line_charpos[line - 1],
+                  mid_statement, pc);
+
   current_source_line = line;
   first_line_listed = line;
   last_line_listed = line;
@@ -790,14 +965,14 @@ identify_source_line (s, line, mid_statement)
   return 1;
 }
 \f
+
 /* Print source lines from the file of symtab S,
-   starting with line number LINE and stopping before line number STOPLINE.  */
+   starting with line number LINE and stopping before line number STOPLINE. */
 
-void
-print_source_lines (s, line, stopline, noerror)
-     struct symtab *s;
-     int line, stopline;
-     int noerror;
+static void print_source_lines_base (struct symtab *s, int line, int stopline,
+                                    int noerror);
+static void
+print_source_lines_base (struct symtab *s, int line, int stopline, int noerror)
 {
   register int c;
   register int desc;
@@ -809,17 +984,56 @@ print_source_lines (s, line, stopline, noerror)
   current_source_line = line;
   first_line_listed = line;
 
-  desc = open_source_file (s);
+#ifdef UI_OUT
+  /* If printing of source lines is disabled, just print file and line number */
+  if (ui_out_test_flags (uiout, ui_source_list))
+    {
+#endif
+      /* Only prints "No such file or directory" once */
+      if ((s != last_source_visited) || (!last_source_error))
+       {
+         last_source_visited = s;
+         desc = open_source_file (s);
+       }
+      else
+       {
+         desc = last_source_error;
+         noerror = 1;
+       }
+#ifdef UI_OUT
+    }
+  else
+    {
+      desc = -1;
+      noerror = 1;
+    }
+#endif
+
   if (desc < 0)
     {
-      if (! noerror) {
-       char *name = alloca (strlen (s->filename) + 100);
-       sprintf (name, "%s:%d", s->filename, line);
-        print_sys_errmsg (name, errno);
-      }
+      last_source_error = desc;
+
+      if (!noerror)
+       {
+         char *name = alloca (strlen (s->filename) + 100);
+         sprintf (name, "%d\t%s", line, s->filename);
+         print_sys_errmsg (name, errno);
+       }
+      else
+#ifdef UI_OUT
+       ui_out_field_int (uiout, "line", line);
+      ui_out_text (uiout, "\tin ");
+      ui_out_field_string (uiout, "file", s->filename);
+      ui_out_text (uiout, "\n");
+#else
+       printf_filtered ("%d\tin %s\n", line, s->filename);
+#endif
+
       return;
     }
 
+  last_source_error = 0;
+
   if (s->line_charpos == 0)
     find_source_lines (s, desc);
 
@@ -836,54 +1050,130 @@ print_source_lines (s, line, stopline, noerror)
       perror_with_name (s->filename);
     }
 
-  stream = fdopen (desc, "r");
+  stream = fdopen (desc, FDOPEN_MODE);
   clearerr (stream);
 
   while (nlines-- > 0)
     {
+#ifdef UI_OUT
+      char buf[20];
+
+      c = fgetc (stream);
+      if (c == EOF)
+       break;
+      last_line_listed = current_source_line;
+      sprintf (buf, "%d\t", current_source_line++);
+      ui_out_text (uiout, buf);
+      do
+       {
+         if (c < 040 && c != '\t' && c != '\n' && c != '\r')
+           {
+             sprintf (buf, "^%c", c + 0100);
+             ui_out_text (uiout, buf);
+           }
+         else if (c == 0177)
+           ui_out_text (uiout, "^?");
+#ifdef CRLF_SOURCE_FILES
+         else if (c == '\r')
+           {
+             /* Skip a \r character, but only before a \n.  */
+             int c1 = fgetc (stream);
+
+             if (c1 != '\n')
+               printf_filtered ("^%c", c + 0100);
+             if (c1 != EOF)
+               ungetc (c1, stream);
+           }
+#endif
+         else
+           {
+             sprintf (buf, "%c", c);
+             ui_out_text (uiout, buf);
+           }
+       }
+      while (c != '\n' && (c = fgetc (stream)) >= 0);
+#else
       c = fgetc (stream);
-      if (c == EOF) break;
+      if (c == EOF)
+       break;
       last_line_listed = current_source_line;
       printf_filtered ("%d\t", current_source_line++);
       do
        {
          if (c < 040 && c != '\t' && c != '\n' && c != '\r')
-             printf_filtered ("^%c", c + 0100);
+           printf_filtered ("^%c", c + 0100);
          else if (c == 0177)
            printf_filtered ("^?");
+#ifdef CRLF_SOURCE_FILES
+         else if (c == '\r')
+           {
+             /* Just skip \r characters.  */
+           }
+#endif
          else
            printf_filtered ("%c", c);
-       } while (c != '\n' && (c = fgetc (stream)) >= 0);
+       }
+      while (c != '\n' && (c = fgetc (stream)) >= 0);
+#endif
     }
 
   fclose (stream);
 }
 \f
+/* Show source lines from the file of symtab S, starting with line
+   number LINE and stopping before line number STOPLINE.  If this is the
+   not the command line version, then the source is shown in the source
+   window otherwise it is simply printed */
 
+void
+print_source_lines (struct symtab *s, int line, int stopline, int noerror)
+{
+#if defined(TUI)
+  if (!tui_version ||
+      m_winPtrIsNull (srcWin) || !srcWin->generic.isVisible)
+    print_source_lines_base (s, line, stopline, noerror);
+  else
+    {
+      TuiGenWinInfoPtr locator = locatorWinInfoPtr ();
+extern void tui_vAddWinToLayout (va_list);
+extern void tui_vUpdateSourceWindowsWithLine (va_list);
+
+      /* Regardless of whether we can open the file,
+         set current_source_symtab. */
+      current_source_symtab = s;
+      current_source_line = line;
+      first_line_listed = line;
+
+      /* make sure that the source window is displayed */
+      tuiDo ((TuiOpaqueFuncPtr) tui_vAddWinToLayout, SRC_WIN);
+
+      tuiDo ((TuiOpaqueFuncPtr) tui_vUpdateSourceWindowsWithLine, s, line);
+      tuiDo ((TuiOpaqueFuncPtr) tui_vUpdateLocatorFilename, s->filename);
+    }
+#else
+  print_source_lines_base (s, line, stopline, noerror);
+#endif
+}
+\f
+
+
+/* Print a list of files and line numbers which a user may choose from
+   in order to list a function which was specified ambiguously (as with
+   `list classname::overloadedfuncname', for example).  The vector in
+   SALS provides the filenames and line numbers.  */
 
-/* 
-  C++
-  Print a list of files and line numbers which a user may choose from
-  in order to list a function which was specified ambiguously
-  (as with `list classname::overloadedfuncname', for example).
-  The vector in SALS provides the filenames and line numbers.
-  */
 static void
-ambiguous_line_spec (sals)
-     struct symtabs_and_lines *sals;
+ambiguous_line_spec (struct symtabs_and_lines *sals)
 {
   int i;
 
   for (i = 0; i < sals->nelts; ++i)
-    printf_filtered("file: \"%s\", line number: %d\n",
-                   sals->sals[i].symtab->filename, sals->sals[i].line);
+    printf_filtered ("file: \"%s\", line number: %d\n",
+                    sals->sals[i].symtab->filename, sals->sals[i].line);
 }
 
-
 static void
-list_command (arg, from_tty)
-     char *arg;
-     int from_tty;
+list_command (char *arg, int from_tty)
 {
   struct symtabs_and_lines sals, sals_end;
   struct symtab_and_line sal, sal_end;
@@ -895,7 +1185,7 @@ list_command (arg, from_tty)
   int linenum_beg = 0;
   char *p;
 
-  if (!have_full_symbols () && !have_partial_symbols())
+  if (!have_full_symbols () && !have_partial_symbols ())
     error ("No symbol table is loaded.  Use the \"file\" command.");
 
   /* Pull in a current source symtab if necessary */
@@ -905,7 +1195,7 @@ list_command (arg, from_tty)
 
   /* "l" or "l +" lists next ten lines.  */
 
-  if (arg == 0 || !strcmp (arg, "+"))
+  if (arg == 0 || STREQ (arg, "+"))
     {
       if (current_source_symtab == 0)
        error ("No default source file yet.  Do \"help list\".");
@@ -915,7 +1205,7 @@ list_command (arg, from_tty)
     }
 
   /* "l -" lists previous ten lines, the ones before the ten just listed.  */
-  if (!strcmp (arg, "-"))
+  if (STREQ (arg, "-"))
     {
       if (current_source_symtab == 0)
        error ("No default source file yet.  Do \"help list\".");
@@ -936,18 +1226,19 @@ list_command (arg, from_tty)
     dummy_beg = 1;
   else
     {
-      sals = decode_line_1 (&arg1, 0, 0, 0);
+      sals = decode_line_1 (&arg1, 0, 0, 0, 0);
 
-      if (! sals.nelts) return;  /*  C++  */
+      if (!sals.nelts)
+       return;                 /*  C++  */
       if (sals.nelts > 1)
        {
          ambiguous_line_spec (&sals);
-         free (sals.sals);
+         xfree (sals.sals);
          return;
        }
 
       sal = sals.sals[0];
-      free (sals.sals);
+      xfree (sals.sals);
     }
 
   /* Record whether the BEG arg is all digits.  */
@@ -968,19 +1259,19 @@ list_command (arg, from_tty)
       else
        {
          if (dummy_beg)
-           sals_end = decode_line_1 (&arg1, 0, 0, 0);
+           sals_end = decode_line_1 (&arg1, 0, 0, 0, 0);
          else
-           sals_end = decode_line_1 (&arg1, 0, sal.symtab, sal.line);
-         if (sals_end.nelts == 0) 
+           sals_end = decode_line_1 (&arg1, 0, sal.symtab, sal.line, 0);
+         if (sals_end.nelts == 0)
            return;
          if (sals_end.nelts > 1)
            {
              ambiguous_line_spec (&sals_end);
-             free (sals_end.sals);
+             xfree (sals_end.sals);
              return;
            }
          sal_end = sals_end.sals[0];
-         free (sals_end.sals);
+         xfree (sals_end.sals);
        }
     }
 
@@ -992,7 +1283,7 @@ list_command (arg, from_tty)
     error ("Specified start and end are in different files.");
   if (dummy_beg && dummy_end)
     error ("Two empty args do not say what lines to list.");
+
   /* if line was specified by address,
      first print exactly which line, and which file.
      In this case, sal.symtab == 0 means address is outside
@@ -1000,25 +1291,30 @@ list_command (arg, from_tty)
   if (*arg == '*')
     {
       if (sal.symtab == 0)
-       error ("No source file for address %s.", local_hex_string(sal.pc));
+       /* FIXME-32x64--assumes sal.pc fits in long.  */
+       error ("No source file for address %s.",
+              local_hex_string ((unsigned long) sal.pc));
       sym = find_pc_function (sal.pc);
       if (sym)
        {
-         printf_filtered ("%s is in ", local_hex_string(sal.pc));
-         fprint_symbol (stdout, SYMBOL_NAME (sym));
+         print_address_numeric (sal.pc, 1, gdb_stdout);
+         printf_filtered (" is in ");
+         fputs_filtered (SYMBOL_SOURCE_NAME (sym), gdb_stdout);
          printf_filtered (" (%s:%d).\n", sal.symtab->filename, sal.line);
        }
       else
-       printf_filtered ("%s is at %s:%d.\n",
-                        local_hex_string(sal.pc), 
-                        sal.symtab->filename, sal.line);
+       {
+         print_address_numeric (sal.pc, 1, gdb_stdout);
+         printf_filtered (" is at %s:%d.\n",
+                          sal.symtab->filename, sal.line);
+       }
     }
 
   /* If line was not specified by just a line number,
      and it does not imply a symtab, it must be an undebuggable symbol
      which means no source code.  */
 
-  if (! linenum_beg && sal.symtab == 0)
+  if (!linenum_beg && sal.symtab == 0)
     error ("No line number known for %s.", arg);
 
   /* If this command is repeated with RET,
@@ -1036,9 +1332,14 @@ list_command (arg, from_tty)
   else if (sal.symtab == 0)
     error ("No default source file yet.  Do \"help list\".");
   else if (no_end)
-    print_source_lines (sal.symtab,
-                       max (sal.line - (lines_to_list / 2), 1),
-                       sal.line + (lines_to_list / 2), 0);
+    {
+      int first_line = sal.line - lines_to_list / 2;
+
+      if (first_line < 1) first_line = 1;
+
+      print_source_lines (sal.symtab, first_line, first_line + lines_to_list,
+                         0);
+    }
   else
     print_source_lines (sal.symtab, sal.line,
                        (dummy_end
@@ -1050,15 +1351,15 @@ list_command (arg, from_tty)
 /* Print info on range of pc's in a specified line.  */
 
 static void
-line_info (arg, from_tty)
-     char *arg;
-     int from_tty;
+line_info (char *arg, int from_tty)
 {
   struct symtabs_and_lines sals;
   struct symtab_and_line sal;
   CORE_ADDR start_pc, end_pc;
   int i;
 
+  INIT_SAL (&sal);             /* initialize to zeroes */
+
   if (arg == 0)
     {
       sal.symtab = current_source_symtab;
@@ -1071,11 +1372,8 @@ line_info (arg, from_tty)
   else
     {
       sals = decode_line_spec_1 (arg, 0);
-      
-      /* If this command is repeated with RET,
-        turn it into the no-arg variant.  */
-      if (from_tty)
-       *arg = 0;
+
+      dont_repeat ();
     }
 
   /* C++  More than one line may have been specified, as when the user
@@ -1083,49 +1381,106 @@ line_info (arg, from_tty)
   for (i = 0; i < sals.nelts; i++)
     {
       sal = sals.sals[i];
-      
-      if (sal.symtab == 0)
-       error ("No source file specified.");
 
-      if (sal.line > 0
-         && find_line_pc_range (sal.symtab, sal.line, &start_pc, &end_pc))
+      if (sal.symtab == 0)
+       {
+         printf_filtered ("No line number information available");
+         if (sal.pc != 0)
+           {
+             /* This is useful for "info line *0x7f34".  If we can't tell the
+                user about a source line, at least let them have the symbolic
+                address.  */
+             printf_filtered (" for address ");
+             wrap_here ("  ");
+             print_address (sal.pc, gdb_stdout);
+           }
+         else
+           printf_filtered (".");
+         printf_filtered ("\n");
+       }
+      else if (sal.line > 0
+              && find_line_pc_range (sal, &start_pc, &end_pc))
        {
          if (start_pc == end_pc)
-           printf_filtered ("Line %d of \"%s\" is at pc %s but contains no code.\n",
-                            sal.line, sal.symtab->filename, local_hex_string(start_pc));
+           {
+             printf_filtered ("Line %d of \"%s\"",
+                              sal.line, sal.symtab->filename);
+             wrap_here ("  ");
+             printf_filtered (" is at address ");
+             print_address (start_pc, gdb_stdout);
+             wrap_here ("  ");
+             printf_filtered (" but contains no code.\n");
+           }
          else
            {
-             printf_filtered ("Line %d of \"%s\" starts at pc %s",
-                              sal.line, sal.symtab->filename, 
-                              local_hex_string(start_pc));
-             printf_filtered (" and ends at %s.\n",
-                              local_hex_string(end_pc));
+             printf_filtered ("Line %d of \"%s\"",
+                              sal.line, sal.symtab->filename);
+             wrap_here ("  ");
+             printf_filtered (" starts at address ");
+             print_address (start_pc, gdb_stdout);
+             wrap_here ("  ");
+             printf_filtered (" and ends at ");
+             print_address (end_pc, gdb_stdout);
+             printf_filtered (".\n");
            }
+
          /* x/i should display this line's code.  */
          set_next_address (start_pc);
+
          /* Repeating "info line" should do the following line.  */
          last_line_listed = sal.line + 1;
+
+         /* If this is the only line, show the source code.  If it could
+            not find the file, don't do anything special.  */
+         if (annotation_level && sals.nelts == 1)
+           identify_source_line (sal.symtab, sal.line, 0, start_pc);
        }
       else
+       /* Is there any case in which we get here, and have an address
+          which the user would want to see?  If we have debugging symbols
+          and no line numbers?  */
        printf_filtered ("Line number %d is out of range for \"%s\".\n",
                         sal.line, sal.symtab->filename);
     }
+  xfree (sals.sals);
 }
 \f
 /* Commands to search the source file for a regexp.  */
 
 /* ARGSUSED */
 static void
-forward_search_command (regex, from_tty)
-     char *regex;
-     int from_tty;
+forward_search_command (char *regex, int from_tty)
 {
   register int c;
   register int desc;
   register FILE *stream;
-  int line = last_line_listed + 1;
+  int line;
   char *msg;
 
+#if defined(TUI)
+  /* 
+     ** If this is the TUI, search from the first line displayed in 
+     ** the source window, otherwise, search from last_line_listed+1 
+     ** in current_source_symtab 
+   */
+  if (!tui_version)
+    line = last_line_listed;
+  else
+    {
+      if (srcWin->generic.isVisible && srcWin->generic.contentSize > 0)
+       line = ((TuiWinContent)
+        srcWin->generic.content)[0]->whichElement.source.lineOrAddr.lineNo;
+      else
+       {
+         printf_filtered ("No source displayed.\nExpression not found.\n");
+         return;
+       }
+    }
+  line++;
+#else
+  line = last_line_listed + 1;
+#endif
+
   msg = (char *) re_comp (regex);
   if (msg)
     error (msg);
@@ -1133,8 +1488,6 @@ forward_search_command (regex, from_tty)
   if (current_source_symtab == 0)
     select_source_symtab (0);
 
-  /* Search from last_line_listed+1 in current_source_symtab */
-
   desc = open_source_file (current_source_symtab);
   if (desc < 0)
     perror_with_name (current_source_symtab->filename);
@@ -1154,33 +1507,61 @@ forward_search_command (regex, from_tty)
       perror_with_name (current_source_symtab->filename);
     }
 
-  stream = fdopen (desc, "r");
+  stream = fdopen (desc, FDOPEN_MODE);
   clearerr (stream);
-  while (1) {
-/* FIXME!!!  We walk right off the end of buf if we get a long line!!! */
-    char buf[4096];            /* Should be reasonable??? */
-    register char *p = buf;
-
-    c = getc (stream);
-    if (c == EOF)
-      break;
-    do {
-      *p++ = c;
-    } while (c != '\n' && (c = getc (stream)) >= 0);
-
-    /* we now have a source line in buf, null terminate and match */
-    *p = 0;
-    if (re_exec (buf) > 0)
-      {
-       /* Match! */
-       fclose (stream);
-       print_source_lines (current_source_symtab,
-                          line, line+1, 0);
-       current_source_line = max (line - lines_to_list / 2, 1);
-       return;
-      }
-    line++;
-  }
+  while (1)
+    {
+      static char *buf = NULL;
+      register char *p;
+      int cursize, newsize;
+
+      cursize = 256;
+      buf = xmalloc (cursize);
+      p = buf;
+
+      c = getc (stream);
+      if (c == EOF)
+       break;
+      do
+       {
+         *p++ = c;
+         if (p - buf == cursize)
+           {
+             newsize = cursize + cursize / 2;
+             buf = xrealloc (buf, newsize);
+             p = buf + cursize;
+             cursize = newsize;
+           }
+       }
+      while (c != '\n' && (c = getc (stream)) >= 0);
+
+#ifdef CRLF_SOURCE_FILES
+      /* Remove the \r, if any, at the end of the line, otherwise
+         regular expressions that end with $ or \n won't work.  */
+      if (p - buf > 1 && p[-2] == '\r')
+       {
+         p--;
+         p[-1] = '\n';
+       }
+#endif
+
+      /* we now have a source line in buf, null terminate and match */
+      *p = 0;
+      if (re_exec (buf) > 0)
+       {
+         /* Match! */
+         fclose (stream);
+         if (tui_version)
+           print_source_lines_base (current_source_symtab, line, line + 1, 0);
+         print_source_lines (current_source_symtab, line, line + 1, 0);
+         set_internalvar (lookup_internalvar ("_"),
+                          value_from_longest (builtin_type_int,
+                                              (LONGEST) line));
+         current_source_line = max (line - lines_to_list / 2, 1);
+         return;
+       }
+      line++;
+    }
 
   printf_filtered ("Expression not found\n");
   fclose (stream);
@@ -1188,15 +1569,36 @@ forward_search_command (regex, from_tty)
 
 /* ARGSUSED */
 static void
-reverse_search_command (regex, from_tty)
-     char *regex;
-     int from_tty;
+reverse_search_command (char *regex, int from_tty)
 {
   register int c;
   register int desc;
   register FILE *stream;
-  int line = last_line_listed - 1;
+  int line;
   char *msg;
+#if defined(TUI)
+  /*
+     ** If this is the TUI, search from the first line displayed in
+     ** the source window, otherwise, search from last_line_listed-1
+     ** in current_source_symtab
+   */
+  if (!tui_version)
+    line = last_line_listed;
+  else
+    {
+      if (srcWin->generic.isVisible && srcWin->generic.contentSize > 0)
+       line = ((TuiWinContent)
+        srcWin->generic.content)[0]->whichElement.source.lineOrAddr.lineNo;
+      else
+       {
+         printf_filtered ("No source displayed.\nExpression not found.\n");
+         return;
+       }
+    }
+  line--;
+#else
+  line = last_line_listed - 1;
+#endif
 
   msg = (char *) re_comp (regex);
   if (msg)
@@ -1205,8 +1607,6 @@ reverse_search_command (regex, from_tty)
   if (current_source_symtab == 0)
     select_source_symtab (0);
 
-  /* Search from last_line_listed-1 in current_source_symtab */
-
   desc = open_source_file (current_source_symtab);
   if (desc < 0)
     perror_with_name (current_source_symtab->filename);
@@ -1226,7 +1626,7 @@ reverse_search_command (regex, from_tty)
       perror_with_name (current_source_symtab->filename);
     }
 
-  stream = fdopen (desc, "r");
+  stream = fdopen (desc, FDOPEN_MODE);
   clearerr (stream);
   while (line > 1)
     {
@@ -1237,9 +1637,21 @@ reverse_search_command (regex, from_tty)
       c = getc (stream);
       if (c == EOF)
        break;
-      do {
-       *p++ = c;
-      } while (c != '\n' && (c = getc (stream)) >= 0);
+      do
+       {
+         *p++ = c;
+       }
+      while (c != '\n' && (c = getc (stream)) >= 0);
+
+#ifdef CRLF_SOURCE_FILES
+      /* Remove the \r, if any, at the end of the line, otherwise
+         regular expressions that end with $ or \n won't work.  */
+      if (p - buf > 1 && p[-2] == '\r')
+       {
+         p--;
+         p[-1] = '\n';
+       }
+#endif
 
       /* We now have a source line in buf; null terminate and match.  */
       *p = 0;
@@ -1247,8 +1659,12 @@ reverse_search_command (regex, from_tty)
        {
          /* Match! */
          fclose (stream);
-         print_source_lines (current_source_symtab,
-                             line, line+1, 0);
+         if (tui_version)
+           print_source_lines_base (current_source_symtab, line, line + 1, 0);
+         print_source_lines (current_source_symtab, line, line + 1, 0);
+         set_internalvar (lookup_internalvar ("_"),
+                          value_from_longest (builtin_type_int,
+                                              (LONGEST) line));
          current_source_line = max (line - lines_to_list / 2, 1);
          return;
        }
@@ -1266,17 +1682,30 @@ reverse_search_command (regex, from_tty)
 }
 \f
 void
-_initialize_source ()
+_initialize_source (void)
 {
+  struct cmd_list_element *c;
   current_source_symtab = 0;
   init_source_path ();
 
-  add_com ("directory", class_files, directory_command,
-          "Add directory DIR to beginning of search path for source files.\n\
+  /* The intention is to use POSIX Basic Regular Expressions.
+     Always use the GNU regex routine for consistency across all hosts.
+     Our current GNU regex.c does not have all the POSIX features, so this is
+     just an approximation.  */
+  re_set_syntax (RE_SYNTAX_GREP);
+
+  c = add_cmd ("directory", class_files, directory_command,
+              "Add directory DIR to beginning of search path for source files.\n\
 Forget cached info on source file locations and line positions.\n\
 DIR can also be $cwd for the current working directory, or $cdir for the\n\
 directory in which the source file was compiled into object code.\n\
-With no argument, reset the search path to $cdir:$cwd, the default.");
+With no argument, reset the search path to $cdir:$cwd, the default.",
+              &cmdlist);
+
+  if (dbx_commands)
+    add_com_alias ("use", "directory", class_files, 0);
+
+  c->completer = filename_completer;
 
   add_cmd ("directories", no_class, show_directories,
           "Current search path for finding source files.\n\
@@ -1284,47 +1713,74 @@ $cwd in the path means the current working directory.\n\
 $cdir in the path means the compilation directory of the source file.",
           &showlist);
 
+  if (xdb_commands)
+    {
+      add_com_alias ("D", "directory", class_files, 0);
+      add_cmd ("ld", no_class, show_directories,
+              "Current search path for finding source files.\n\
+$cwd in the path means the current working directory.\n\
+$cdir in the path means the compilation directory of the source file.",
+              &cmdlist);
+    }
+
   add_info ("source", source_info,
            "Information about the current source file.");
 
   add_info ("line", line_info,
-           "Core addresses of the code for a source line.\n\
+           concat ("Core addresses of the code for a source line.\n\
 Line can be specified as\n\
   LINENUM, to list around that line in current file,\n\
   FILE:LINENUM, to list around that line in that file,\n\
   FUNCTION, to list around beginning of that function,\n\
   FILE:FUNCTION, to distinguish among like-named static functions.\n\
+", "\
 Default is to describe the last source line that was listed.\n\n\
 This sets the default address for \"x\" to the line's first instruction\n\
 so that \"x/i\" suffices to start examining the machine code.\n\
-The address is also stored as the value of \"$_\".");
+The address is also stored as the value of \"$_\".", NULL));
 
   add_com ("forward-search", class_files, forward_search_command,
-          "Search for regular expression (see regex(3)) from last line listed.");
+          "Search for regular expression (see regex(3)) from last line listed.\n\
+The matching line number is also stored as the value of \"$_\".");
   add_com_alias ("search", "forward-search", class_files, 0);
 
   add_com ("reverse-search", class_files, reverse_search_command,
-          "Search backward for regular expression (see regex(3)) from last line listed.");
+          "Search backward for regular expression (see regex(3)) from last line listed.\n\
+The matching line number is also stored as the value of \"$_\".");
+
+  if (xdb_commands)
+    {
+      add_com_alias ("/", "forward-search", class_files, 0);
+      add_com_alias ("?", "reverse-search", class_files, 0);
+    }
 
   add_com ("list", class_files, list_command,
-          "List specified function or line.\n\
+          concat ("List specified function or line.\n\
 With no argument, lists ten more lines after or around previous listing.\n\
 \"list -\" lists the ten lines before a previous ten-line listing.\n\
 One argument specifies a line, and ten lines are listed around that line.\n\
 Two arguments with comma between specify starting and ending lines to list.\n\
+", "\
 Lines can be specified in these ways:\n\
   LINENUM, to list around that line in current file,\n\
   FILE:LINENUM, to list around that line in that file,\n\
   FUNCTION, to list around beginning of that function,\n\
   FILE:FUNCTION, to distinguish among like-named static functions.\n\
   *ADDRESS, to list around the line containing that address.\n\
-With two args if one is empty it stands for ten lines away from the other arg.");
-  add_com_alias ("l", "list", class_files, 1);
+With two args if one is empty it stands for ten lines away from the other arg.", NULL));
+
+  if (!xdb_commands)
+    add_com_alias ("l", "list", class_files, 1);
+  else
+    add_com_alias ("v", "list", class_files, 1);
+
+  if (dbx_commands)
+    add_com_alias ("file", "list", class_files, 1);
 
   add_show_from_set
     (add_set_cmd ("listsize", class_support, var_uinteger,
-                 (char *)&lines_to_list,
-       "Set number of source lines gdb will list by default.",
+                 (char *) &lines_to_list,
+                 "Set number of source lines gdb will list by default.",
                  &setlist),
      &showlist);
 }
This page took 0.050394 seconds and 4 git commands to generate.