[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Use Interval for representing skyline X coordinates (issue 571980043
From: |
hanwenn |
Subject: |
Re: Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden) |
Date: |
Thu, 16 Apr 2020 13:57:16 -0700 |
https://codereview.appspot.com/571980043/diff/579590043/lily/skyline.cc
File lily/skyline.cc (right):
https://codereview.appspot.com/571980043/diff/579590043/lily/skyline.cc#newcode100
lily/skyline.cc:100: x_[START] = start;
On 2020/04/16 20:50:20, Dan Eble wrote:
> Initialize x_ in a member initializer list instead of allowing it to
be
> default-initialized and then assigning it here.
Done.
https://codereview.appspot.com/571980043/diff/579590043/lily/skyline.cc#newcode107
lily/skyline.cc:107: Real start = b[horizon_axis][LEFT];
On 2020/04/16 20:50:20, Dan Eble wrote:
> simplify here too
>
> Building::Building (Box const &b, Axis horizon_axis, Direction sky) :
> x_ (b[horizon_axis])
> {
> Real height = sky * b[other_axis (horizon_axis)][sky];
> precompute (x_[LEFT], height, height, x_[RIGHT]);
> }
Done.
https://codereview.appspot.com/571980043/
- Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden), jonas . hahnfeld, 2020/04/16
- Re: Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden), dak, 2020/04/16
- Re: Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden), hanwenn, 2020/04/16
- Re: Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden), nine . fierce . ballads, 2020/04/16
- Re: Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden),
hanwenn <=
- Re: Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden), nine . fierce . ballads, 2020/04/16
- Re: Use Interval for representing skyline X coordinates (issue 571980043 by address@hidden), hanwenn, 2020/04/17