| Issue 71: | #-comments | |
| 1 person starred this issue and may be notified of changes. | Back to list |
Hi, could you pls replace '#' comments with the '//' Thanks
Sep 18, 2013
Project Member
#1
mewp...@gmail.com
Sep 18, 2013
well, there is nothing wrong with linux shell comments in php,as official documentation states... It is just a feature request... I am trying to conform this http://kohanaframework.org/3.3/guide/kohana/conventions#comments maybe it also makes sense for you... also these sharp comments got my syntax highlighter just crazy :)
Sep 18, 2013
if you have not a time for that then I could prepare a patch
Sep 19, 2013
sh (and other shells), python, perl, ruby, php, tcl, and other languages use #-comments. If your syntax highlighter can't handle that, it's a bug with the highlighter, not with LightOpenID code. php, c/c++, and java use //-comments. I personally think that #-commments are more fitting for a dynamic language, but regardless -- I don't see a reason to include a patch changing a completely arbitrary thing in the code, that doesn't affect functionality, readability, robustness, or really anything else. Furthermore, I don't consider kohana framework style conventions a superior (or, for that matter, inferior) to other conventions, including the one that LightOpenID uses. In one sentence: It would be a useless change.
Status:
WontFix
|