Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Comments containing urls #5

Open
rwaldron opened this Issue · 1 comment

2 participants

@rwaldron

Example:

// See: http://www...

Will produce the following error:

missing blank after start comment

I'm guessing it refers to the // in http://

@trentm

Fixed with this patch against my trentfix branch (i.e. after my pull request is taken):

diff --git a/jsstyle b/jsstyle
index 0d3288f..2031c6d 100755
--- a/jsstyle
+++ b/jsstyle
@@ -508,7 +508,7 @@ line: while (<$filehandle>) {
            !(/$lint_re/ || ($config{"splint"} && /$splint_re/))) {
                err("missing blank before close comment");
        }
-       if ($config{"blank-after-start-comment"} && /\/\/\S/) { # C++ comments
+       if ($config{"blank-after-start-comment"} && /(?<!\w:)\/\/\S/) { # C++ comments
                err("missing blank after start comment");
        }
        # check for unterminated single line comments, but allow them when
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.