Re: [PATCH] tests: remove double space

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 17, 2018 at 02:02:54PM +0200, Ján Tomko wrote:
Commit 70255fa5 introduced a double space before the newline-escaping
backslash and broke syntax check.

Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx>
---
Pushed as a trivial build-breaker fix.

tests/qemuxml2argvtest.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


Thanks for the fix, I confirmed *again* that I have `c-auto-align-backslashes`
set to `nil`, so this should not happen more than like last three or four times.

diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
index 43f07efc1..509fe776e 100644
--- a/tests/qemuxml2argvtest.c
+++ b/tests/qemuxml2argvtest.c
@@ -668,7 +668,7 @@ mymain(void)
                       testCompareXMLToArgv, &info) < 0) \
            ret = -1; \
        if (virTestRun("QEMU XML-2-startup-XML " name, \
-                       testCompareXMLToStartupXML, &info) < 0)  \
+                       testCompareXMLToStartupXML, &info) < 0) \
            ret = -1; \
        virObjectUnref(info.qemuCaps); \
        virObjectUnref(info.vm); \
--
2.16.1

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

Attachment: signature.asc
Description: Digital signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux