[Bug 1647] Modular SuperCore Scheduler

[bugzilla-daemon at rtems.org]

https://www.rtems.org/bugzilla/show_bug.cgi?id=1647

--- Comment #13 from Joel Sherrill <joel.sherrill at oarcorp.com> 2010-08-12 06:47:46 CDT ---
(In reply to comment #11)
> (In reply to comment #7)
> 

> > + Your Doxygen comments do not document the parameters.
> > 
> Do I need to add param descriptions if I use the @a paramname in the
> description of what the function does?

I agree with Sebastian that there is no universal Doxygen style
in RTEMS and we are in disagreement over the value of @param.
However, this code goes in the score and the score has them.
It would be inconsistent in the score.

> > + readyqueue.h has cases where parameters do not line up.  Please check
> > all and fix.
> > 
> Still not sure what you meant. ;)

int    x;
int     y;
int    z;

Sometimes the arguments in a prototype or in the method don't line
up.

> 
> > + Comment on _Thread_Calculate_heir.  Says "returns" .. really sets 
> > _Thread_Heir. 
> > 
> Where is this? I got rid of  _Thread_Calculate_heir

Looks like I read code in a deletion.  Ignore.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.