[rekonq] Re: Review Request: keep downloaded suggestions in memory

Andrea Diamantini adjam7 at gmail.com
Mon Jan 17 12:36:27 CET 2011



> On Jan. 17, 2011, 11:23 a.m., Andrea Diamantini wrote:
> > src/opensearch/opensearchengine.h, line 120
> > <http://git.reviewboard.kde.org/r/100398/diff/2/?file=7146#file7146line120>
> >
> >     Could we change this name (or the other)? Having the same one when you are not really overriding it, is not good IMHO.

gasp.. this is obviously an overloading, not an overriding. Sorry.


- Andrea


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


On Jan. 16, 2011, 4:01 p.m., Lionel Chauvin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100398/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2011, 4:01 p.m.)
> 
> 
> Review request for rekonq.
> 
> 
> Summary
> -------
> 
> it is the first step of a cache system for suggestions.
> keep downloaded suggestions in memory.
> prevent downloaded it again.
> the completion list is more reactive.
> 
> 
> Diffs
> -----
> 
>   src/opensearch/opensearchengine.h c05501f 
>   src/opensearch/opensearchengine.cpp ef0f3f7 
>   src/opensearch/opensearchmanager.cpp 41e46ec 
> 
> Diff: http://git.reviewboard.kde.org/r/100398/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Lionel
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/rekonq/attachments/20110117/e7726ce3/attachment.htm 


More information about the rekonq mailing list