Patch & freemail

Hi!

Sorry for spamming, it seems my mail service provider
(freemail.hu) is corrupt and ruins the attached texts :(
Furthermore it ruins the message body also with unecessary
newlines. Let's try the gzip.

However, the head of the attached text is good but just
extended with rubbish.

  Attila

>
> Hi Attila!
>
> It looks like your patch was corrupted. Would you mind
> resending it?
>
> Thanks again!
> Dom
>
> --- Sir Basic <phoenix@freemail.hu> wrote:
>
> > Hi!
> >
> > I've discovered some points in "buildFont" function
> > that can
> > be bugs. The [-1] element of "fontFile" could be
> > accessed in
> > some cases. If the extension is ".font" then the
> > postfix
> > change seems improper. Please check the attached
> > file for
> > modifications.
> >
> > Meanwhile I've profiled the startup several times.
> > It seems
> > the overall running time of "buildFont" is not as
> > long as
> > I've thought before (something like 6%, proportional
> > to the
> > number of fonts installed). But please note that the
> > function spends one half of its time with those
> > string
> > operations. The nice-looking rows
> > metricFile.ucs4_str()[ffs - 3] = 'a';
> > metricFile.ucs4_str()[ffs - 2] = 'f';
> > metricFile.ucs4_str()[ffs - 1] = 'm';
> > cost 37% of execution time of the whole function
> > because of
> > the repeated call on "ucs4_str()".
> >
> > Attila> --- xap_UnixFontManager~.cpp 2004-10-22
> > 13:28:50.000000000 +0200
> > +++ xap_UnixFontManager.cpp 2004-10-22
> > 13:28:59.000000000 +0200
> > @@ -111,12 +111,15 @@
> > // TODO: We should follow symlinks.
> > metricFile = reinterpret_cast<char*>(fontFile);
> > size_t ffs = metricFile.size();
> > - if (ffs < 4 || (fontFile[ffs - 4] != '.' &&
> > fontFile[ffs - 5] != '.'))
> > + if ( ! ( (ffs >= 4 && fontFile[ffs - 4] == '.') ||
> > (ffs >= 5 && fontFile[ffs - 5] == '.') ) )
> > return NULL;
> >
> > // handle '.font'
> > if (fontFile[ffs - 5] == '.')
> > + {
> > metricFile =
> > UT_UTF8String(metricFile.ucs4_str().substr(0,
> > metricFile.size() - 1));
> > + --ffs;
> > + }
> >
> > metricFile.ucs4_str()[ffs - 3] = 'a';
> > metricFile.ucs4_str()[ffs - 2] = 'f';
> >
> Ś§Wę&#65533;Xľ&#65533;.ű&#65533;2ŹQBćč&#65533;g7¨˝tłt5`IÔ&#65533;î*ĄnQąę_ĺDXŔÜcŘ˙ÄT0ŕ˛
> > &#65533;¸t&#65533;Ńt&#65533;&#65533;&#65533;˛z&#65533;¸×}*d×ÉźË9ť
> > ćŰâz&#65533;ĎŕŇiÎv@švőŘ&#65533;&#65533;Ő+ôĂPĐ
>
JŠ4ÔŁ~*4ć˝ĺNlsłEQ4֝ޭĐ_éôaĂý&#65533;ćBq5䜤TźŮHĽ´L{&#65533;&#65533;a¨L$Ewß%×+ţÍ&#65533;&#65533;yčiŇç&}Áˇp=ĘŹ&#65533;{´ßÂUž:ĺî0Ś:lV­6&#65533;N&#65533;ź^ŹĽ
>
>
>
>
> _______________________________
> Do you Yahoo!?
> Declare Yourself - Register online to vote today!
> http://vote.yahoo.com
>