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>