[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Rewrite Skyline code (issue 547980044 by address@hidden)
From: |
jonas . hahnfeld |
Subject: |
Re: Rewrite Skyline code (issue 547980044 by address@hidden) |
Date: |
Mon, 27 Apr 2020 23:49:58 -0700 |
https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc
File lily/skyline.cc (right):
https://codereview.appspot.com/547980044/diff/572060043/lily/skyline.cc#newcode175
lily/skyline.cc:175: Building front_;
On 2020/04/27 21:30:18, hanwenn wrote:
> On 2020/04/27 07:01:28, hahnjo wrote:
> > In any case, this causes copies on every call to next(). If you need
to retain
> > this data structure, is there a reason not to use a pointer to the
current
> > element? AFAICS you don't modify the underlying vector when using a
> > BuildingQueue
>
> see the split_off() function.
That could be made a function taking an iterator as argument, or am I
missing something? AFAICS it's making a copy of the current element,
then modifies both versions, and possibly advances the iterator.
https://codereview.appspot.com/547980044/
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), (continued)
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), nine . fierce . ballads, 2020/04/21
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), hanwenn, 2020/04/21
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), hanwenn, 2020/04/21
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), jonas . hahnfeld, 2020/04/24
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), hanwenn, 2020/04/24
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), hanwenn, 2020/04/24
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), hanwenn, 2020/04/24
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), dak, 2020/04/24
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), jonas . hahnfeld, 2020/04/27
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), hanwenn, 2020/04/27
- Re: Rewrite Skyline code (issue 547980044 by address@hidden),
jonas . hahnfeld <=
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), hanwenn, 2020/04/28
- Re: Rewrite Skyline code (issue 547980044 by address@hidden), jonas . hahnfeld, 2020/04/28