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/