Merge pull request #668 from colszowka/even-faster-no_cov_line

TIL there is an `o` modifier doing the same thanks to @eregon
simplecov-ruby · Mar 10, 2018 · fbd3c3e · fbd3c3e
2 parents c3d4936 + 3dc01d6
commit fbd3c3e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,7 +5,7 @@ unreleased
 
 * Relax version constraint on `docile`, per SemVer
 * exception that occurred on exit is available as `exit_exception`! See [#639](https://github.com/colszowka/simplecov/pull/639)  (thanks @thomas07vt)
-* Performance: processing results now runs from 2.5x to 3.75x faster. See [#662](https://github.com/colszowka/simplecov/pull/662) (thanks @BMorearty)
+* Performance: processing results now runs from 2.5x to 3.75x faster. See [#662](https://github.com/colszowka/simplecov/pull/662) (thanks @BMorearty & @eregon)
 
 ## Bugfixes
 

diff --git a/lib/simplecov/lines_classifier.rb b/lib/simplecov/lines_classifier.rb
@@ -13,7 +13,7 @@ class LinesClassifier
     WHITESPACE_OR_COMMENT_LINE = Regexp.union(WHITESPACE_LINE, COMMENT_LINE)
 
     def self.no_cov_line
-      @no_cov_line ||= /^(\s*)#(\s*)(\:#{SimpleCov.nocov_token}\:)/
+      /^(\s*)#(\s*)(\:#{SimpleCov.nocov_token}\:)/o
     end
 
     def classify(lines)