Subject: Re: [boost] [Review Request] Multiprecision Arithmetic Library
From: John Maddock (boost.regex_at_[hidden])
Date: 2012-04-15 04:52:55
>Cool! Thanks for properly scaling the argument to mysin().
>I believe the expected output in the comment should be
Both sin and mysin(pi/4) output
me - and my calculator agrees :-)
>I refactored the constants and got the tests running WRT trunk.
>Two tests did not compile at first: test_arithmetic.cpp (line 468)
>and test_float_io.cpp (line 204). The culprit is the compiler's failure to
>resolve frexp(Real, int64). The compiler required a plain
>integer argument. I believe that the compiler is correct
>(Visual Studio 2010, SP1, in this case). Do you also experience
My mistake - I forgot to commit changes to default_ops.hpp - update again
and that should get it. Sorry about that.
>All tests except one pass with my refactored code.
>Unfortunately, test_float_io.cpp fails when round tripping
>from cpp_dec_float to a string with a large exponent back
>to cpp_dec_float. I believe that this problem can be solved
>because I identified a potentially incorrect rounding and/or
>digit inclusion when creating an exponent string in my
OK good, I'd like that to work if possible, it seems like something users
would want and/or expect?
>I have candidate changes in:
>Would you like me to commit my changes or would
>you prefer to receive the files and commit them yourself?
Other than the expected value for sin(pi/4) please do go ahead and commit!
Many thanks, John.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk