https://codereview.appspot.com/583750043/diff/557770050/lily/skyline.cc
File lily/skyline.cc (right):

https://codereview.appspot.com/583750043/diff/557770050/lily/skyline.cc#newcode584
lily/skyline.cc:584: for (auto const b : buildings_)
On 2020/04/26 11:28:19, hahnjo wrote:
> I think this will do a copy of every building. Did you actually want
to use
> for (auto const &b : buildings_)
> ?

Done.

https://codereview.appspot.com/583750043/

Reply via email to