ID: 44092
Updated by: [EMAIL PROTECTED]
-Summary: zoe
Reported By: [EMAIL PROTECTED]
Status: Feedback
Bug Type: *Math Functions
Operating System: Linux 32
PHP Version: 5.3CVS-2008-02-10 (CVS)
New Comment:
(fix summary)
Previous Comments:
------------------------------------------------------------------------
[2008-02-12 07:36:16] [EMAIL PROTECTED]
commited fix to /ext/standard/math.c and
/ext/standard/tests/math/exp_error.phpt in PHP5.3 and PHP 5.2
------------------------------------------------------------------------
[2008-02-11 16:22:30] [EMAIL PROTECTED]
Felipe - I think they should probably all change. I'm happy to work
through one at a time (or maybe you'd like to fix some?), I don't really
want to do them all in one update. I used exp() to make sure that I was
doing the right thing. Quite a few of the tests would also need changing
(because I worked around this for a while). I think sin() and cos() at
least are wrong. I've also got bugs pending in number_format() and
hypot() - which *might* be the same issue.
Zoe
------------------------------------------------------------------------
[2008-02-11 16:07:30] [EMAIL PROTECTED]
Hello Ilia,
in this case, should all math functions in ext/standard/ use the new
parsing parameter, no? Only fix the exp() sounds wrong to me...
------------------------------------------------------------------------
[2008-02-11 14:23:35] [EMAIL PROTECTED]
Go ahead and apply the patch to the code & the test.
------------------------------------------------------------------------
[2008-02-10 17:27:40] [EMAIL PROTECTED]
If this patch goes in, the test exp_error.phpt will need a small change
too. I'm happy to do that.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/44092
--
Edit this bug report at http://bugs.php.net/?id=44092&edit=1
