[PATCH 0/9] syntax-check: make spacing check faster

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

 



Simplify some regexes to increase coverage and speed up the execution.

Reduces (git ls-files "*.c" | xargs build-aux/bracket-spacing.pl) by
half a second (from 1.9s to 1.33s), the effect on complete syntax-check
is only ~150 ms.

Ján Tomko (9):
  vbox: reformat multi-line error reports
  maint: remove whitespace from closing parentheses
  Rename virAssertCmpInt to testAssertEq
  Rename bracket-spacing.pl to check-spacing.pl
  check-spacing: rewrite whitespace check before (semi)colon
  check-spacing: rewrite regex for checking the closing parenthesis
  check-spacing: simplify keyword spacing check
  check-spacing: remove virAssertCmpInt exception
  check-spacing: use non-capturing groups

 build-aux/bracket-spacing.pl  | 204 ------------------------------------------
 build-aux/check-spacing.pl    | 204 ++++++++++++++++++++++++++++++++++++++++++
 cfg.mk                        |   6 +-
 src/qemu/qemu_command.c       |   3 +-
 src/vbox/vbox_common.c        |   8 +-
 src/vbox/vbox_snapshot_conf.c |  68 +++++++-------
 src/vz/vz_driver.c            |   3 +-
 src/vz/vz_sdk.c               |   3 +-
 tests/viratomictest.c         |  52 +++++------
 9 files changed, 274 insertions(+), 277 deletions(-)
 delete mode 100755 build-aux/bracket-spacing.pl
 create mode 100755 build-aux/check-spacing.pl

-- 
2.7.3

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