[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 0/4] yank: Add chardev tests and fixes
From: |
no-reply |
Subject: |
Re: [PATCH v6 0/4] yank: Add chardev tests and fixes |
Date: |
Mon, 29 Mar 2021 10:09:47 -0700 (PDT) |
Patchew URL: cover.1617035720.git.lukasstraub2@web.de/">https://patchew.org/QEMU/cover.1617035720.git.lukasstraub2@web.de/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: cover.1617035720.git.lukasstraub2@web.de
Subject: [PATCH v6 0/4] yank: Add chardev tests and fixes
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
- [tag update] patchew/20210329132632.68901-1-mreitz@redhat.com ->
patchew/20210329132632.68901-1-mreitz@redhat.com
- [tag update] patchew/cover.1616570702.git.viresh.kumar@linaro.org ->
patchew/cover.1616570702.git.viresh.kumar@linaro.org
* [new tag] patchew/cover.1617035720.git.lukasstraub2@web.de ->
patchew/cover.1617035720.git.lukasstraub2@web.de
Switched to a new branch 'test'
2f8555b tests: Add tests for yank with the chardev-change case
df55f5e chardev: Fix yank with the chardev-change case
ac94aea chardev/char.c: Always pass id to chardev_new
545c6dc chardev/char.c: Move object_property_try_add_child out of chardev_new
=== OUTPUT BEGIN ===
1/4 Checking commit 545c6dcdbf05 (chardev/char.c: Move
object_property_try_add_child out of chardev_new)
2/4 Checking commit ac94aea1496c (chardev/char.c: Always pass id to chardev_new)
3/4 Checking commit df55f5e93c68 (chardev: Fix yank with the chardev-change
case)
4/4 Checking commit 2f8555b5dc94 (tests: Add tests for yank with the
chardev-change case)
ERROR: Macros with multiple statements should be enclosed in a do - while loop
#231: FILE: tests/unit/test-yank.c:180:
+#define CHAR_CHANGE_TEST(name, _old_yank, _new_yank)
\
+ g_test_add_data_func("/yank/char_change/success/" # name,
\
+ &(CharChangeTestConfig) { .addr = &tcpaddr,
\
+ .old_yank =
(_old_yank),\
+ .new_yank =
(_new_yank),\
+ .fail = false },
\
+ char_change_test);
\
+ g_test_add_data_func("/yank/char_change/fail/" # name,
\
+ &(CharChangeTestConfig) { .addr = &tcpaddr,
\
+ .old_yank =
(_old_yank),\
+ .new_yank =
(_new_yank),\
+ .fail = true },
\
+ char_change_test);
total: 1 errors, 0 warnings, 216 lines checked
Patch 4/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
cover.1617035720.git.lukasstraub2@web.de/testing.checkpatch/?type=message">http://patchew.org/logs/cover.1617035720.git.lukasstraub2@web.de/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
- [PATCH v6 0/4] yank: Add chardev tests and fixes, Lukas Straub, 2021/03/29
- [PATCH v6 1/4] chardev/char.c: Move object_property_try_add_child out of chardev_new, Lukas Straub, 2021/03/29
- [PATCH v6 2/4] chardev/char.c: Always pass id to chardev_new, Lukas Straub, 2021/03/29
- [PATCH v6 3/4] chardev: Fix yank with the chardev-change case, Lukas Straub, 2021/03/29
- [PATCH v6 4/4] tests: Add tests for yank with the chardev-change case, Lukas Straub, 2021/03/29
- Re: [PATCH v6 0/4] yank: Add chardev tests and fixes,
no-reply <=
- Re: [PATCH v6 0/4] yank: Add chardev tests and fixes, Marc-André Lureau, 2021/03/29