On 2020/05/09 01:42:54, Dan Eble wrote: > I looked around. This is the only place I found using just the size of the > vector. I don't think it's worth adding and maintaining a > Staff_symbol::line_count just for this case.
Well, it wouldn’t be "adding and maintaining" since it already exists as an internal, private function (and line_positions actually relies on it, thus using that as a vector only adds an additional, unnecessary step). What would be the cost of making it public, GC and performance-wise? (See my latest patch set as a POC.) Cheers, -- V. https://codereview.appspot.com/576090043/
