I approve of your use of braces for even single-statement blocks.

https://codereview.appspot.com/581580043/diff/561400052/lily/pitch.cc
File lily/pitch.cc (right):

https://codereview.appspot.com/581580043/diff/561400052/lily/pitch.cc#newcode162
lily/pitch.cc:162: if (qt < int (sizeof (accname) / sizeof
(accname[0])))
seems like you shouldn't need to cast it to int anymore
aside from that, LGTM

https://codereview.appspot.com/581580043/

Reply via email to