Commit f0707d2e authored by Cédric Le Goater's avatar Cédric Le Goater Committed by Paolo Bonzini

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.htmlSigned-off-by: 's avatarCédric Le Goater <clg@fr.ibm.com>
Message-Id: <1459503606-31603-1-git-send-email-clg@fr.ibm.com>
Signed-off-by: 's avatarPaolo Bonzini <pbonzini@redhat.com>
parent 48e1a45c
...@@ -212,6 +212,7 @@ our @typeList = ( ...@@ -212,6 +212,7 @@ our @typeList = (
qr{${Ident}_t}, qr{${Ident}_t},
qr{${Ident}_handler}, qr{${Ident}_handler},
qr{${Ident}_handler_fn}, qr{${Ident}_handler_fn},
qr{target_(?:u)?long},
); );
# This can be modified by sub possible. Since it can be empty, be careful # This can be modified by sub possible. Since it can be empty, be careful
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment