[libvirt] [PATCH v2 2/2] Added syntax-check rule for return with parentheses

Eric Blake eblake at redhat.com
Mon Mar 26 20:55:28 UTC 2012


On 03/26/2012 03:48 AM, Martin Kletzander wrote:
> 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.
>>>

>>
>> Looks good from my point of view, ACK. But we might need to
>> push these later after 0.9.11.

I see no problem with going for it now, as it is supposed to be no
semantic change; and the longer we wait, the more likely it is to have
conflicts when we rebase it later.

> 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.

I fixed that up, and pushed the series.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120326/1445251a/attachment-0001.sig>


More information about the libvir-list mailing list