Remove "noisy" parameter from clear_complaints

After the previous patch, the "noisy" parameter to clear_complaints is
no longer used, so this patch removes it.

gdb/ChangeLog
2018-05-23  Tom Tromey  <tom@tromey.com>

	* complaints.c (clear_complaints): Remove "noisy" parameter.
	* complaints.h (clear_complaints): Update.
	* symfile.c (syms_from_objfile_1, finish_new_objfile)
	(reread_symbols): Update.

gdb/testsuite/ChangeLog
2018-05-23  Tom Tromey  <tom@tromey.com>

	* gdb.gdb/complaints.exp (test_empty_complaints): Update.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4056481..6b5c336 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,12 @@
 2018-05-23  Tom Tromey  <tom@tromey.com>
 
+       * complaints.c (clear_complaints): Remove "noisy" parameter.
+       * complaints.h (clear_complaints): Update.
+       * symfile.c (syms_from_objfile_1, finish_new_objfile)
+       (reread_symbols): Update.
+
+2018-05-23  Tom Tromey  <tom@tromey.com>
+
        * complaints.c (enum complaint_series): Remove FIRST_MESSAGE,
        SUBSEQUENT_MESSAGE.
        (vcomplaint, clear_complaints): Update.

diff --git a/gdb/complaints.c b/gdb/complaints.c
index 71d36e6..d29b3c7 100644 (file)
--- a/gdb/complaints.c
+++ b/gdb/complaints.c
@@ -220,12 +220,10 @@ complaint_internal (struct complaints **complaints, const char *fmt, ...)
    incremented.  If LESS_VERBOSE is 1, be less verbose about
    successive complaints, since the messages are appearing all
    together during a command that is reporting a contiguous block of
-   complaints (rather than being interleaved with other messages).  If
-   noisy is 1, we are in a noisy command, and our caller will print
-   enough context for the user to figure it out.  */
+   complaints (rather than being interleaved with other messages).  */
 
 void
-clear_complaints (struct complaints **c, int less_verbose, int noisy)
+clear_complaints (struct complaints **c, int less_verbose)
 {
   struct complaints *complaints = get_complaints (c);
   struct complain *p;

diff --git a/gdb/complaints.h b/gdb/complaints.h
index 37648fb..652827e 100644 (file)
--- a/gdb/complaints.h
+++ b/gdb/complaints.h
@@ -56,7 +56,7 @@ extern void complaint_internal (struct complaints **complaints,
    enough context for the user to figure it out.  */
 
 extern void clear_complaints (struct complaints **complaints,
-                             int less_verbose, int noisy);
+                             int less_verbose);
 
 
 #endif /* !defined (COMPLAINTS_H) */

diff --git a/gdb/symfile.c b/gdb/symfile.c
index 7bc03fb..e48089a 100644 (file)
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -986,7 +986,7 @@ syms_from_objfile_1 (struct objfile *objfile,
      initial symbol reading for this file.  */
 
   (*objfile->sf->sym_init) (objfile);
-  clear_complaints (&symfile_complaints, 1, add_flags & SYMFILE_VERBOSE);
+  clear_complaints (&symfile_complaints, 1);
 
   (*objfile->sf->sym_offsets) (objfile, *addrs);
 
@@ -1033,7 +1033,7 @@ finish_new_objfile (struct objfile *objfile, symfile_add_flags add_flags)
     }
 
   /* We're done reading the symbol file; finish off complaints.  */
-  clear_complaints (&symfile_complaints, 0, add_flags & SYMFILE_VERBOSE);
+  clear_complaints (&symfile_complaints, 0);
 }
 
 /* Process a symbol file, as either the main file or as a dynamically
@@ -2457,7 +2457,7 @@ reread_symbols (void)
            }
 
          (*objfile->sf->sym_init) (objfile);
-         clear_complaints (&symfile_complaints, 1, 1);
+         clear_complaints (&symfile_complaints, 1);
 
          objfile->flags &= ~OBJF_PSYMTABS_READ;
 
@@ -2487,7 +2487,7 @@ reread_symbols (void)
            }
 
          /* We're done reading the symbol file; finish off complaints.  */
-         clear_complaints (&symfile_complaints, 0, 1);
+         clear_complaints (&symfile_complaints, 0);
 
          /* Getting new symbols may change our opinion about what is
             frameless.  */

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 5650461..e0ac116 100644 (file)
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,9 @@
 2018-05-23  Tom Tromey  <tom@tromey.com>
 
+       * gdb.gdb/complaints.exp (test_empty_complaints): Update.
+
+2018-05-23  Tom Tromey  <tom@tromey.com>
+
        * gdb.gdb/complaints.exp (test_serial_complaints): Remove.
        (test_short_complaints): Update.
 

diff --git a/gdb/testsuite/gdb.gdb/complaints.exp b/gdb/testsuite/gdb.gdb/complaints.exp
index 1d532b7..774ff89 100644 (file)
--- a/gdb/testsuite/gdb.gdb/complaints.exp
+++ b/gdb/testsuite/gdb.gdb/complaints.exp
@@ -88,7 +88,7 @@ proc test_initial_complaints { } {
 # For short complaints, all are the same
 
 proc test_short_complaints { } {
-    gdb_test_exact "call clear_complaints (&symfile_complaints, 1, 1)" "" "short start"
+    gdb_test_exact "call clear_complaints (&symfile_complaints, 1)" "" "short start"
 
     # Prime the system
     test_complaint \
@@ -131,14 +131,10 @@ proc test_empty_complaint { cmd msg } {
 
 proc test_empty_complaints { } {
 
-    test_empty_complaint "call clear_complaints(&symfile_complaints,0,0)" \
-           "empty non-verbose non-noisy clear"
-    test_empty_complaint "call clear_complaints(&symfile_complaints,1,0)" \
-           "empty verbose non-noisy clear"
-    test_empty_complaint "call clear_complaints(&symfile_complaints,1,1)" \
-           "empty verbose noisy clear"
-    test_empty_complaint "call clear_complaints(&symfile_complaints,0,1)" \
-           "empty non-verbose noisy clear"
+    test_empty_complaint "call clear_complaints(&symfile_complaints,0)" \
+           "empty non-verbose clear"
+    test_empty_complaint "call clear_complaints(&symfile_complaints,1)" \
+           "empty verbose clear"
 
     return 0
 }