Re: [Qemu-devel] [PATCH] trace: Use fprintf_function (format checking)

[Anthony Liguori]

On 11/15/2010 02:17 PM, Stefan Weil wrote:

fprintf_function adds format checking with GCC_FMT_ATTR.

Cc: Blue Swirl<blauwir...@gmail.com>
Signed-off-by: Stefan Weil<w...@mail.berlios.de>

Applied.  Thanks.

Regards,

Anthony Liguori

---
  simpletrace.h |    6 +++---
  1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/simpletrace.h b/simpletrace.h
index 72614ec..2f44ed3 100644
--- a/simpletrace.h
+++ b/simpletrace.h
@@ -29,10 +29,10 @@ void trace3(TraceEventID event, uint64_t x1, uint64_t x2, 
uint64_t x3);
  void trace4(TraceEventID event, uint64_t x1, uint64_t x2, uint64_t x3, 
uint64_t x4);
  void trace5(TraceEventID event, uint64_t x1, uint64_t x2, uint64_t x3, 
uint64_t x4, uint64_t x5);
  void trace6(TraceEventID event, uint64_t x1, uint64_t x2, uint64_t x3, 
uint64_t x4, uint64_t x5, uint64_t x6);
-void st_print_trace(FILE *stream, int (*stream_printf)(FILE *stream, const 
char *fmt, ...));
-void st_print_trace_events(FILE *stream, int (*stream_printf)(FILE *stream, 
const char *fmt, ...));
+void st_print_trace(FILE *stream, fprintf_function stream_printf);
+void st_print_trace_events(FILE *stream, fprintf_function stream_printf);
  bool st_change_trace_event_state(const char *tname, bool tstate);
-void st_print_trace_file_status(FILE *stream, int (*stream_printf)(FILE 
*stream, const char *fmt, ...));
+void st_print_trace_file_status(FILE *stream, fprintf_function stream_printf);
  void st_set_trace_file_enabled(bool enable);
  bool st_set_trace_file(const char *file);
  void st_flush_trace_buffer(void);