[cairo] [PATCH 1/2] stroke: Fix large line widths for fallback stroke shaper
Chris Wilson
chris at chris-wilson.co.uk
Thu Mar 14 01:49:27 PDT 2013
On Wed, Mar 13, 2013 at 05:19:19PM -0700, Martin Robinson wrote:
> On 03/13/2013 04:57 AM, Chris Wilson wrote:
> > Hmm, can you do this as a separate patch. Preferably with another to try
> > and fix the current code before replacing it. As you should be able to
> > avoid the division to check the intersection against [0,1] so I want a
> > closer look at what you consider the bug to be.
>
> Sure, I think that the attached patch fixes the intersection code.
> If that's correct we can keep both the slow version (floating point)
> and the fast version (fixed point) around.
>
>
> commit c3042c097e1a508fc56c87781cb72096ee735833
> Author: Martin Robinson <mrobinson at igalia.com>
> Date: Wed Mar 13 17:08:23 2013 -0700
>
> path: Fix a bug in line intersection
>
> Before the intersection code was not taking into account that both
> quotients are required to be in the range (0,1) for the segments to
> intersect or handling the case of negative numerators and denominators.
D'oh. Yes, that's definitely an oversight I should have picked up. Patch
looks like it does what you want.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the cairo
mailing list