Closed
Bug 1191167
Opened 10 years ago
Closed 10 years ago
Upgrade font support to Graphite 1.3.0 just released
Categories
(Core :: Graphics: Text, defect)
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: martin_hosken, Assigned: jfkthame)
Details
Attachments
(3 files)
261.15 KB,
patch
|
jtd
:
review+
|
Details | Diff | Splinter Review |
3.99 KB,
patch
|
gerv
:
review+
|
Details | Diff | Splinter Review |
3.54 KB,
patch
|
martin_hosken
:
feedback+
|
Details | Diff | Splinter Review |
Graphite2 1.3.0 has just been released. No API changes and the integration has already been tested in bug #1184473.
Release source can be downloaded from:
http://projects.palaso.org/attachments/download/427/graphite2-1.3.0.tgz
http://sourceforge.net/projects/silgraphite/files/graphite2/graphite2-1.3.0.tgz/download
Assignee | ||
Comment 1•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8643635 -
Flags: review?(jdaggett)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•10 years ago
|
||
Martin, I notice the BSD license block in Decompressor.{h,cpp} is different from the GPL/MPL headers in most files, and claims "Copyright (c) 2012, Siyuan Fu ..." in addition to SIL. Could you confirm where this comes from, so we can make sure to attribute it appropriately? From a quick Google search, I'm guessing it may be derived from https://code.google.com/p/data-shrinker/, though it looks like you've massaged the code considerably; or is there a different (newer?) upstream source?
Flags: needinfo?(martin_hosken)
Reporter | ||
Comment 4•10 years ago
|
||
I can confirm that the original code came from https://code.google.com/p/data-shrinker but that it has been heavily modified by us.
Flags: needinfo?(martin_hosken)
Assignee | ||
Comment 5•10 years ago
|
||
I assume we should add this, to ensure we properly acknowledge the copyright on these two files.
Attachment #8643757 -
Flags: review?(gerv)
Comment 6•10 years ago
|
||
Comment on attachment 8643757 [details] [diff] [review]
pt 2 - Add data-shrinker license to about:license
Review of attachment 8643757 [details] [diff] [review]:
-----------------------------------------------------------------
r=gerv.
Gerv
Attachment #8643757 -
Flags: review?(gerv) → review+
Updated•10 years ago
|
Attachment #8643635 -
Flags: review?(jdaggett) → review+
Assignee | ||
Comment 7•10 years ago
|
||
url: https://hg.mozilla.org/integration/mozilla-inbound/rev/fc9a4891714ba18330acfc4bdd7ea822f05a0e8c
changeset: fc9a4891714ba18330acfc4bdd7ea822f05a0e8c
user: Jonathan Kew <jkew@mozilla.com>
date: Mon Aug 10 09:38:46 2015 +0100
description:
Bug 1191167 - Update graphite2 library to release 1.3.0. r=jdaggett
url: https://hg.mozilla.org/integration/mozilla-inbound/rev/675cecb64f753eaa8769a71d43bf8fdac272e006
changeset: 675cecb64f753eaa8769a71d43bf8fdac272e006
user: Jonathan Kew <jkew@mozilla.com>
date: Mon Aug 10 09:39:15 2015 +0100
description:
Bug 1191167 pt 2 - Add data-shrinker license to about:license. r=gerv
Assignee | ||
Comment 8•10 years ago
|
||
It turns out the new graphite version has increased the number of static constructors in Gecko by 3. We try to avoid this, because they hurt startup time. Looking at the code, I think it's straightforward to avoid the increase here; Martin, please take a look at this patch, and apply upstream if appropriate. Thanks.
Attachment #8645828 -
Flags: feedback?(martin_hosken)
https://hg.mozilla.org/mozilla-central/rev/fc9a4891714b
https://hg.mozilla.org/mozilla-central/rev/675cecb64f75
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Reporter | ||
Updated•10 years ago
|
Attachment #8645828 -
Flags: feedback?(martin_hosken) → feedback+
You need to log in
before you can comment on or make changes to this bug.
Description
•