Add ATTRIBUTE_PRINTF to trace_start_error
authorSimon Marchi <simon.marchi@ericsson.com>
Sat, 17 Jun 2017 21:19:25 +0000 (23:19 +0200)
committerSimon Marchi <simon.marchi@ericsson.com>
Sat, 17 Jun 2017 21:19:25 +0000 (23:19 +0200)
clang complains that the fmt passed to vwarning in trace_start_error is
not a literal.  This looks like a fair warning, which can be removed by
adding ATTRIBUTE_PRINTF to the declaration of trace_start_error.

gdb/ChangeLog:

* nat/fork-inferior.h (trace_start_error): Add ATTRIBUTE_PRINTF.

gdb/ChangeLog
gdb/nat/fork-inferior.h

index a08c781..85ee3cf 100644 (file)
@@ -1,5 +1,9 @@
 2017-06-17  Simon Marchi  <simon.marchi@ericsson.com>
 
+       * nat/fork-inferior.h (trace_start_error): Add ATTRIBUTE_PRINTF.
+
+2017-06-17  Simon Marchi  <simon.marchi@ericsson.com>
+
        * configure: Re-generate.
        * warning.m4 (build_warnings): Add -Wno-mismatched-tags.
 
index 10e3832..d369cff 100644 (file)
@@ -95,7 +95,7 @@ extern void gdb_flush_out_err ();
    (i.e., when the "traceme_fun" callback is called on fork_inferior)
    and bail out.  This function does not return.  */
 extern void trace_start_error (const char *fmt, ...)
-  ATTRIBUTE_NORETURN;
+  ATTRIBUTE_NORETURN ATTRIBUTE_PRINTF (1, 2);
 
 /* Like "trace_start_error", but the error message is constructed by
    combining STRING with the system error message for errno.  This