Closed (fixed)
Project:
Drupal Code Sniffer
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Nov 2011 at 20:32 UTC
Updated:
27 Nov 2011 at 21:50 UTC
The inline comment sniff has an issue with this piece of code.
Which is exported by ctools/feature.
$strongarm->disabled = FALSE; /* Edit this to true to make a default strongarm disabled initially */
It returns an error with the MIssing space after the //.
This isn't correct.
Comments
Comment #1
klausiBut the code line is not correct either. 1) "//" style comments should be used on inline code, 2) a comment should be one line before the code line.
Comment #2
ericduran commented@klausi, Yea, I'm wondering if instead we should file a bug in ctools, to fix the way the comment is render.
Comment #3
klausiJust ran into a similar issue, should be fixed in this commit: http://drupalcode.org/project/drupalcs.git/commit/0fafb58
Comment #4
ericduran commentedCool.