aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@ericsson.com>2017-06-17 23:19:25 +0200
committerSimon Marchi <simon.marchi@ericsson.com>2017-06-17 23:19:25 +0200
commit1d4fbac99e05c2d2ea98984f9bc6f7d7f1a183b7 (patch)
tree4954bca031046a569bf8a6535c7c7652a9b9e931
parentae3e2ccfe75e5c62749c15ead5c95e052aad21ca (diff)
downloadgdb-1d4fbac99e05c2d2ea98984f9bc6f7d7f1a183b7.zip
gdb-1d4fbac99e05c2d2ea98984f9bc6f7d7f1a183b7.tar.gz
gdb-1d4fbac99e05c2d2ea98984f9bc6f7d7f1a183b7.tar.bz2
Add ATTRIBUTE_PRINTF to trace_start_error
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.
-rw-r--r--gdb/ChangeLog4
-rw-r--r--gdb/nat/fork-inferior.h2
2 files changed, 5 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index a08c781..85ee3cf 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -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.
diff --git a/gdb/nat/fork-inferior.h b/gdb/nat/fork-inferior.h
index 10e3832..d369cff 100644
--- a/gdb/nat/fork-inferior.h
+++ b/gdb/nat/fork-inferior.h
@@ -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