Closed Bug 797863 Opened 12 years ago Closed 12 years ago

update graphite2 library to release 1.2

Categories

(Core :: Graphics: Text, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(3 files)

      No description provided.
A few minor fixes that tryserver tells me we need to keep our compilers happy. (Also reported upstream.)
Attachment #667977 - Flags: review?(jdaggett)
And a hack to work around brokenness in stl_wrappers, as per bug 721839
Attachment #667979 - Flags: review?(jdaggett)
Attachment #667976 - Flags: review?(jdaggett) → review+
Comment on attachment 667977 [details] [diff] [review]
fixes for minor build problems in graphite2 v1.2

Why does the GRAPHITE2_NFILEFACE need to be moved around?  Just curious...
Attachment #667977 - Flags: review?(jdaggett) → review+
Attachment #667979 - Flags: review?(jdaggett) → review+
Because when that is defined, the entire content of the FileFace.h header is #ifdef'd out, with the result that the graphite2 namespace isn't declared. clang complains at the residual "using namespace" line when it hasn't heard of the namespace at all.

(I expect upstream didn't hit this because they don't build with GRAPHITE2_NFILEFACE.)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: