[libvirt] [PATCH v2 2/2] Added syntax-check rule for return with parentheses
Martin Kletzander
mkletzan at redhat.com
Mon Mar 26 09:48:46 UTC 2012
On 03/26/2012 05:43 AM, Osier Yang wrote:
> On 2012年03月23日 15:34, Martin Kletzander wrote:
>> After cleanup introduced with previous commit, there is a need for
>> syntax-check rule taking care of return(). Regexp used in 'prohibit'
>> parameter is taken from the cleanup commit and modified so it fits
>> 'grep -E' format. Semicolon at the end is needed, otherwise the regexp
>> could match return with cast.
>>
>> Exception is created for python source files because we don't have any
>> documentation restricting the use of return that matches this case.
>> ---
>> cfg.mk | 8 ++++++++
>> 1 files changed, 8 insertions(+), 0 deletions(-)
>>
>> diff --git a/cfg.mk b/cfg.mk
>> index 24e6a69..eae629b 100644
>> --- a/cfg.mk
>> +++ b/cfg.mk
>> @@ -469,6 +469,12 @@ sc_prohibit_xmlURI:
>> halt='use virURI(Parse|Format), not xml(ParseURI|SaveUri)' \
>> $(_sc_search_regexp)
>>
>> +# we don't want old old-style return with parentheses around argument
>> +sc_prohibit_return_as_function:
>> + @prohibit='\<return *\(([^()]*(\([^()]*\)[^()]*)*)\) *;' \
>> + halt='avoid extra () with return statements' \
>> + $(_sc_search_regexp)
>> +
>> # ATTRIBUTE_UNUSED should only be applied in implementations, not
>> # header declarations
>> sc_avoid_attribute_unused_in_header:
>> @@ -791,6 +797,8 @@ exclude_file_name_regexp--sc_prohibit_xmlGetProp =
>> ^src/util/xml\.c$$
>>
>> exclude_file_name_regexp--sc_prohibit_xmlURI = ^src/util/viruri\.c$$
>>
>> +exclude_file_name_regexp--sc_prohibit_return_as_function = ^\.py$$
>> +
>> exclude_file_name_regexp--sc_require_config_h = ^examples/
>>
>> exclude_file_name_regexp--sc_require_config_h_first = ^examples/
>> --
>> 1.7.3.4
>
> Looks good from my point of view, ACK. But we might need to
> push these later after 0.9.11.
>
> Regards,
> Osier
>
We've totally missed the '^' in the exception, that shouldn't be there,
so either please fix this before pushing it or send me NACK or ping me
on IRC so I know if I should send fixed version, thanks and sorry for
the trouble.
Martin
More information about the libvir-list
mailing list