aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Henderson <richard.henderson@linaro.org>2022-04-17 11:29:57 -0700
committerRichard Henderson <richard.henderson@linaro.org>2022-04-20 10:51:11 -0700
commit3c06a41746fd0399a182839a6a25c79f1c4ee83a (patch)
treee6ef7b0e79043b84f6ee24cf7c0c41e641c70efe
parent095e9855b774b9da9b84c9cf21cbd856ae482670 (diff)
downloadqemu-3c06a41746fd0399a182839a6a25c79f1c4ee83a.zip
qemu-3c06a41746fd0399a182839a6a25c79f1c4ee83a.tar.gz
qemu-3c06a41746fd0399a182839a6a25c79f1c4ee83a.tar.bz2
util/log: Drop return value from qemu_log
The only user of this feature, tcg_dump_ops, has been converted to use fprintf directly. Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-Id: <20220417183019.755276-18-richard.henderson@linaro.org>
-rw-r--r--include/qemu/log-for-trace.h2
-rw-r--r--util/log.c13
2 files changed, 3 insertions, 12 deletions
diff --git a/include/qemu/log-for-trace.h b/include/qemu/log-for-trace.h
index 5e41517..d47c9cd 100644
--- a/include/qemu/log-for-trace.h
+++ b/include/qemu/log-for-trace.h
@@ -30,6 +30,6 @@ static inline bool qemu_loglevel_mask(int mask)
}
/* main logging function */
-int G_GNUC_PRINTF(1, 2) qemu_log(const char *fmt, ...);
+void G_GNUC_PRINTF(1, 2) qemu_log(const char *fmt, ...);
#endif
diff --git a/util/log.c b/util/log.c
index 090bc3b..2bd4dfb 100644
--- a/util/log.c
+++ b/util/log.c
@@ -59,26 +59,17 @@ void qemu_log_unlock(FILE *fd)
}
}
-/* Return the number of characters emitted. */
-int qemu_log(const char *fmt, ...)
+void qemu_log(const char *fmt, ...)
{
FILE *f = qemu_log_trylock();
- int ret = 0;
-
if (f) {
va_list ap;
va_start(ap, fmt);
- ret = vfprintf(f, fmt, ap);
+ vfprintf(f, fmt, ap);
va_end(ap);
qemu_log_unlock(f);
-
- /* Don't pass back error results. */
- if (ret < 0) {
- ret = 0;
- }
}
- return ret;
}
static void __attribute__((__constructor__)) qemu_logfile_init(void)