Closed Bug 421422 Opened 12 years ago Closed 12 years ago

Upgrade cairo to 1.5.12-56-ga33351f


(Core :: Graphics, defect, P1)






(Reporter: vlad, Assigned: vlad)




(2 files)

Tracking bug for bugs fixed by upgrading; ??? will be filled in when I do the upgrade, probably early next week.
Flags: blocking1.9+
pixman-0.9.6-43-gddfb69a (now with moar MMX!) is coming along for the ride.
Summary: Upgrade cairo to (???) → Upgrade cairo to 1.5.12-56-ga33351f
Closed: 12 years ago
Resolution: --- → FIXED
Does this patch touch text-rendering on Mac ?
Text looks more fuzzy and 'lighter' than it does on pre-patch builds, or with Safari.

With Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b5pre) Gecko/2008031422 
Some of the code changed, though I'm pretty sure that we don't actually set any of the flags that would cause us to hit the new code.  Can you attach some screenshots?  (Ideally with before-and-after.)
Whoops, found it -- typo was accidentally forcing subpixel AA off.  Fix checked in.
On the left a Camino build, around 17:00 PDT
right Minefield post patch

Look at the navigation bar at the top 'news' > 'world news'

(In reply to comment #4)
> Whoops, found it -- typo was accidentally forcing subpixel AA off.  Fix checked
> in.
Cool! That is much better now on a quickly made home-build Camino.

Depends on: 423115
For some reason the first typedef for ullong doesn't appear to be inside an #ifdef although the second one is. Also, there are a few cases that use unsigned long long where it looks as if it was meant to use ullong.

Sorry if this is the wrong bug but it was the one mentioned in your check in.
Attachment #309783 - Flags: review?(vladimir)
So this drops support for gcc3, what with the *-growth CFLAGS.  Is there no way to not use those based on compiler?
Boris, I fixed that compiler thing for OS/2 in bug 423115 but if you know a way to make that depend on compiler version (using CC_VERSION?) instead of platform that would be much better.
Comment on attachment 309783 [details] [diff] [review]
Fix unusual usage

Looks fine, I've pushed this upstream into pixman -- I'll pull it into our own tree early next week.
Attachment #309783 - Flags: review?(vladimir) → review+
You need to log in before you can comment on or make changes to this bug.