Closed (outdated)
Project:
Drupal core
Version:
6.x-dev
Component:
other
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
10 Jun 2007 at 05:45 UTC
Updated:
2 Mar 2016 at 22:18 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Steven commentedCommitted to HEAD.
Comment #2
drummCommitted to 5.
Comment #3
dmitrig01 commentedFound errors everywhere, so here are some of them
Comment #4
joshk commentedApplies clean to HEAD. Hooray comment consistency!
Comment #5
Steven commentedWhy use doxygen-style in CSS?
Comment #6
dmitrig01 commentedWhy not?
It should be consistent throughout Drupal core, and that is used in JS too.
Comment #7
Steven commentedThere is a specific reason to use doxygen blocks, and they are not just 'our way' of doing multi-line comments.
Comment #8
tr commentedI'm not sure what the situation was four years ago when this thread was opened, but current Drupal coding standards agree with dmitrig01 here. See http://drupal.org/node/302199.
Most of the issues addressed by the patch still need fixing in Drupal 6.x. They all appear to be fixed in Drupal 7.x though.