Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | graphite: slow performance with Graphite fonts in large docs | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | gsl | Reporter: | devel | ||||||
Component: | code | Assignee: | stefan.baltzer | ||||||
Status: | CLOSED FIXED | QA Contact: | issues@gsl <issues> | ||||||
Severity: | Trivial | ||||||||
Priority: | P3 | CC: | fonts-bugs, hdu, issues | ||||||
Version: | DEV300m83 | Keywords: | performance | ||||||
Target Milestone: | OOo 3.3 | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Issue Type: | PATCH | Latest Confirmation in: | --- | ||||||
Developer Difficulty: | --- | ||||||||
Issue Depends on: | |||||||||
Issue Blocks: | 108220, 111112 | ||||||||
Attachments: |
|
Description
devel
2010-06-27 11:18:52 UTC
Created attachment 70244 [details]
patch to vcl and graphite/makefile.mk
Created attachment 70245 [details]
updated graphite patch
Graphite patch attached as separate attachment since patches of patches are hard to read. I tested the patches against DEV300_m83 on Linux Ubuntu 10.04 (64bit) and Vista (32 bit). For some reason I can't change the version field to a more sensible value. Thanks! Looks good to me. I created a CWS for it (http://hg.services.openoffice.org/cws/graphite03) which is based on the latest milestone (DEV300_m83). You could apply the changes there or I could do it, depending on your preference. Thanks for committing the changes. I'll test the resulting builds soon. Are there more changes pending for this CWS? I have nothing more planned at this stage. I was rather hoping these changes might be able to get into 3.3, since they are more bug fixes than an enhancement. At this stage of OOo33 regular bugfixes do not make it into that release, only showstoppers etc. On the other hand looking at the "releases" mailing list the hurdle is still low. Get approval from UL or MD for this issue being an OOo33 showstopper and we can change the target to OOo33. With a simple but large testdoc for graphite the slowness could be recreated. For some scenarios like scrolling the improvement was a factor of 2..3 (ratio of handstopped times), nice! Looking at the changes everything is high quality and I see no risks for other code not related to graphite, so an approval for OOo33 would be fine with me. @sba: please verify in CWS graphite03 Adjusted target Verified in CWS graphite03. |