On quarta-feira, 21 de setembro de 2016 03:56:31 PDT Kevin Kofler wrote:
> Thiago Macieira wrote:
> > And I will reject any patch that adds more complexity in qFuzzyCompare for
> > a 0.01% corner-case. Instead, we should document that it only works for
> > finite numbers far enough away from zero. If your number can be infinite
> > or NaN or zero, you have to find something else.
> 
> This is just ridiculous, the code would be fixing a clear bug, and avoid
> having to very dangerously change the behavior of QVariant.

The clear bug is in QVariant and my original patch fixed qvariant.cpp 
numericCompare function. It's ok to call fpclassify() there, since QVariant 
comparison is not expected to be really fast. Besides, it's out-of-line code.

However, I will oppose adding fpclassify() or isinf() to qFuzzyCompare. That 
function has existed for 10 years (or more) and this is this is the first we 
ever discuss it having a problem. The bug report isn't even about 
qFuzzyCompare, so as far as I know, no user has ever complained about it not 
working for infinities.

I do remember a discussion some 8 or 9 years ago about using qFuzzyCompare to 
compare to zero. That's why qFuzzyIsNull exists. Which also means that 
QVariant comparisons to exact-zero or when the numbers are close to zero 
aren't fuzzy.

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center

_______________________________________________
Development mailing list
Development@qt-project.org
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to