diff options
| author | Cédric Le Goater | 2016-04-01 11:40:06 +0200 |
|---|---|---|
| committer | Paolo Bonzini | 2016-04-05 11:46:52 +0200 |
| commit | f0707d2e03186092f778269a9d22ec78481f12dc (patch) | |
| tree | f50266b82d3896950b7afcbf66c71e91ff73909e /scripts | |
| parent | target-i386: assert that KVM_GET/SET_MSRS can set all requested MSRs (diff) | |
| download | qemu-f0707d2e03186092f778269a9d22ec78481f12dc.tar.gz qemu-f0707d2e03186092f778269a9d22ec78481f12dc.tar.xz qemu-f0707d2e03186092f778269a9d22ec78481f12dc.zip | |
checkpatch: add target_ulong to typelist
In some occasions, a patch [1] can start with a hunk containing a
simple type cast. At the time annotate_values() is run, the type is
unknown and the cast type is misinterpreted as a identifier, resulting
in an error if it is followed with a negative value:
ERROR: spaces required around that '-' (ctx:WxV)
It seems complex to catch all possible types in a cast expression. So,
as a fallback solution, let's add some common qemu types to the
typeList array.
[1] http://lists.nongnu.org/archive/html/qemu-devel/2016-03/msg06741.html
Signed-off-by: Cédric Le Goater <clg@fr.ibm.com>
Message-Id: <1459503606-31603-1-git-send-email-clg@fr.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/checkpatch.pl | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index c26f76ed09..c9554ba644 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -212,6 +212,7 @@ our @typeList = ( qr{${Ident}_t}, qr{${Ident}_handler}, qr{${Ident}_handler_fn}, + qr{target_(?:u)?long}, ); # This can be modified by sub possible. Since it can be empty, be careful |