diff options
author | Richard Henderson <richard.henderson@linaro.org> | 2022-04-17 11:29:57 -0700 |
---|---|---|
committer | Richard Henderson <richard.henderson@linaro.org> | 2022-04-20 10:51:11 -0700 |
commit | 3c06a41746fd0399a182839a6a25c79f1c4ee83a (patch) | |
tree | e6ef7b0e79043b84f6ee24cf7c0c41e641c70efe | |
parent | 095e9855b774b9da9b84c9cf21cbd856ae482670 (diff) | |
download | qemu-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.h | 2 | ||||
-rw-r--r-- | util/log.c | 13 |
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 @@ -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) |