Patchew URL: https://patchew.org/QEMU/20190910063724.28470-1-armbru@redhat.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [Qemu-devel] [PATCH v2 00/16] qapi: Schema language cleanups & doc improvements
Message-id: 20190910063724.28470-1-armbru@redhat.com
Type: series
=== 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/20190910063724.28470-1-armbru@redhat.com -> patchew/20190910063724.28470-1-armbru@redhat.com
Switched to a new branch 'test'
446351c qapi: Tweak code to match docs/devel/qapi-code-gen.txt
16bdba8 docs/devel/qapi-code-gen: Improve QAPI schema language doc
cb81f1d docs/devel/qapi-code-gen: Rewrite introduction to schema
1536401 docs/devel/qapi-code-gen: Rewrite compatibility considerations
4d82648 docs/devel/qapi-code-gen: Reorder sections for readability
9ac7ecd qapi: Adjust frontend errors to say enum value, not member
e106b86 qapi: Permit omitting all flat union branches
81dc6c5 qapi: Permit alternates with just one branch
bf905ed qapi: Permit 'boxed' with empty type
312a323 qapi: Drop support for escape sequences other than \\
2a77c2f qapi: Restrict strings to printable ASCII
f6aaeaa tests/qapi-schema: Demonstrate bad reporting of funny characters
bd4e9cc docs/devel/qapi-code-gen: Minor specification fixes
88cff11 qapi: Drop support for boxed alternate arguments
b9e08ee qapi: Drop check_type()'s redundant parameter @allow_optional
6860186 scripts/git.orderfile: Match QAPI schema more precisely
=== OUTPUT BEGIN ===
1/16 Checking commit 6860186b207b (scripts/git.orderfile: Match QAPI schema more precisely)
2/16 Checking commit b9e08eee03e3 (qapi: Drop check_type()'s redundant parameter @allow_optional)
3/16 Checking commit 88cff11d85d8 (qapi: Drop support for boxed alternate arguments)
4/16 Checking commit bd4e9cc1f433 (docs/devel/qapi-code-gen: Minor specification fixes)
5/16 Checking commit f6aaeaa74c63 (tests/qapi-schema: Demonstrate bad reporting of funny characters)
6/16 Checking commit 2a77c2f2e0ec (qapi: Restrict strings to printable ASCII)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#98:
new file mode 100644
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#115: FILE: tests/qapi-schema/string-code-point-127.json:2:
+{ 'command': '⌦' }
^
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#141: FILE: tests/qapi-schema/string-code-point-31.json:2:
+{ 'command': '␟' }
^
total: 2 errors, 1 warnings, 67 lines checked
Patch 6/16 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
7/16 Checking commit 312a323451bc (qapi: Drop support for escape sequences other than \\)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#69:
deleted file mode 100644
total: 0 errors, 1 warnings, 49 lines checked
Patch 7/16 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
8/16 Checking commit bf905eda09c9 (qapi: Permit 'boxed' with empty type)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#129:
deleted file mode 100644
total: 0 errors, 1 warnings, 129 lines checked
Patch 8/16 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
9/16 Checking commit 81dc6c516180 (qapi: Permit alternates with just one branch)
10/16 Checking commit e106b8638164 (qapi: Permit omitting all flat union branches)
11/16 Checking commit 9ac7ecd2cc24 (qapi: Adjust frontend errors to say enum value, not member)
12/16 Checking commit 4d8264892e00 (docs/devel/qapi-code-gen: Reorder sections for readability)
13/16 Checking commit 1536401f46a4 (docs/devel/qapi-code-gen: Rewrite compatibility considerations)
14/16 Checking commit cb81f1def7f2 (docs/devel/qapi-code-gen: Rewrite introduction to schema)
15/16 Checking commit 16bdba8b8274 (docs/devel/qapi-code-gen: Improve QAPI schema language doc)
16/16 Checking commit 446351ca29ad (qapi: Tweak code to match docs/devel/qapi-code-gen.txt)
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20190910063724.28470-1-armbru@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com