diff options
| author | Daniel Bevenius <daniel.bevenius@gmail.com> | 2017-10-23 08:07:09 +0200 |
|---|---|---|
| committer | Daniel Bevenius <daniel.bevenius@gmail.com> | 2017-10-25 11:24:50 +0200 |
| commit | 3e68e4414a96af2b46e66d6403f3a4ab0bc84021 (patch) | |
| tree | 2eab1debe9ef10255cefd8c96c0b3779976f94b9 /tools/cpplint.py | |
| parent | 50d727587e1e6664ea690a3d1785e3b5c9849b65 (diff) | |
| download | android-node-v8-3e68e4414a96af2b46e66d6403f3a4ab0bc84021.tar.gz android-node-v8-3e68e4414a96af2b46e66d6403f3a4ab0bc84021.tar.bz2 android-node-v8-3e68e4414a96af2b46e66d6403f3a4ab0bc84021.zip | |
src: remove empty comment in node_http2.h
This commit removes an "empty" comment in node_http2.h that I don't
think was intentional and as far as I can tell not a doxygen comment or
anything like that.
This was not picked up by the cpp linter so a suggestion has also been
added to the CheckComment function to detect these in the future.
PR-URL: https://github.com/nodejs/node/pull/16400
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'tools/cpplint.py')
| -rw-r--r-- | tools/cpplint.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/cpplint.py b/tools/cpplint.py index 845f2d2f39..241195f076 100644 --- a/tools/cpplint.py +++ b/tools/cpplint.py @@ -3021,7 +3021,7 @@ def CheckComment(line, filename, linenum, next_line_start, error): # If the comment contains an alphanumeric character, there # should be a space somewhere between it and the // unless # it's a /// or //! Doxygen comment. - if (Match(r'//[^ ]*\w', comment) and + if (Match(r'(?://[^ ]*\w)|(?:////\s*$)', comment) and not Match(r'(///|//\!)(\s+|$)', comment)): error(filename, linenum, 'whitespace/comments', 4, 'Should have a space between // and comment') |