[ovs-git] [openvswitch/ovs] 6ed806: checkpatch: introduce constants for the parse stat...

GitHub noreply at github.com
Sun Apr 1 16:37:28 UTC 2018


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 6ed80686247199eb120778f8e2dc7c5165cb3067
      https://github.com/openvswitch/ovs/commit/6ed80686247199eb120778f8e2dc7c5165cb3067
  Author: Aaron Conole <aconole at redhat.com>
  Date:   2018-04-01 (Sun, 01 Apr 2018)

  Changed paths:
    M utilities/checkpatch.py

  Log Message:
  -----------
  checkpatch: introduce constants for the parse states

It's just easier to read.  Should be no functional change.

Signed-off-by: Aaron Conole <aconole at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: 8361e647399b5e64de7755aa55477381e37c4619
      https://github.com/openvswitch/ovs/commit/8361e647399b5e64de7755aa55477381e37c4619
  Author: Aaron Conole <aconole at redhat.com>
  Date:   2018-04-01 (Sun, 01 Apr 2018)

  Changed paths:
    M utilities/checkpatch.py

  Log Message:
  -----------
  checkpatch: filter comment contents

For the infix operator whitespace checks, some of these operators are
used within comments.  In those cases, it probably doesn't make sense
to warn about whitespacing.

There may be other checks that could use this kind of filter, but
that can wait for a future commit (and someone ambitious enough to
test each case).

Signed-off-by: Aaron Conole <aconole at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: 999c7773a60b296b68a99e85a0e1b4577ec6b6dc
      https://github.com/openvswitch/ovs/commit/999c7773a60b296b68a99e85a0e1b4577ec6b6dc
  Author: Aaron Conole <aconole at redhat.com>
  Date:   2018-04-01 (Sun, 01 Apr 2018)

  Changed paths:
    M utilities/checkpatch.py

  Log Message:
  -----------
  checkpatch: add a comment spell-checker

Grow a new opt-in feature to check comments for possible spelling
mistakes.  Uses the 'enchant' library to provide a default link to
aspell/ispell as the backend.

Additionally, a custom set of kewords is included inline to match what
would be possibly encountered in 'the wild'.  The list is fairly
comprehensive at this point.

Signed-off-by: Aaron Conole <aconole at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/5dbf6b17aea9...999c7773a60b


More information about the git mailing list