Re: [Qemu-devel] [PATCH v2 00/16] qapi: Schema language cleanups & doc improvements

[no-reply]

Patchew URL: https://patchew.org/QEMU/address@hidden/



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: address@hidden
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/address@hidden -> patchew/address@hidden
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/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden