From 256f06f3f6f833a3ac2abd5fcb341399028cc03c Mon Sep 17 00:00:00 2001 From: Doug Evans Date: Thu, 10 Nov 2011 22:14:16 +0000 Subject: [PATCH] * psymtab.c (psymtab_to_fullname): Use cached copy if it exists. * source.c (symtab_to_fullname): Ditto. --- gdb/ChangeLog | 3 +++ gdb/psymtab.c | 9 +++++++-- gdb/source.c | 9 +++++++-- 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 + * 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) -- 2.30.2