diff options
| author | Richard Henderson | 2022-04-17 20:29:43 +0200 |
|---|---|---|
| committer | Richard Henderson | 2022-04-20 19:51:11 +0200 |
| commit | e2c7c6a454c2c4221461aa04e4061dbb91b4986c (patch) | |
| tree | e60843c94e954127b0a92a41099c97a840750fa4 /include | |
| parent | target/hexagon: Remove qemu_set_log in hexagon_translate_init (diff) | |
| download | qemu-e2c7c6a454c2c4221461aa04e4061dbb91b4986c.tar.gz qemu-e2c7c6a454c2c4221461aa04e4061dbb91b4986c.tar.xz qemu-e2c7c6a454c2c4221461aa04e4061dbb91b4986c.zip | |
util/log: Return bool from qemu_set_log_filename
Per the recommendations in qapi/error.h, return false on failure.
Use the return value in the monitor, the only place we aren't
already passing error_fatal or error_abort.
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220417183019.755276-4-richard.henderson@linaro.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/qemu/log.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/qemu/log.h b/include/qemu/log.h index ed59ebd4a2..fabfbe41da 100644 --- a/include/qemu/log.h +++ b/include/qemu/log.h @@ -148,7 +148,7 @@ typedef struct QEMULogItem { extern const QEMULogItem qemu_log_items[]; void qemu_set_log(int log_flags); -void qemu_set_log_filename(const char *filename, Error **errp); +bool qemu_set_log_filename(const char *filename, Error **errp); void qemu_set_dfilter_ranges(const char *ranges, Error **errp); bool qemu_log_in_addr_range(uint64_t addr); int qemu_str_to_log_mask(const char *str); |