http://codereview.appspot.com/5651069/diff/1/lily/accidental-placement.cc
File lily/accidental-placement.cc (right):

http://codereview.appspot.com/5651069/diff/1/lily/accidental-placement.cc#newcode211
lily/accidental-placement.cc:211: * @return A vector of
Accidental_placement_entrys
On 2012/02/11 12:27:31, Milimetr88 wrote:
Do you mean @param and @return or the comment to the function? What
comment
would you propose?

I'm complaining about the @return comment, since it's redundant (ie. no
need to change it, just remove that line). I'm ok with the @param
comment, because you can't tell from the function signature that accs is
supposed to be a list.

http://codereview.appspot.com/5651069/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to