Am Sonntag, 13. Juni 2010, um 18:48:28 schrieb Lôrinc:
> I have just started browsing the lily source and would like to send my
> first cosmetics patch. All I did was changed a few "*= 1 /" to"/=" and
> wrote the needed overloaded /= operators in interval and offset.

Looks good to me. 
I wouldn't add the empty line in flower/offset.hh, though.

> Please let me know if I did anything wrong, as I would like to send some
> more patches in the future.

Yes, of course. If you are using git, you might want to look at git-cl (please 
search in the mailing list archives) and upload patches to 
codereview.appspot.com

Cheers,
Reinhold

-- 
------------------------------------------------------------------
Reinhold Kainhofer, reinh...@kainhofer.com, http://reinhold.kainhofer.com/
 * Financial & Actuarial Math., Vienna Univ. of Technology, Austria
 * http://www.fam.tuwien.ac.at/, DVR: 0005886
 * LilyPond, Music typesetting, http://www.lilypond.org

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

Reply via email to