Re: shortened flags: choosing appropriate flag (issue4410049)

[ianhulin44]

Spelling commenting and formatting changes only.
Cheers,
Ian


http://codereview.appspot.com/4410049/diff/16001/input/regression/shortened-flags-cues.ly
File input/regression/shortened-flags-cues.ly (right):

http://codereview.appspot.com/4410049/diff/16001/input/regression/shortened-flags-cues.ly#newcode6
input/regression/shortened-flags-cues.ly:6: }
Spelling nitpick:
chosing => choosing

http://codereview.appspot.com/4410049/diff/16001/input/regression/shortened-flags-cues.ly#newcode11
input/regression/shortened-flags-cues.ly:11: \new CueVoice {
Use four-space indents in this file rather then eight-space ones.

http://codereview.appspot.com/4410049/diff/16001/lily/stem.cc
File lily/stem.cc (right):

http://codereview.appspot.com/4410049/diff/16001/lily/stem.cc#newcode757
lily/stem.cc:757: when a flagged stem has manually overriden length, the
flag should be
Nitpick:
overriden => overridden

http://codereview.appspot.com/4410049/diff/16001/lily/stem.cc#newcode790
lily/stem.cc:790: else
// (flag_style is not blank)
It's been a long time since the if() and there was lots of involved
stuff in the then {} clause.

http://codereview.appspot.com/4410049/diff/16001/lily/stem.cc#newcode792
lily/stem.cc:792:
/*
   Look up the font character allowing for the variant stem length
*/
You've done a lot of heavy lifting to get here, this is the  fix; shout
about it for the benefit of maintainers.

http://codereview.appspot.com/4410049/