m...@apollinemike.com writes:

> On Jan 5, 2012, at 7:51 AM, David Kastrup wrote:
>
>> Grob::get_vertical_axis_group is not protected against the case where g
>> has an axis group interface but no Y_AXIs parent.
>
> I thought it was.  If g has no Y_AXIS parent, then when
> get_vertical_axis_group (g->get_parent (Y_AXIS)); is called, the first
> test if (!g) will return 0, no?

Grob::get_vertical_axis_group (Grob *g)
{
  if (!g)
    return 0;
  if (Axis_group_interface::has_interface (g)
      && Align_interface::has_interface (g->get_parent (Y_AXIS)))
    return g;
  return get_vertical_axis_group (g->get_parent (Y_AXIS));

}

You call Align_interface::has_interface (g->get_parent (Y_AXIS)) here
before you call get_vertical_axis_group.


-- 
David Kastrup


_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to