On Mon, Aug 12, 2019 at 01:54:18PM +0200, Michal Privoznik wrote:
Similarly to the previous commit, VIR_TEST_VERBOSE should put
'\n' at the end of each call so that the output is not broken.

Signed-off-by: Michal Privoznik <mpriv...@redhat.com>
---
tests/cputest.c                  | 14 +++---
tests/qemublocktest.c            | 10 ++---
tests/qemuhotplugtest.c          | 10 ++---
tests/qemumonitorjsontest.c      |  8 ++--
tests/securityselinuxlabeltest.c |  4 +-
tests/testutils.h                |  4 +-
tests/testutilsqemu.c            |  9 ++--
tests/testutilsqemuschema.c      |  8 ++--
tests/virerrortest.c             | 12 +++---
tests/virhashtest.c              | 18 ++++----
tests/virjsontest.c              | 74 ++++++++++++++++----------------
11 files changed, 87 insertions(+), 84 deletions(-)

diff --git a/tests/testutils.h b/tests/testutils.h
index cfc60084b2..2d0cea7826 100644
--- a/tests/testutils.h
+++ b/tests/testutils.h
@@ -97,8 +97,10 @@ unsigned int virTestGetRegenerate(void);

#define VIR_TEST_VERBOSE(...) \
    do { \
-        if (virTestGetVerbose()) \
+        if (virTestGetVerbose()) { \
            fprintf(stderr, __VA_ARGS__); \
+            fprintf(stderr, "\n"); \
+        } \
    } while (0)

char *virTestLogContentAndReset(void);

This macro is not used enough to reach the limit, but if you make the
change in the first patch, please change this one as well.

Reviewed-by: Ján Tomko <jto...@redhat.com>

Jano

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to