[prev in list] [next in list] [prev in thread] [next in thread]
List: koffice-devel
Subject: Re: PATCH: kspread html export
From: shaheed <srhaque () iee ! org>
Date: 2001-07-30 2:48:16
[Download RAW message or body]
On Sunday 29 July 2001 5:33 pm, Carsten Pfeiffer wrote:
> On Sun, Jul 29, 2001 at 10:32:18PM +0200, Nicolas Goutte wrote:
> > Just as a note, you could also write it:
> >
> > text.replace(QRegExp("&"), "&")
> > .replace(QRegExp("<"), "<")
> > .replace(QRegExp(">"), ">");
>
> I can imagine this being pretty slow. Why use a regular expression,
> if all you want to do is replacing one character with some others?
> Especially doing this three times. Better iterate over all characters
> of `text' and replace all the &,<,> manually.
Pretty much all the filters have this code. While it is certainly not going
to be fast, I didn't consider the obfuscation of the alternatives worth the
bother for the MsWord filter.
Warning: I'm only writing this note because its too hot to sleep, and because
I have spoken to Carsten for a while :-)
_______________________________________________
Koffice-devel mailing list
Koffice-devel@master.kde.org
http://master.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic