[Nepomuk] Review Request 109991: Regexp cache optimization in Nepomuk fileindexer.

[Lukasz Olender]

> On April 29, 2013, 10:37 a.m., Vishesh Handa wrote:
> > common/regexpcache.h, line 67
> > <http://git.reviewboard.kde.org/r/109991/diff/4/?file=140678#file140678line67>
> >
> >     const QString& outerPatten

I can't do that since I modify it. Should I create a temporary variable?


> On April 29, 2013, 10:37 a.m., Vishesh Handa wrote:
> > common/regexpcache.h, line 68
> > <http://git.reviewboard.kde.org/r/109991/diff/4/?file=140678#file140678line68>
> >
> >     const QString&

There is a modification of that parameter too.


> On April 29, 2013, 10:37 a.m., Vishesh Handa wrote:
> > common/regexpcache.h, line 81
> > <http://git.reviewboard.kde.org/r/109991/diff/4/?file=140678#file140678line81>
> >
> >     I'm just thinking aloud over here, but maybe we just use user and int instead of size_t. It's easier to read and wouldn't really make any difference.
> >     
> >     Opinions?

I've changed all occurrences of size_t to int. It will be more consistent with Qt. I've just looked into QString, QVector and QList classes, their sizes are also returned as ints (also in Qt5), but, to be honest, I really don't know why.


- Lukasz


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109991/#review31740
-----------------------------------------------------------


On May 3, 2013, 11:11 a.m., Lukasz Olender wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109991/
> -----------------------------------------------------------
> 
> (Updated May 3, 2013, 11:11 a.m.)
> 
> 
> Review request for Nepomuk and Vishesh Handa.
> 
> 
> Description
> -------
> 
> It's related with https://bugs.kde.org/show_bug.cgi?id=303654. 
> P.S. I accidentally deleted author's and license info in patch. Isolated performance tests are also uploaded to http://www.sendspace.com/file/mkihdp (previous link not always work). It's my first patch.
> 
> 
> This addresses bug 303654.
>     http://bugs.kde.org/show_bug.cgi?id=303654
> 
> 
> Diffs
> -----
> 
>   common/regexpcache.h d89f968 
>   common/regexpcache.cpp df45277 
> 
> Diff: http://git.reviewboard.kde.org/r/109991/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Lukasz Olender
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20130503/4523ee07/attachment.html>