D16826: Treat uints and doubles the same as integers
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Sat Dec 1 17:26:49 GMT 2018
bruns added a comment.
I think support for non-integers is useful in general, but there are a few things we should consider:
Currently, only integral numbers can be searched. For floats, we definitely want Less/Greater matches to work, exact matches for floats are not very useful.
As you are referring to the TermGenerator, I suppose the (float) variant gets converted to something like "-5.73", which is mangled by the TermGenerator to something like "X42-5.73", i.e. dropping the sign? (See D17284 <https://phabricator.kde.org/D17284>)
Ideally, this were backed by some unit tests. The comparision test can be added to the postingdbtest, but AFAICS there are currently no tests covering `Baloo::Result`
REPOSITORY
R293 Baloo
REVISION DETAIL
https://phabricator.kde.org/D16826
To: astippich, bruns, #baloo
Cc: ngraham, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181201/99959ca7/attachment.html>
More information about the Kde-frameworks-devel
mailing list