[Webkit-unassigned] [Bug 128422] Web Inspector: update check-webkit-style to flag single quotes in WebInspectorUI projects

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Feb 18 01:35:23 PST 2014


https://bugs.webkit.org/show_bug.cgi?id=128422





--- Comment #10 from Diego Pino <dpino at igalia.com>  2014-02-18 01:32:35 PST ---
(In reply to comment #6)
> (In reply to comment #5)
> > Nit: You can just use python "# foo" comment syntax here. Unless there is an advantage to the """....""" syntax that I don't know about. These just look like generic inline comments.
> 
> Convention is to use triple-quote comments for docstrings:
> http://en.wikibooks.org/wiki/Python_Programming/Source_Documentation_and_Comments#Documentation_Strings

I didn't know about this convention. I checked a few .py file of the codebase and I only found examples of these type of comments being used, so I thought hash comments were not allowed (which I was clearly wrong because they are used extensively in other files such as cpp.py).

I will change the comments to "# foo" where appropriate.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list