[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Use aligned-on-x-parent instead of other callbacks for some grobs (i
From: |
pkx166h |
Subject: |
Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden) |
Date: |
Sat, 30 Aug 2014 09:35:26 +0000 |
On 2014/08/29 22:21:31, dak wrote:
https://codereview.appspot.com/127860043/diff/80001/lily/note-column.cc
File lily/note-column.cc (right):
https://codereview.appspot.com/127860043/diff/80001/lily/note-column.cc#newcode167
lily/note-column.cc:167: Grob *main_head;
This is a catastrophic error. If get_stem (me) returns 0 and there
are no grobs
in note-heads, main_head is not initialized at all but gets a random
value. The
rest of the code more likely than not then calls Grob::extent on this
random
value.
It would have been easy to notice this error: GCC puts out a quite
accurate
warning.
Ignoring GCC's warnings is rarely a good idea.
Oh sorry. I would have thought perhaps Patchy would have 'borked' at
some point?
https://codereview.appspot.com/127860043/
- Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden), k-ohara5a5a, 2014/08/10
- Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden), k-ohara5a5a, 2014/08/18
- Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden), janek . lilypond, 2014/08/18
- Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden), dak, 2014/08/20
- Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden), janek . lilypond, 2014/08/27
- Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden), dak, 2014/08/29
- Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden),
pkx166h <=
- Re: Use aligned-on-x-parent instead of other callbacks for some grobs (issue 127860043 by address@hidden), dak, 2014/08/30