On 2020/04/18 11:38:17, Dan Eble wrote: > https://codereview.appspot.com/583750043/diff/577780043/lily/skyline.cc > File lily/skyline.cc (right): > > https://codereview.appspot.com/583750043/diff/577780043/lily/skyline.cc#newcode75 > lily/skyline.cc:75: i.print (); > I'm sorry, I gave you imperfect advice here because I had an iterator on my > mind. Now that I see i.print(), I realize that it should be for (auto &i : b) > to guarantee that it uses a reference to the item rather than making a copy of > the item. Again, sorry.
Thanks for the correction! It's debugging code, so either way is fine. https://codereview.appspot.com/583750043/