blitzpp / blitz

Blitz++ Multi-Dimensional Array Library for C++
https://github.com/blitzpp/blitz/wiki
Other
405 stars 84 forks source link

apparent typo-bug in expr.h (SF bug no. 59) #40

Closed slayoo closed 6 years ago

slayoo commented 6 years ago

Would be great to prove it with a test...

lutorm commented 6 years ago

Concur, that looks like a typo. It's hard to test since it doesn't affect the results, only the vectorization. The best you could do might be verifying the result of that function on a four-operand expression.

/Patrik

On Mon, Jun 25, 2018, 22:22 Sylwester Arabas notifications@github.com wrote:

Would be great to prove it with a test...

You can view, comment on, or merge this pull request online at:

https://github.com/blitzpp/blitz/pull/40 Commit Summary

  • apparent typo-bug in expr.h (SF bug no. 59)

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/blitzpp/blitz/pull/40, or mute the thread https://github.com/notifications/unsubscribe-auth/AFK8GLDf8mr_uDw9qY1DVJ3fzQmziI9mks5uAUZsgaJpZM4U2yzH .

slayoo commented 6 years ago

Let's then merge and move on...