[libvirt] [PATCH 3/3] scripts: check-aclrules: use regular expressions less often
Erik Skultety
eskultet at redhat.com
Fri Nov 22 08:00:12 UTC 2019
On Wed, Nov 20, 2019 at 07:32:46PM +0100, Ján Tomko wrote:
> Use a simple
> if "substr" in line
> before running a regular expression, which saves time,
> especially if the regex has a capture group.
>
> This reduces runtime of the check by almost 78 % for me.
>
> Signed-off-by: Ján Tomko <jtomko at redhat.com>
> ---
> scripts/check-aclrules.py | 26 +++++++++++++++++++-------
> 1 file changed, 19 insertions(+), 7 deletions(-)
...
> # every func listed there, has an impl which calls
> # an ACL function
> if intable:
> - assign = re.search(r'''\.(\w+)\s*=\s*(\w+),?''', line)
> + assign = None
> + if '"' in line:
^This doesn't quite work, does it? I don't see any '"' in the driver API
mapping structures. The only reliable pattern I see we could use is ', /* '
but in order to use that, we'd have to unify a few source files, mainly
vz_driver.c, doing:
git grep -Enpi ", \s+/\* " | grep driver.c
should help identifying them.
With this one addressed:
Reviewed-by: Erik Skultety <eskultet at redhat.com>
More information about the libvir-list
mailing list