In #2567055: Inline comment sniff is too permissive with ending punctuation the detection of comments that do not end with a period was tightened, but now a false positive is thrown when a comment ends with a function call without arguments.

Correctly detected:

  // See my_function($my_arg)

False positive:

  // See my_function()

The regex has a '+' where it should have a '*' :-/

CommentFileSizeAuthor
#2 2568215-2.patch2.74 KBpfrenssen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen created an issue. See original summary.

pfrenssen’s picture

Status: Active » Needs review
FileSize
2.74 KB
klausi’s picture

Status: Needs review » Fixed

Committed, thanks!

  • klausi committed 29315d8 on 8.x-2.x authored by pfrenssen
    Issue #2568215 by pfrenssen: False positive when comment ends with a...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.