On 2020/04/21 09:15:28, hahnjo wrote: > LGTM otherwise > > https://codereview.appspot.com/573730043/diff/567470043/lily/axis-group-interface.cc > File lily/axis-group-interface.cc (right): > > https://codereview.appspot.com/573730043/diff/567470043/lily/axis-group-interface.cc#newcode206 > lily/axis-group-interface.cc:206: auto it = lower_bound (break_ranks.begin (), > break_ranks.end (), start); > Could you add a comment that break_ranks is sorted by construction in > Paper_score::find_break_indices() -- if that is true: I had a quick look at the > code and this seems the most likely explanation why a binary search is legit. > But please double check and let me know if that guess is correct.
added a comment to paper-score.hh https://codereview.appspot.com/573730043/
