This is the mail archive of the archer@sourceware.org mailing list for the Archer project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[delayed-symfile] make print conditions parallel


Jan pointed out some regressions in F11 caused by spurious "no
debugging symbols found" prints.  I could only reproduce with all the
RPM patches applied.

I tracked this down to a missing clause on a condition.  This patch
makes all the prints in this function have the same condition, so that
this particular message does not appear unless other messages already
have.

I'm checking this in on the delayed-symfile branch.

Jan, this fixed the regressions you mentioned to me.  I'm running a
full regression test for the delayed-symfile branch, just to be
careful -- but since I could only reproduce the failure with more
things applied, I doubt this will do much.  I'm going to push this
once the tests are finished.

Tom

2009-03-06  Tom Tromey  <tromey@redhat.com>

	* symfile.c (symbol_file_add_with_addrs_or_offsets): Make "no
	debugging symbols found" message use same condition as other
	prints.

diff --git a/gdb/symfile.c b/gdb/symfile.c
index a95a8c1..92272a4 100644
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -1066,7 +1066,8 @@ symbol_file_add_with_addrs_or_offsets (bfd *abfd, int from_tty,
 
   /* has_any_debug_symbols is not fully compatible with the former calls which
      would just be needlessly expensive here.  */
-  if (print_symbol_loading && !has_any_debug_symbols (objfile) && mainline)
+  if ((from_tty || info_verbose) && print_symbol_loading
+      && !has_any_debug_symbols (objfile) && mainline)
     {
       wrap_here ("");
       printf_unfiltered (_("(no debugging symbols found)"));


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]