Re: [PATCH 2/2] Add syntax-check rule to look for 'function (...args...)' violations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/17/2012 11:31 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
> 
> Checking for 'function (...args...)' is quite difficult since it
> needs to ignore valid usage with keywords like 'if (...test...)'
> and while/for/switch. It must also ignore source comments and
> quoted strings.
> 
> It is not possible todo this with a simple regex in the normal
> syntax-check style. So a short Perl script is created instead
> to analyse the source. In practice this works well enough. The
> only thing it can't cope with is multi-line quoted strings of
> the form
> 
>  "start of string\
> more lines\
> more line\
> the end"
> 
> but this can and should be written as
> 
>  "start of string"
>  "more lines"
>  "more line"
>  "the end"
> 
> with this simple change, the bracket checking script does not
> have any false positives across libvirt source, provided it

Unfortunately there's a false negative, see my comment to other patch.

> is only run against .c files.

Why not .h files?

> ---
>  build-aux/bracket-spacing.pl | 75 ++++++++++++++++++++++++++++++++++++++++++++
>  cfg.mk                       |  7 ++++-
>  2 files changed, 81 insertions(+), 1 deletion(-)
>  create mode 100755 build-aux/bracket-spacing.pl
> 

Sorry, I really don't feel competent enough to review the perl part,
since the last perl line I wrote was more than 5 years ago.

Martin

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]