Adding more comments to the new mobility code
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
HIPL |
New
|
Medium
|
Unassigned |
Bug Description
From Rene: can you please make sure that the update.h contains doxygen comments
along with the static functions in update.c. It would also be great to
have some more comments in update.c in the function bodies.
(Miika: I would actually prefer to move all doxygen comments to the c files
because that's what developers really read)
---
* Doxygen headers must be present for every
function and every parameter (including return) must be documente.
* Parameters
that are modified in the function must be marked in the doxygen header, too.
* Tabs and spaces are missing. Tab = 8 spaces
* The keyword const is used only 3! times. Const correctness is one of the
cornerstones for good coding. This definitely
needs to be enhanced
---
Two update.h functions are doxygened. The rest of the functions are removed
from the header because they weren't used by other files.
Related branches
Changed in hipl: | |
assignee: | nobody → Baris Boyvat (baris-boyvat) |
Changed in hipl: | |
status: | New → In Progress |
importance: | Undecided → Medium |
Changed in hipl: | |
status: | In Progress → New |
Someone still needs to get const-correctness into the mobility extension. Is Baris actively working on this?