On 08/03/2018 20:02, Ian Jackson wrote:
> This avoids checkpatch misparsing (as statements) long function
> definitions or declarations, which sometimes start with constructs
> like this:
>
> static inline int xendevicemodel_relocate_memory(
> xendevicemodel_handle *dmod, domid_t domid, ...
>
> CC: Eric Blake <ebl...@redhat.com>
> CC: Paolo Bonzini <pbonz...@redhat.com>
> CC: Daniel P. Berrange <berra...@redhat.com>
> Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com>
> ---
> v6.1: New patch
> ---
> scripts/checkpatch.pl | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index d1fe79b..3e488f7 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -266,6 +266,7 @@ our @typeList = (
> qr{target_(?:u)?long},
> qr{hwaddr},
> qr{xml${Ident}},
> + qr{xendevicemodel_handle},
> );
>
> # This can be modified by sub possible. Since it can be empty, be careful
> --
Or just rename it so that it is CamelCase. Then checkpatch will be happy.
Paolo
