Re: Limit looping in Grob::common_refpoint (issue 4079) (issue 134600043 by address@hidden)

[David Kastrup]

Mike Solomon <address@hidden> writes:

> +1.  The only other solution I see is to check when the parental
> relationship is being created, but this check results in a performance
> hit.

There is no real point in checking for a bug which one does not know how
to recover from sensibly.

> If someone cooked up a speedy algorithm, then I think it would be a
> good idea to check at relationship-creation time.

When the typical error case is that of multiple identical engravers in
the same context, it would seem that the place to check is when adding
engravers.

>> https://codereview.appspot.com/134600043/

-- 
David Kastrup