Thanks, applied to master.
On Fri, Jan 26, 2018 at 12:36:37PM -0800, Justin Pettit wrote: > D’oh. > > Acked-by: Justin Pettit<jpet...@ovn.org.> > > --Justin > > > > On Jan 26, 2018, at 11:46 AM, Ben Pfaff <b...@ovn.org> wrote: > > > > Fixes the following warnings: > > > > ../utilities/checkpatch.py:219:1: E302 expected 2 blank lines, found 1 > > ../utilities/checkpatch.py:224:1: E302 expected 2 blank lines, found 1 > > ../utilities/checkpatch.py:228:1: E302 expected 2 blank lines, found 1 > > > > CC: Justin Pettit <jpet...@ovn.org> > > Fixes: 4e99b70dfae0 ("checkpatch.py: Add check for "xxx" in comments.") > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > --- > > utilities/checkpatch.py | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py > > index 42777e6fcb15..8a3895240357 100755 > > --- a/utilities/checkpatch.py > > +++ b/utilities/checkpatch.py > > @@ -216,15 +216,18 @@ def is_comment_line(line): > > """Returns TRUE if the current line is part of a block comment.""" > > return __regex_is_comment_line.match(line) is not None > > > > + > > def has_comment(line): > > """Returns TRUE if the current line contains a comment or is part of > > a block comment.""" > > return __regex_has_comment.match(line) is not None > > > > + > > def trailing_operator(line): > > """Returns TRUE if the current line ends with an operatorsuch as ? or > > :""" > > return __regex_trailing_operator.match(line) is not None > > > > + > > def has_xxx_mark(line): > > """Returns TRUE if the current line contains 'xxx'.""" > > return __regex_has_xxx_mark.match(line) is not None > > -- > > 2.10.2 > > > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev