2003-08-04 David Carlton <carlton@kealia.com>
authorDavid Carlton <carlton@bactrian.org>
Mon, 4 Aug 2003 17:08:24 +0000 (17:08 +0000)
committerDavid Carlton <carlton@bactrian.org>
Mon, 4 Aug 2003 17:08:24 +0000 (17:08 +0000)
* charset.c (cached_iconv_convert): Add __FILE__ and __LINE__ args
to internal_error call.
* source.c (forward_search_command): Add "%s" format argument.
(reverse_search_command): Ditto.
* top.c (quit_confirm): Ditto.
* cli/cli-setshow.c (do_setshow_command): Ditto.
* cp-valprint.c (cp_print_class_method): Replace
{f,}printf_{un,}filtered by {f,}puts_{un,}filtered.
(cp_print_class_member): Ditto.
* event-top.c (command_line_handler): Ditto.
* linux-proc.c (linux_info_proc_cmd): Ditto.
* p-typeprint.c (pascal_type_print_base): Ditto.
* p-valprint.c (pascal_object_print_class_method): Ditto.
(pascal_object_print_class_member): Ditto.
* printcmd.c (print_scalar_formatted,printf_command): Ditto.
* remote.c (remote_cisco_section_offsets): Ditto.
* top.c (command_line_input): Ditto.
* utils.c (vwarning,error_stream,quit): Ditto.
* valprint.c (print_floating,print_binary_chars)
(print_octal_chars,print_decimal_chars,print_hex_chars): Ditto.

2003-08-04  David Carlton  <carlton@kealia.com>

* mi-main.c (mi_error_last_message): Add "%s" second argument to
xasprintf call.

2003-08-04  David Carlton  <carlton@kealia.com>

* generic/gdbtk.c (gdbtk_source_start_file): Add "%s" first
argument to error call.
(tk_command): Ditto.
(view_command): Ditto.

16 files changed:
gdb/ChangeLog
gdb/charset.c
gdb/cli/cli-setshow.c
gdb/cp-valprint.c
gdb/event-top.c
gdb/linux-proc.c
gdb/mi/ChangeLog
gdb/mi/mi-main.c
gdb/p-typeprint.c
gdb/p-valprint.c
gdb/printcmd.c
gdb/remote.c
gdb/source.c
gdb/top.c
gdb/utils.c
gdb/valprint.c

index de60e7daba9daff7a55e832f5c8d2a2ff292a2d7..8b7f2eaf854fbf42288da3764d59d90cf9738be4 100644 (file)
@@ -1,3 +1,26 @@
+2003-08-04  David Carlton  <carlton@kealia.com>
+
+       * charset.c (cached_iconv_convert): Add __FILE__ and __LINE__ args
+       to internal_error call.
+       * source.c (forward_search_command): Add "%s" format argument.
+       (reverse_search_command): Ditto.
+       * top.c (quit_confirm): Ditto.
+       * cli/cli-setshow.c (do_setshow_command): Ditto.
+       * cp-valprint.c (cp_print_class_method): Replace
+       {f,}printf_{un,}filtered by {f,}puts_{un,}filtered.
+       (cp_print_class_member): Ditto.
+       * event-top.c (command_line_handler): Ditto.
+       * linux-proc.c (linux_info_proc_cmd): Ditto.
+       * p-typeprint.c (pascal_type_print_base): Ditto.
+       * p-valprint.c (pascal_object_print_class_method): Ditto.
+       (pascal_object_print_class_member): Ditto.
+       * printcmd.c (print_scalar_formatted,printf_command): Ditto.
+       * remote.c (remote_cisco_section_offsets): Ditto.
+       * top.c (command_line_input): Ditto.
+       * utils.c (vwarning,error_stream,quit): Ditto.
+       * valprint.c (print_floating,print_binary_chars)
+       (print_octal_chars,print_decimal_chars,print_hex_chars): Ditto.
+
 2003-08-04  Andrew Cagney  <cagney@redhat.com>
 
        * frame.c (frame_func_unwind): Use frame_unwind_address_in_block.
index c311447ea4415bd85581a17da306df5fa3a0010e..fa683743cb7becdc63c477edac8c15c536b457d1 100644 (file)
@@ -443,7 +443,8 @@ cached_iconv_convert (struct cached_iconv *ci, int from_char, int *to_char)
         return 0;
 
       /* Anything else is mysterious.  */
-      internal_error ("Error converting character `%d' from `%s' to `%s' "
+      internal_error (__FILE__, __LINE__,
+                     "Error converting character `%d' from `%s' to `%s' "
                       "character set: %s",
                       from_char, ci->from->name, ci->to->name,
                       safe_strerror (errno));
index 1d68ae4aa1a69e2266838f118f5f1db15a279ed0..64a0d7353a2c9d7e7b904c1f84f75e0efe529cf8 100644 (file)
@@ -213,7 +213,7 @@ do_setshow_command (char *arg, int from_tty, struct cmd_list_element *c)
                    strcat (msg, c->enums[i]);
                  }
                strcat (msg, ".");
-               error (msg);
+               error ("%s", msg);
              }
 
            p = strchr (arg, ' ');
index a591d1859a5a3c5525f6ab59eb023ad94fbb7788..27d8ec327cebad672603f2e4bf96acc3a8bf4e32 100644 (file)
@@ -143,7 +143,7 @@ cp_print_class_method (char *valaddr,
       char *demangled_name;
 
       fprintf_filtered (stream, "&");
-      fprintf_filtered (stream, kind);
+      fputs_filtered (kind, stream);
       demangled_name = cplus_demangle (TYPE_FN_FIELD_PHYSNAME (f, j),
                                       DMGL_ANSI | DMGL_PARAMS);
       if (demangled_name == NULL)
@@ -720,7 +720,7 @@ cp_print_class_member (char *valaddr, struct type *domain,
   if (i < len)
     {
       char *name;
-      fprintf_filtered (stream, prefix);
+      fputs_filtered (prefix, stream);
       name = type_name_no_tag (domain);
       if (name)
        fputs_filtered (name, stream);
index 5ab324df758839b04c34b8f5284f51d22d2ac172..9db28fb38be4c768c946dc54ae241a05b68ac993 100644 (file)
@@ -612,7 +612,7 @@ command_line_handler (char *rl)
   if (annotation_level > 1 && instream == stdin)
     {
       printf_unfiltered ("\n\032\032post-");
-      printf_unfiltered (async_annotation_suffix);
+      puts_unfiltered (async_annotation_suffix);
       printf_unfiltered ("\n");
     }
 
index e4ea747008b80d9329f79aad501329b55cf18f31..282a3ec33fb09c67bdc98904e42c6cfe536628ab 100644 (file)
@@ -446,7 +446,7 @@ linux_info_proc_cmd (char *args, int from_tty)
       if ((procfile = fopen (fname1, "r")) > 0)
        {
          while (fgets (buffer, sizeof (buffer), procfile) != NULL)
-           printf_filtered (buffer);
+           puts_filtered (buffer);
          fclose (procfile);
        }
       else
index f4a4ad810dcd9f4c2c52c93f1c369adef90e62bd..5af17c3fc51b948c3d5ae53fa6fdf3ea0743cfdc 100644 (file)
@@ -1,3 +1,8 @@
+2003-08-04  David Carlton  <carlton@kealia.com>
+
+       * mi-main.c (mi_error_last_message): Add "%s" second argument to
+       xasprintf call.
+
 2003-06-28  Daniel Jacobowitz  <drow@mvista.com>
 
        * mi-out.c (mi_ui_out_impl): Add NULL for redirect member.
index 5e55913fb94a34ac514c8a117a4498b3eb6b62b4..64a28fd7f6990a6d1d9a4a72c0a4869efbcdb1de 100644 (file)
@@ -101,7 +101,7 @@ void
 mi_error_last_message (void)
 {
   char *s = error_last_message ();
-  xasprintf (&mi_error_message, s);
+  xasprintf (&mi_error_message, "%s", s);
   xfree (s);
 }
 
index 293de70b575c6ee2d3d31f800bbbb82855ae108d..286f22d95a571ca236bedc67361f8bd3f8ad387d 100644 (file)
@@ -460,8 +460,8 @@ pascal_type_print_base (struct type *type, struct ui_file *stream, int show,
   /* void pointer */
   if ((TYPE_CODE (type) == TYPE_CODE_PTR) && (TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_VOID))
     {
-      fprintf_filtered (stream,
-                       TYPE_NAME (type) ? TYPE_NAME (type) : "pointer");
+      fputs_filtered (TYPE_NAME (type) ? TYPE_NAME (type) : "pointer",
+                     stream);
       return;
     }
   /* When SHOW is zero or less, and there is a valid type name, then always
index 4867dcb47b63168e9c67d2a6150bc75cb767a2c4..718d28731d358fc9df519a4fa1ba0c446dcfa534 100644 (file)
@@ -661,7 +661,7 @@ common:
       char *demangled_name;
 
       fprintf_filtered (stream, "&");
-      fprintf_filtered (stream, kind);
+      fputs_filtered (kind, stream);
       demangled_name = cplus_demangle (TYPE_FN_FIELD_PHYSNAME (f, j),
                                       DMGL_ANSI | DMGL_PARAMS);
       if (demangled_name == NULL)
@@ -1080,7 +1080,7 @@ pascal_object_print_class_member (char *valaddr, struct type *domain,
   if (i < len)
     {
       char *name;
-      fprintf_filtered (stream, prefix);
+      fputs_filtered (prefix, stream);
       name = type_name_no_tag (domain);
       if (name)
        fputs_filtered (name, stream);
index 3f0bc3c21a8abf7cf331af4183c7fff8d0961bba..dd7f636621026b3e66669d037c332ce1867d37b3 100644 (file)
@@ -507,7 +507,7 @@ print_scalar_formatted (void *valaddr, struct type *type, int format, int size,
        strcpy (buf, local_binary_format_prefix ());
        strcat (buf, cp);
        strcat (buf, local_binary_format_suffix ());
-       fprintf_filtered (stream, buf);
+       fputs_filtered (buf, stream);
       }
       break;
 
@@ -2003,7 +2003,7 @@ printf_command (char *arg, int from_tty)
        current_substring += strlen (current_substring) + 1;
       }
     /* Print the portion of the format string after the last argument.  */
-    printf_filtered (last_arg);
+    puts_filtered (last_arg);
   }
   do_cleanups (old_cleanups);
 }
index 75e1e08ea37833e1a359432c1ca240771552648b..e6c4fbc44908167ab524a0e7b476f7058cf6d1a9 100644 (file)
@@ -2071,7 +2071,7 @@ remote_cisco_section_offsets (bfd_vma text_addr,
       sprintf_vma (tmp + strlen (tmp), data_addr);
       sprintf (tmp + strlen (tmp), " bss = 0x");
       sprintf_vma (tmp + strlen (tmp), bss_addr);
-      fprintf_filtered (gdb_stdlog, tmp);
+      fputs_filtered (tmp, gdb_stdlog);
       fprintf_filtered (gdb_stdlog,
                        "Reloc offset: text = 0x%s data = 0x%s bss = 0x%s\n",
                        paddr_nz (*text_offs),
index ae94addb6135f99e240cb197d12be94582f567eb..749617bb5781491926ab77fd5f8d54bf0efb8bbd 100644 (file)
@@ -1361,7 +1361,7 @@ forward_search_command (char *regex, int from_tty)
 
   msg = (char *) re_comp (regex);
   if (msg)
-    error (msg);
+    error ("%s", msg);
 
   if (current_source_symtab == 0)
     select_source_symtab (0);
@@ -1457,7 +1457,7 @@ reverse_search_command (char *regex, int from_tty)
 
   msg = (char *) re_comp (regex);
   if (msg)
-    error (msg);
+    error ("%s", msg);
 
   if (current_source_symtab == 0)
     select_source_symtab (0);
index 3efebecb3eca90ccf11b4a1b1334ad5025aeae66..e4f9642fcf4314927d9b0f091975dc3d7992b12f 100644 (file)
--- a/gdb/top.c
+++ b/gdb/top.c
@@ -1201,9 +1201,9 @@ command_line_input (char *prompt_arg, int repeat, char *annotation_suffix)
 
       if (annotation_level > 1 && instream == stdin)
        {
-         printf_unfiltered ("\n\032\032pre-");
-         printf_unfiltered (annotation_suffix);
-         printf_unfiltered ("\n");
+         puts_unfiltered ("\n\032\032pre-");
+         puts_unfiltered (annotation_suffix);
+         puts_unfiltered ("\n");
        }
 
       /* Don't use fancy stuff if not talking to stdin.  */
@@ -1222,9 +1222,9 @@ command_line_input (char *prompt_arg, int repeat, char *annotation_suffix)
 
       if (annotation_level > 1 && instream == stdin)
        {
-         printf_unfiltered ("\n\032\032post-");
-         printf_unfiltered (annotation_suffix);
-         printf_unfiltered ("\n");
+         puts_unfiltered ("\n\032\032post-");
+         puts_unfiltered (annotation_suffix);
+         puts_unfiltered ("\n");
        }
 
       if (!rl || rl == (char *) EOF)
@@ -1686,7 +1686,7 @@ quit_confirm (void)
       else
        s = "The program is running.  Exit anyway? ";
 
-      if (!query (s))
+      if (!query ("%s", s))
        return 0;
     }
 
index 4dfe89b6428661c1f6f3511202eb52ef23358519..a7c22eecc6fb843cd298ec53d969cc14ba0e77ff 100644 (file)
@@ -578,7 +578,7 @@ vwarning (const char *string, va_list args)
       wrap_here ("");          /* Force out any buffered output */
       gdb_flush (gdb_stdout);
       if (warning_pre_print)
-       fprintf_unfiltered (gdb_stderr, warning_pre_print);
+       fputs_unfiltered (warning_pre_print, gdb_stderr);
       vfprintf_unfiltered (gdb_stderr, string, args);
       fprintf_unfiltered (gdb_stderr, "\n");
       va_end (args);
@@ -644,7 +644,7 @@ error_stream (struct ui_file *stream)
   gdb_flush (gdb_stdout);
   annotate_error_begin ();
   if (error_pre_print)
-    fprintf_filtered (gdb_stderr, error_pre_print);
+    fputs_filtered (error_pre_print, gdb_stderr);
   ui_file_put (stream, do_write, gdb_stderr);
   fprintf_filtered (gdb_stderr, "\n");
 
@@ -922,7 +922,7 @@ quit (void)
 
   /* Don't use *_filtered; we don't want to prompt the user to continue.  */
   if (quit_pre_print)
-    fprintf_unfiltered (gdb_stderr, quit_pre_print);
+    fputs_unfiltered (quit_pre_print, gdb_stderr);
 
 #ifdef __MSDOS__
   /* No steenking SIGINT will ever be coming our way when the
index d4b8bf5faf758804ce4f9ee66b4ddf67e15b64d6..71e391a0908aaff484c2d8ceed331daf24b8f851 100644 (file)
@@ -452,9 +452,9 @@ print_floating (char *valaddr, struct type *type, struct ui_file *stream)
       if (floatformat_is_negative (fmt, valaddr))
        fprintf_filtered (stream, "-");
       fprintf_filtered (stream, "nan(");
-      fprintf_filtered (stream, local_hex_format_prefix ());
-      fprintf_filtered (stream, floatformat_mantissa (fmt, valaddr));
-      fprintf_filtered (stream, local_hex_format_suffix ());
+      fputs_filtered (local_hex_format_prefix (), stream);
+      fputs_filtered (floatformat_mantissa (fmt, valaddr), stream);
+      fputs_filtered (local_hex_format_suffix (), stream);
       fprintf_filtered (stream, ")");
       return;
     }
@@ -515,7 +515,7 @@ print_binary_chars (struct ui_file *stream, unsigned char *valaddr,
 
   /* FIXME: We should be not printing leading zeroes in most cases.  */
 
-  fprintf_filtered (stream, local_binary_format_prefix ());
+  fputs_filtered (local_binary_format_prefix (), stream);
   if (TARGET_BYTE_ORDER == BFD_ENDIAN_BIG)
     {
       for (p = valaddr;
@@ -553,7 +553,7 @@ print_binary_chars (struct ui_file *stream, unsigned char *valaddr,
            }
        }
     }
-  fprintf_filtered (stream, local_binary_format_suffix ());
+  fputs_filtered (local_binary_format_suffix (), stream);
 }
 
 /* VALADDR points to an integer of LEN bytes.
@@ -602,7 +602,7 @@ print_octal_chars (struct ui_file *stream, unsigned char *valaddr, unsigned len)
   cycle = (len * BITS_IN_BYTES) % BITS_IN_OCTAL;
   carry = 0;
 
-  fprintf_filtered (stream, local_octal_format_prefix ());
+  fputs_filtered (local_octal_format_prefix (), stream);
   if (TARGET_BYTE_ORDER == BFD_ENDIAN_BIG)
     {
       for (p = valaddr;
@@ -701,7 +701,7 @@ print_octal_chars (struct ui_file *stream, unsigned char *valaddr, unsigned len)
        }
     }
 
-  fprintf_filtered (stream, local_octal_format_suffix ());
+  fputs_filtered (local_octal_format_suffix (), stream);
 }
 
 /* VALADDR points to an integer of LEN bytes.
@@ -744,7 +744,7 @@ print_decimal_chars (struct ui_file *stream, unsigned char *valaddr,
       digits[i] = 0;
     }
 
-  fprintf_filtered (stream, local_decimal_format_prefix ());
+  fputs_filtered (local_decimal_format_prefix (), stream);
 
   /* Ok, we have an unknown number of bytes of data to be printed in
    * decimal.
@@ -841,7 +841,7 @@ print_decimal_chars (struct ui_file *stream, unsigned char *valaddr,
     }
   xfree (digits);
 
-  fprintf_filtered (stream, local_decimal_format_suffix ());
+  fputs_filtered (local_decimal_format_suffix (), stream);
 }
 
 /* VALADDR points to an integer of LEN bytes.  Print it in hex on stream.  */
@@ -853,7 +853,7 @@ print_hex_chars (struct ui_file *stream, unsigned char *valaddr, unsigned len)
 
   /* FIXME: We should be not printing leading zeroes in most cases.  */
 
-  fprintf_filtered (stream, local_hex_format_prefix ());
+  fputs_filtered (local_hex_format_prefix (), stream);
   if (TARGET_BYTE_ORDER == BFD_ENDIAN_BIG)
     {
       for (p = valaddr;
@@ -872,7 +872,7 @@ print_hex_chars (struct ui_file *stream, unsigned char *valaddr, unsigned len)
          fprintf_filtered (stream, "%02x", *p);
        }
     }
-  fprintf_filtered (stream, local_hex_format_suffix ());
+  fputs_filtered (local_hex_format_suffix (), stream);
 }
 
 /*  Called by various <lang>_val_print routines to print elements of an