[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 18942
From: |
nine . fierce . ballads |
Subject: |
Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden) |
Date: |
Thu, 01 Jan 2015 23:08:56 +0000 |
On 2015/01/01 22:57:31, benko.pal wrote:
https://codereview.appspot.com/189420043/diff/1/lily/breathing-sign.cc
File lily/breathing-sign.cc (right):
https://codereview.appspot.com/189420043/diff/1/lily/breathing-sign.cc#newcode122
lily/breathing-sign.cc:122: if (ydim[DOWN] < val && line_pos.begin ()
< it - 1)
I'd rather write
line_pos.begin () + 1 < it
but good catch anyway, thanks!
I also find the whole surrounding code difficult to read, but I don't
want to spend any more time on this than I have to. Thanks for the
feedback.
https://codereview.appspot.com/189420043/
- Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), nine . fierce . ballads, 2015/01/01
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), benko . pal, 2015/01/01
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden),
nine . fierce . ballads <=
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), benko . pal, 2015/01/02
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), dak, 2015/01/02
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), benko . pal, 2015/01/02
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), nine . fierce . ballads, 2015/01/02
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), dak, 2015/01/02
- Re: Issue 4212: fix out-of-bounds index in division_maior() (issue 189420043 by address@hidden), nine . fierce . ballads, 2015/01/02