Re: [PULL 0/9] Fuzzer, configure, gitlab and misc patches

[no-reply]

Patchew URL: 20200713105534.10872-1-thuth@redhat.com/">https://patchew.org/QEMU/20200713105534.10872-1-thuth@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-thuth@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
20200713105534.10872-1-thuth@redhat.com/testing.checkpatch/?type=message">http://patchew.org/logs/20200713105534.10872-1-thuth@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com