X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=gdb%2Fcomplaints.c;h=c5b5471ee181a5980265fdc0d83a7fdf1819e8be;hb=81491aa0968e5bcab0388914f909ccb12e084e32;hp=6c9d72849cb34346cbce5ee72ac8fde9c4568859;hpb=fba45db2faf619e71856ee38ec63949c0ef6903e;p=deliverable%2Fbinutils-gdb.git diff --git a/gdb/complaints.c b/gdb/complaints.c index 6c9d72849c..c5b5471ee1 100644 --- a/gdb/complaints.c +++ b/gdb/complaints.c @@ -1,11 +1,12 @@ /* Support for complaint handling during symbol reading in GDB. - Copyright (C) 1990, 1991, 1992 Free Software Foundation, Inc. + + Copyright (C) 1990-2019 Free Software Foundation, Inc. This file is part of GDB. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by - the Free Software Foundation; either version 2 of the License, or + the Free Software Foundation; either version 3 of the License, or (at your option) any later version. This program is distributed in the hope that it will be useful, @@ -14,142 +15,72 @@ GNU General Public License for more details. You should have received a copy of the GNU General Public License - along with this program; if not, write to the Free Software - Foundation, Inc., 59 Temple Place - Suite 330, - Boston, MA 02111-1307, USA. */ + along with this program. If not, see . */ #include "defs.h" #include "complaints.h" +#include "command.h" #include "gdbcmd.h" +#include -extern void _initialize_complaints (void); - -/* Structure to manage complaints about symbol file contents. */ - -struct complaint complaint_root[1] = -{ - { - (char *) NULL, /* Complaint message */ - 0, /* Complaint counter */ - complaint_root /* Next complaint. */ - } -}; - -/* How many complaints about a particular thing should be printed before - we stop whining about it? Default is no whining at all, since so many - systems have ill-constructed symbol files. */ - -static unsigned int stop_whining = 0; +/* Map format strings to counters. */ -/* Should each complaint be self explanatory, or should we assume that - a series of complaints is being produced? - case 0: self explanatory message. - case 1: First message of a series that must start off with explanation. - case 2: Subsequent message, when user already knows we are reading - symbols and we can just state our piece. */ +static std::unordered_map counters; -static int complaint_series = 0; +/* How many complaints about a particular thing should be printed + before we stop whining about it? Default is no whining at all, + since so many systems have ill-constructed symbol files. */ -/* External variables and functions referenced. */ +int stop_whining = 0; -extern int info_verbose; - - -/* Functions to handle complaints during symbol reading. */ - -/* Print a complaint about the input symbols, and link the complaint block - into a chain for later handling. */ +/* See complaints.h. */ void -complain (struct complaint *complaint,...) +complaint_internal (const char *fmt, ...) { va_list args; - va_start (args, complaint); - complaint->counter++; - if (complaint->next == NULL) - { - complaint->next = complaint_root->next; - complaint_root->next = complaint; - } - if (complaint->counter > stop_whining) - { - return; - } - wrap_here (""); + if (++counters[fmt] > stop_whining) + return; - switch (complaint_series + (info_verbose << 1)) - { + va_start (args, fmt); - /* Isolated messages, must be self-explanatory. */ - case 0: - begin_line (); - fputs_filtered ("During symbol reading, ", gdb_stderr); - wrap_here (""); - vfprintf_filtered (gdb_stderr, complaint->message, args); - fputs_filtered (".\n", gdb_stderr); - break; - - /* First of a series, without `set verbose'. */ - case 1: - begin_line (); - fputs_filtered ("During symbol reading...", gdb_stderr); - vfprintf_filtered (gdb_stderr, complaint->message, args); - fputs_filtered ("...", gdb_stderr); - wrap_here (""); - complaint_series++; - break; - - /* Subsequent messages of a series, or messages under `set verbose'. - (We'll already have produced a "Reading in symbols for XXX..." - message and will clean up at the end with a newline.) */ - default: - vfprintf_filtered (gdb_stderr, complaint->message, args); - fputs_filtered ("...", gdb_stderr); - wrap_here (""); + if (deprecated_warning_hook) + (*deprecated_warning_hook) (fmt, args); + else + { + fputs_filtered (_("During symbol reading: "), gdb_stderr); + vfprintf_filtered (gdb_stderr, fmt, args); + fputs_filtered ("\n", gdb_stderr); } - /* If GDB dumps core, we'd like to see the complaints first. Presumably - GDB will not be sending so many complaints that this becomes a - performance hog. */ - gdb_flush (gdb_stderr); + va_end (args); } -/* Clear out all complaint counters that have ever been incremented. - If sym_reading is 1, be less verbose about successive complaints, - since the messages are appearing all together during a command that - reads symbols (rather than scattered around as psymtabs get fleshed - out into symtabs at random times). If noisy is 1, we are in a - noisy symbol reading command, and our caller will print enough - context for the user to figure it out. */ +/* See complaints.h. */ void -clear_complaints (int sym_reading, int noisy) +clear_complaints () { - struct complaint *p; - - for (p = complaint_root->next; p != complaint_root; p = p->next) - { - p->counter = 0; - } - - if (!sym_reading && !noisy && complaint_series > 1) - { - /* Terminate previous series, since caller won't. */ - puts_filtered ("\n"); - } + counters.clear (); +} - complaint_series = sym_reading ? 1 + noisy : 0; +static void +complaints_show_value (struct ui_file *file, int from_tty, + struct cmd_list_element *cmd, const char *value) +{ + fprintf_filtered (file, _("Max number of complaints about incorrect" + " symbols is %s.\n"), + value); } void _initialize_complaints (void) { - add_show_from_set - (add_set_cmd ("complaints", class_support, var_zinteger, - (char *) &stop_whining, - "Set max number of complaints about incorrect symbols.", - &setlist), - &showlist); - + add_setshow_zinteger_cmd ("complaints", class_support, + &stop_whining, _("\ +Set max number of complaints about incorrect symbols."), _("\ +Show max number of complaints about incorrect symbols."), NULL, + NULL, complaints_show_value, + &setlist, &showlist); }