* [expr-cumulative] [commit] Remove excessive debug printf's
@ 2010-03-05 9:31 Jan Kratochvil
0 siblings, 0 replies; only message in thread
From: Jan Kratochvil @ 2010-03-05 9:31 UTC (permalink / raw)
To: archer
commit f78a980bcef45ee74bf267dfcc9633cd9c607a56
Remove excessive debug printf's.
708 FAILs -> 1218 PASSes.
The only remaining regression is the well known:
gdb.java/jprint.exp: print a java.util.Properties
---
gdb/eval.c | 1 -
gdb/symtab.c | 18 +++++-------------
2 files changed, 5 insertions(+), 14 deletions(-)
diff --git a/gdb/eval.c b/gdb/eval.c
index 201ead3..524b09e 100644
--- a/gdb/eval.c
+++ b/gdb/eval.c
@@ -1896,7 +1896,6 @@ evaluate_subexp_standard (struct type *expect_type,
struct symbol *sym;
struct type **arg_types;
(*pos) += 3;
- printf ("TYPE_INSTANCE_LOOKUP\n");
arg_types = (struct type **) alloca (TYPE_NFIELDS (expect_type)
* sizeof (struct type *));
for (i = 0; i < TYPE_NFIELDS (expect_type); ++i)
diff --git a/gdb/symtab.c b/gdb/symtab.c
index e3822b1..520f5a4 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -1326,13 +1326,9 @@ struct symbol *
lookup_symbol (const char *name, const struct block *block,
domain_enum domain, int *is_a_field_of_this)
{
- printf("symtab.c::lookup_symbol %s\n",name);
- struct symbol *sym = lookup_symbol_in_language (name, block, domain,
+ return lookup_symbol_in_language (name, block, domain,
current_language->la_language,
is_a_field_of_this);
-
- printf("symtab.c::lookup_symbol return %p\n", sym);
- return sym;
}
/* Behave like lookup_symbol except that NAME is the natural name
@@ -1348,7 +1344,6 @@ lookup_symbol_aux (const char *name, const struct block *block,
struct symbol *sym;
const struct language_defn *langdef;
- printf("symtab.c::lookup_symbol_aux\n", name);
/* Make sure we do something sensible with is_a_field_of_this, since
the callers that set this parameter to some non-null value will
certainly use it later and expect it to be either 0 or 1.
@@ -1361,17 +1356,14 @@ lookup_symbol_aux (const char *name, const struct block *block,
STATIC_BLOCK or GLOBAL_BLOCK. */
sym = lookup_symbol_aux_local (name, block, domain, language, is_a_field_of_this);
- if (sym != NULL){
- printf("symtab.c::lookup_symbol_aux return 1 %p\n", sym);
+ if (sym != NULL)
return sym;
- }
+
/* this symbol was found to be a member variable
do not perform the global search. */
- if (is_a_field_of_this && *is_a_field_of_this){
- printf("symtab.c::lookup_symbol_aux return 2 %p %d\n", NULL, *is_a_field_of_this);
-
+ if (is_a_field_of_this && *is_a_field_of_this)
return NULL;
- }
+
/* Now do whatever is appropriate for LANGUAGE to look
up static and global variables. */
langdef = language_def (language);
--
1.6.6.1
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2010-03-05 9:31 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-05 9:31 [expr-cumulative] [commit] Remove excessive debug printf's Jan Kratochvil
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).