Edit report at https://bugs.php.net/bug.php?id=52798&edit=1

 ID:               52798
 Updated by:       [email protected]
 Reported by:      ebinachon at pierre-vacances dot fr
 Summary:          dateTime => function diff => days KO
-Status:           Feedback
+Status:           No Feedback
 Type:             Bug
 Package:          Date/time related
 Operating System: windows xp
 PHP Version:      5.3.3
 Assigned To:      derick

 New Comment:

No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Open". Thank you.


Previous Comments:
------------------------------------------------------------------------
[2012-02-24 17:51:52] kissifrot at gmail dot com

Reproduced on PHP Win32 5.3.10 VC6, but not the VC9.

------------------------------------------------------------------------
[2011-01-30 11:22:53] [email protected]

Also, please try vc9 build.

------------------------------------------------------------------------
[2011-01-30 10:49:36] [email protected]

Please try using this snapshot:

  http://snaps.php.net/php5.3-latest.tar.gz
 
For Windows:

  http://windows.php.net/snapshots/

Works fine for me with current SVN 5.3 HEAD.

------------------------------------------------------------------------
[2010-09-08 18:12:46] [email protected]

The problem was the call to floor().

The compiler gave a warning:
"d:\users\cataphract\documents\php-trunk\ext\date\lib\interval.c(57) : warning 
C4013: 'floor' undefined; assuming extern returning int"

The assumption is wrong, because it calls

double  __cdecl floor(_In_ double _X);

Which takes and returns a larger type.

I removed the call to floor, which is unnecessary since it's an integer 
division.

I also changed the call to abs to a call to llabs to avoid the conversion from 
long long to int and then to long long again.

------------------------------------------------------------------------
[2010-09-08 18:03:46] [email protected]

The following patch has been added/updated:

Patch Name: date_diff
Revision:   1283961826
URL:        
http://bugs.php.net/patch-display.php?bug=52798&patch=date_diff&revision=1283961826

------------------------------------------------------------------------


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

    https://bugs.php.net/bug.php?id=52798


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=52798&edit=1

Reply via email to