marcandre.lur...@redhat.com writes:
> From: Marc-André Lureau <marcandre.lur...@redhat.com>
>
> Let's use a more explicit variable "detailed" instead of calling
> monitor_cur() multiple times.
>
> Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> ---
> util/error-report.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/util/error-report.c b/util/error-report.c
> index 98f242b75bbf..893da10f19bc 100644
> --- a/util/error-report.c
> +++ b/util/error-report.c
> @@ -195,16 +195,17 @@ real_time_iso8601(void)
> */
> static void vreport(report_type type, const char *fmt, va_list ap)
> {
> + bool detailed = !monitor_cur();
> gchar *timestr;
>
> - if (message_with_timestamp && !monitor_cur()) {
> + if (message_with_timestamp && detailed) {
> timestr = real_time_iso8601();
> error_printf("%s ", timestr);
> g_free(timestr);
> }
>
> /* Only prepend guest name if -msg guest-name and -name guest=... are
> set */
> - if (error_with_guestname && error_guest_name && !monitor_cur()) {
> + if (error_with_guestname && error_guest_name && detailed) {
> error_printf("%s ", error_guest_name);
> }
Reviewed-by: Markus Armbruster <arm...@redhat.com>
