Han-Wen, any reason not to use range-based loops in the places I pointed to in the last patch set?
https://codereview.appspot.com/583750043/diff/545930043/lily/skyline.cc File lily/skyline.cc (right): https://codereview.appspot.com/583750043/diff/545930043/lily/skyline.cc#newcode184 lily/skyline.cc:184: for (vector<Building>::iterator i = buildings_.begin (); auto https://codereview.appspot.com/583750043/diff/545930043/lily/skyline.cc#newcode753 lily/skyline.cc:753: for (vector<Building>::const_reverse_iterator i (buildings_.rbegin ()); auto or rather range-based for https://codereview.appspot.com/583750043/