[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 13/17] qemu-print: New qemu_fprintf(), qemu_vfpr
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 13/17] qemu-print: New qemu_fprintf(), qemu_vfprintf() |
Date: |
Fri, 12 Apr 2019 18:53:48 +0100 |
User-agent: |
Mutt/1.11.4 (2019-03-13) |
* Markus Armbruster (address@hidden) wrote:
> Code that doesn't want to know about current monitor vs. stdout
> vs. stderr takes an fprintf_function callback and a FILE * argument to
> pass to it. Actual arguments are either fprintf() and stdout or
> stderr, or monitor_fprintf() and the current monitor cast to FILE *.
> monitor_fprintf() casts it right back, and is otherwise identical to
> monitor_printf(). The type-punning is ugly.
>
> New qemu_fprintf() and qemu_vprintf() address this need without type
> punning: they are like fprintf() and vfprintf(), except they print to
> the current monitor when passed a null FILE *. The next commits will
> put them to use.
>
> Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
> ---
> include/qemu/qemu-print.h | 4 ++++
> util/qemu-print.c | 27 +++++++++++++++++++++++++++
> 2 files changed, 31 insertions(+)
>
> diff --git a/include/qemu/qemu-print.h b/include/qemu/qemu-print.h
> index 8fed32bf42..40b596262f 100644
> --- a/include/qemu/qemu-print.h
> +++ b/include/qemu/qemu-print.h
> @@ -16,4 +16,8 @@
> int qemu_vprintf(const char *fmt, va_list ap) GCC_FMT_ATTR(1, 0);
> int qemu_printf(const char *fmt, ...) GCC_FMT_ATTR(1, 2);
>
> +int qemu_vfprintf(FILE *stream, const char *fmt, va_list ap)
> + GCC_FMT_ATTR(2, 0);
> +int qemu_fprintf(FILE *stream, const char *fmt, ...) GCC_FMT_ATTR(2, 3);
> +
> #endif
> diff --git a/util/qemu-print.c b/util/qemu-print.c
> index 86f9417af8..e79d6b8396 100644
> --- a/util/qemu-print.c
> +++ b/util/qemu-print.c
> @@ -40,3 +40,30 @@ int qemu_printf(const char *fmt, ...)
> va_end(ap);
> return ret;
> }
> +
> +/*
> + * Print like vfprintf()
> + * Print to @stream if non-null, else to current monitor.
> + */
> +int qemu_vfprintf(FILE *stream, const char *fmt, va_list ap)
> +{
> + if (!stream) {
> + return monitor_vprintf(cur_mon, fmt, ap);
> + }
> + return vfprintf(stream, fmt, ap);
> +}
> +
> +/*
> + * Print like fprintf().
> + * Print to @stream if non-null, else to current monitor.
> + */
> +int qemu_fprintf(FILE *stream, const char *fmt, ...)
> +{
> + va_list ap;
> + int ret;
> +
> + va_start(ap, fmt);
> + ret = qemu_vfprintf(stream, fmt, ap);
> + va_end(ap);
> + return ret;
> +}
> --
> 2.17.2
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [Qemu-devel] [PATCH 00/17] Clean up and simplify around fprintf_function, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 02/17] trace: Simplify how st_print_trace_file_status() prints, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 03/17] tcg: Simplify how dump_opcount_info() prints, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 05/17] tcg: Simplify how dump_drift_info() prints, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 01/17] include: Include fprintf-fn.h only where needed, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 13/17] qemu-print: New qemu_fprintf(), qemu_vfprintf(), Markus Armbruster, 2019/04/11
- Re: [Qemu-devel] [PATCH 13/17] qemu-print: New qemu_fprintf(), qemu_vfprintf(),
Dr. David Alan Gilbert <=
- [Qemu-devel] [PATCH 04/17] tcg: Simplify how dump_exec_info() prints, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 06/17] qsp: Simplify how qsp_report() prints, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 12/17] qom/cpu: Simplify how CPUClass::dump_statistics() prints, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 11/17] target/i386: Simplify how x86_cpu_dump_local_apic_state() prints, Markus Armbruster, 2019/04/11
- [Qemu-devel] [PATCH 15/17] monitor: Clean up how monitor_disas() funnels output to monitor, Markus Armbruster, 2019/04/11