stay in double to avoid overflow

[christoph-cullmann]

CoinUtils/src/CoinFactorization4.cpp:2290:42: runtime error: 2.28294e+09 is outside the range of representable values of type 'int'

[christoph-cullmann]

As we only compare it, that seems more sane to me, some overflow could lead to wrong compare results.

[svigerske]

There are two more places like this, could you change these as well?

CoinFactorization4.cpp:799:      int newNumber = static_cast< int >(number * btranAverageAfterU_);
CoinFactorization4.cpp:1949:      int newNumber = static_cast< int >(number * btranAverageAfterU_);

[christoph-cullmann]

Sure, can do that, only did alter first what occurs in my testing.

[christoph-cullmann]

My grep with Kate did show more places that do the same. Should at least covered these now, too.

[svigerske]

Thanks.