[ovs-git] [openvswitch/ovs] df3a6d: netdev-dummy: Fix minor style variation.

GitHub noreply at github.com
Wed Aug 9 23:57:13 UTC 2017


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: df3a6d503e0f892c8c5ddcb7c9454bfab6079444
      https://github.com/openvswitch/ovs/commit/df3a6d503e0f892c8c5ddcb7c9454bfab6079444
  Author: Joe Stringer <joe at ovn.org>
  Date:   2017-08-09 (Wed, 09 Aug 2017)

  Changed paths:
    M lib/netdev-dummy.c

  Log Message:
  -----------
  netdev-dummy: Fix minor style variation.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 09a21284b6bc1b304302bcf4ac48eac24e097df5
      https://github.com/openvswitch/ovs/commit/09a21284b6bc1b304302bcf4ac48eac24e097df5
  Author: Joe Stringer <joe at ovn.org>
  Date:   2017-08-09 (Wed, 09 Aug 2017)

  Changed paths:
    M ofproto/ofproto-dpif-ipfix.c

  Log Message:
  -----------
  ofproto-dpif-ipfix: Fix minor style variation.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 13ad61c46e8a8f6b620818d30bfd855537ffe9e7
      https://github.com/openvswitch/ovs/commit/13ad61c46e8a8f6b620818d30bfd855537ffe9e7
  Author: Joe Stringer <joe at ovn.org>
  Date:   2017-08-09 (Wed, 09 Aug 2017)

  Changed paths:
    M ovn/northd/ovn-northd.c

  Log Message:
  -----------
  ovn-northd: Fix minor style variation.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 0d7b16daea50e16e51ace3ba5b7f6617d5954183
      https://github.com/openvswitch/ovs/commit/0d7b16daea50e16e51ace3ba5b7f6617d5954183
  Author: Joe Stringer <joe at ovn.org>
  Date:   2017-08-09 (Wed, 09 Aug 2017)

  Changed paths:
    M utilities/checkpatch.py

  Log Message:
  -----------
  checkpatch: Check for infix operator whitespace.

The 'Expressions' section of the coding style specifies that one space
should be on either side of infix binary and ternary operators. This
adds a check to checkpatch.py for most of these.

The regex won't match if there are speech marks on the line, because
the style should not apply to the contents of strings.

This check is left at warning level because there isn't a good way to
determine whether a line is within a multiline comment or string, so it
will occasionally flag such lines which contain hyphenated words.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: bcd93351fc2439924da2faea49e279dcd75df266
      https://github.com/openvswitch/ovs/commit/bcd93351fc2439924da2faea49e279dcd75df266
  Author: Joe Stringer <joe at ovn.org>
  Date:   2017-08-09 (Wed, 09 Aug 2017)

  Changed paths:
    M utilities/checkpatch.py

  Log Message:
  -----------
  checkpatch: Fix matching on C filenames.

Most of the prerequisite checks so far matched on filenames that ended
in some character followed by 'c' or 'h', rather than a filename that
ends in '.c' or '.h'. Fix this.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 12f62e9df6d58f45320eb4f4f54f5ab8b0cdd8e2
      https://github.com/openvswitch/ovs/commit/12f62e9df6d58f45320eb4f4f54f5ab8b0cdd8e2
  Author: Joe Stringer <joe at ovn.org>
  Date:   2017-08-09 (Wed, 09 Aug 2017)

  Changed paths:
    M utilities/checkpatch.py

  Log Message:
  -----------
  checkpatch: Check for trailing operators.

The style guide states that lines should not end with '?' or ':'. Check
for this and report an error.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/6595cb95a4a9...12f62e9df6d5


More information about the git mailing list