https://bugs.kde.org/show_bug.cgi?id=254779

--- Comment #4 from Rolf Eike Beer <k...@opensource.sf-tec.de> ---
> I didn't change the storage model (the part that fails the "wtf test") 
> because
> I thought it best to limit my changes to those actually necessary to 
> add the
> feature. But since you mentioned it, I will look into changing the 
> model, and
> if it works out, I'll include both changes in the revised patch.

Please do that in 2 patches for easier review, in any order that fits 
your needs.

> m_expiry was already declared as a private field of SearchResult; it 
> was just
> never used before.

Sorry, I was looking at the frameworks branch, where it got removed for 
exactly that reason ;)

Eike

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to