Patchew URL: https://patchew.org/QEMU/20200713105534.10872-1-th...@redhat.com/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200713105534.10872-1-th...@redhat.com Subject: [PULL 0/9] Fuzzer, configure, gitlab and misc patches === 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 === Switched to a new branch 'test' 2560d08 docs/system/s390x: Improve the 3270 documentation bee82e9 GitLab Gating CI: introduce pipeline-status contrib script 141985e disas/sh4: Add missing fallthrough annotations f9d2388 Remove the CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE switch 06ad69e docs/devel/fuzzing: Fix bugs in documentation 70c34cf tests/qtest/fuzz: Add missing spaces in description 0146291 fuzz: add missing header for rcu_enable_atfork a605351 configure: do not clobber CFLAGS with --enable-fuzzing 2f2ba22 configure: fix malloc check === OUTPUT BEGIN === 1/9 Checking commit 2f2ba221d020 (configure: fix malloc check) 2/9 Checking commit a6053515ccce (configure: do not clobber CFLAGS with --enable-fuzzing) 3/9 Checking commit 0146291d22a7 (fuzz: add missing header for rcu_enable_atfork) 4/9 Checking commit 70c34cfa0e99 (tests/qtest/fuzz: Add missing spaces in description) 5/9 Checking commit 06ad69e9998f (docs/devel/fuzzing: Fix bugs in documentation) 6/9 Checking commit f9d238881217 (Remove the CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE switch) WARNING: architecture specific defines should be avoided #165: FILE: util/coroutine-ucontext.c:255: +#if !defined(__clang__) WARNING: architecture specific defines should be avoided #174: FILE: util/coroutine-ucontext.c:263: +#if !defined(__clang__) total: 0 errors, 2 warnings, 126 lines checked Patch 6/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/9 Checking commit 141985ef98c0 (disas/sh4: Add missing fallthrough annotations) ERROR: code indent should never use tabs #25: FILE: disas/sh4.c:1966: +^I /* fallthrough */$ ERROR: code indent should never use tabs #33: FILE: disas/sh4.c:1976: +^I /* fallthrough */$ total: 2 errors, 0 warnings, 14 lines checked Patch 7/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 8/9 Checking commit bee82e92f2ee (GitLab Gating CI: introduce pipeline-status contrib script) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #24: new file mode 100755 total: 0 errors, 1 warnings, 157 lines checked Patch 8/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 9/9 Checking commit 2560d0831597 (docs/system/s390x: Improve the 3270 documentation) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200713105534.10872-1-th...@redhat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com
