[cairo] infinite loop when CAIRO_FIXED_FRAC_BITS is 8
Behdad Esfahbod
behdad at behdad.org
Sat Feb 16 23:07:50 PST 2008
On Sat, 2008-02-16 at 13:40 -0800, Carl Worth wrote:
> On Fri, 15 Feb 2008 11:52:19 -0500, Behdad Esfahbod wrote:
> > Ah. The fix is still valid and should go in though.
>
> Yes, fair enough. It's still a good fix for _cairo_slope_compare.
>
> So here are two follow-on patches. The first simply eliminates some
> code as a cleanup, (I don't intend any functional change at all---so
> any review to verify that would be appreciated).
Looks right to me.
> Then, the second is as nice a fix as you could hope for I think. It
> simply changes a >= to a > and makes the infinite loop go away.
>
> Again, review is appreciated, but I'm feeling much more confident
> about this stuff now. It seems much more "obviously correct". Plus,
> I'm just plain excited to be ready to push out the next snapshot.
I'm not proficient in the stroker code, but the change looks quite safe
to say the least.
> -Carl
>
> PS. Behdad, I'm trying a different attachment method now, please let
> me know if it isn't kinder for your mailer. I was quite embarrassed to
> realize I'd been sending with an octet-stream mime tag. Hopefully now
> they'll all be nicely tagged as text/plain, (and UTF-8 as
> needed). Oddly enough, my mailer had always displayed the received
> things as plain text and let me comment on them by just clicking
> reply.
Yep, works great. Evo shows them inline in fact.
Cheers,
--
behdad
http://behdad.org/
"Those who would give up Essential Liberty to purchase a little
Temporary Safety, deserve neither Liberty nor Safety."
-- Benjamin Franklin, 1759
More information about the cairo
mailing list