Re: [PATCH] fdo#38090: vcl: remove ImplFontCache::maFontNameList:

2012-12-20 Thread Michael Meeks
On Thu, 2012-12-20 at 13:06 +0100, Michael Stahl wrote: > it is actually a different cache that is situated _before_ the > maFontInstanceList cache that was tweaked to fix those bugs, though i > have no idea what performance impact this has... Ah - nothing like stacking a dozen caches on

Re: [PATCH] fdo#38090: vcl: remove ImplFontCache::maFontNameList:

2012-12-20 Thread Michael Stahl
On 20/12/12 11:43, Michael Meeks wrote: > Hi Michael, > > On Wed, 2012-12-19 at 20:19 +, Michael Stahl wrote: >> https://gerrit.libreoffice.org/1424 >> >> To pull it, you can do: >> >> git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/24/1424/1 >> >> fdo#38090: vcl: remove

Re: [PATCH] fdo#38090: vcl: remove ImplFontCache::maFontNameList:

2012-12-20 Thread Michael Meeks
Hi Michael, On Wed, 2012-12-19 at 20:19 +, Michael Stahl wrote: > https://gerrit.libreoffice.org/1424 > > To pull it, you can do: > > git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/24/1424/1 > > fdo#38090: vcl: remove ImplFontCache::maFontNameList: > > The font cache

[PATCH] fdo#38090: vcl: remove ImplFontCache::maFontNameList:

2012-12-19 Thread Michael Stahl (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/1424 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/24/1424/1 fdo#38090: vcl: remove ImplFontCache::maFontNameList: The font cache in VCL returns different fonts for identica