[Octave-bug-tracker] [bug #64361] sinc() return NaN on sinc(-1:0.20:1-0.20) expression

[Rik]

Follow-up Comment #22, bug #64361 (project octave):

Hmmm, too bad the patch didn't work out.  I suggested it because I noticed
that the code for elem() and array_value() was just slightly different.

In elem() the code is


m_base + T (i) * m_increment


while in array_value() it is


m_base + i * m_increment


In the first construct, the variable 'i' of type octave_idx_type is first
converted to the end type (double) and the C++ compiler sees a "double"
multiplying "double".  In array_value, the C++ compiler sees "long int"
multiplying "double" and I thought the promotion rules might be doing
something funny.

Even if this isn't the problem, it might be good to choose the same coding
style for all of these functions.

I'm doing a full re-compile with Dmitri's suggested flags now.




    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?64361>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/