[libvirt] [PATCH] maint: simplify some syntax check exemptions

[Eric Blake]

On 07/23/2014 12:13 AM, Martin Kletzander wrote:
> On Fri, Jul 18, 2014 at 03:23:51PM -0600, Eric Blake wrote:
>> Commit 5028160 accidentally weakened the strtol prohibitions to
>> skip ALL files under src/util instead of the former situation of
>> just protecting util/virsexpr.c; even though NONE of the files
>> in that directory need any protection.
>>
>> Shorten some long lines while at it.
>>
>> * cfg.mk (exclude_file_name_regexp--sc_prohibit_strtol): No need
>> to exclude all of util.
>> (exclude_file_name_regexp--sc_prohibit_sprintf): Reduce long line.
>> (exclude_file_name_regexp--sc_prohibit_raw_allocation): Likewise.


>>
>> +  ^(docs/hacking\.html\.in|.*stp|.pl)$$
>>
> 
> This fails for me even though the regexp makes sense.  Either (a)
> adding an asterisk before 'pl' (the same way as for
> '.*stp') or (b) encapsulating the caret in the parentheses should
> work, probably one of those you meant to do.

Oops, serves me right for testing one version, then tweaking, then
posting, without retesting.

> 
> ACK with that changed,

Fixed and pushed.

-- 
Eric Blake   eblake 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: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140723/58300928/attachment-0001.sig>