summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--gdb/ChangeLog3
-rw-r--r--gdb/psymtab.c9
-rw-r--r--gdb/source.c9
3 files changed, 17 insertions, 4 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4cdc9ede831..63deaf1db5e 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -10,6 +10,9 @@
2011-11-10 Doug Evans <dje@google.com>
+ * psymtab.c (psymtab_to_fullname): Use cached copy if it exists.
+ * source.c (symtab_to_fullname): Ditto.
+
* defs.h (is_cplus_marker, set_demangling_style): Moved to ...
* gdb-demangle.h: ... here. New file.
* demangle.c: #include "gdb-demangle.h".
diff --git a/gdb/psymtab.c b/gdb/psymtab.c
index 1f3b3dc64c0..38adca2e227 100644
--- a/gdb/psymtab.c
+++ b/gdb/psymtab.c
@@ -1110,6 +1110,7 @@ int find_and_open_source (const char *filename,
If this function fails to find the file that this partial_symtab represents,
NULL will be returned and ps->fullname will be set to NULL. */
+
static char *
psymtab_to_fullname (struct partial_symtab *ps)
{
@@ -1118,8 +1119,12 @@ psymtab_to_fullname (struct partial_symtab *ps)
if (!ps)
return NULL;
- /* Don't check ps->fullname here, the file could have been
- deleted/moved/..., look for it again. */
+ /* Use cached copy if we have it.
+ We rely on forget_cached_source_info being called appropriately
+ to handle cases like the file being moved. */
+ if (ps->fullname)
+ return ps->fullname;
+
r = find_and_open_source (ps->filename, ps->dirname, &ps->fullname);
if (r >= 0)
diff --git a/gdb/source.c b/gdb/source.c
index 6e29172f601..77df54138f6 100644
--- a/gdb/source.c
+++ b/gdb/source.c
@@ -1105,6 +1105,7 @@ open_source_file (struct symtab *s)
If this function fails to find the file that this symtab represents,
NULL will be returned and s->fullname will be set to NULL. */
+
char *
symtab_to_fullname (struct symtab *s)
{
@@ -1113,8 +1114,12 @@ symtab_to_fullname (struct symtab *s)
if (!s)
return NULL;
- /* Don't check s->fullname here, the file could have been
- deleted/moved/..., look for it again. */
+ /* Use cached copy if we have it.
+ We rely on forget_cached_source_info being called appropriately
+ to handle cases like the file being moved. */
+ if (s->fullname)
+ return s->fullname;
+
r = find_and_open_source (s->filename, s->dirname, &s->fullname);
if (r >= 0)