Re: cvs: php-src(PHP_5_2) / NEWS /ext/standard math.c
| From: | Jani Taskinen | Date: | Thu, 30 Oct 2008 10:28:56 +0000 |
| Subject: | Re: cvs: php-src(PHP_5_2) / NEWS /ext/standard math.c | ||
| References: | 1 2 3 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-53791@lists.php.net to get a copy of this message | ||
Hannes Magnusson wrote:
On Wed, Oct 29, 2008 at 21:52, Hannes Magnusson <hannes.magnusson@gmail.com> wrote:It's a bugfix so of course it does. --JaniOn Wed, Oct 29, 2008 at 21:17, Ilia Alshanetsky <iliaa@php.net> wrote:Hah! Fail! You did [DOC] it, the 5.3 commit \o/ But then, does this really belong in 5.2?Is this really something you want to change minutes before latest RC of (probably) the latest 5.2 release without any tests what so ever? It looks to me that the likelihoods of change in behaviour are quite big.. And what about the bug reporters big fat note: ..."DOCUMENT this behaviour so that anybody who wants to use the round function knows exactly what expects him."? That screams to me like he anticipates changes in behaviour which really really should be documented.. and you neither tagged this commit with [DOC] nor closed the report as "To be documented"?iliaa Wed Oct 29 20:17:33 2008 UTCModified files: (Branch: PHP_5_2) /php-src NEWS /php-src/ext/standard math.cLog: MFB: Fixed bug #42294 (Unified solution for round() based on C99 round).