Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:57089 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 73685 invoked from network); 24 Dec 2011 15:18:12 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 24 Dec 2011 15:18:12 -0000 Authentication-Results: pb1.pair.com smtp.mail=ilia@ilia.ws; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=ilia@ilia.ws; sender-id=pass Received-SPF: pass (pb1.pair.com: domain ilia.ws designates 74.125.82.42 as permitted sender) X-PHP-List-Original-Sender: ilia@ilia.ws X-Host-Fingerprint: 74.125.82.42 mail-ww0-f42.google.com Received: from [74.125.82.42] ([74.125.82.42:33699] helo=mail-ww0-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id A2/DD-16446-23DE5FE4 for ; Sat, 24 Dec 2011 10:18:11 -0500 Received: by wgbds13 with SMTP id ds13so14190780wgb.5 for ; Sat, 24 Dec 2011 07:18:08 -0800 (PST) MIME-Version: 1.0 Received: by 10.216.139.71 with SMTP id b49mr15788209wej.47.1324739888115; Sat, 24 Dec 2011 07:18:08 -0800 (PST) Received: by 10.216.16.197 with HTTP; Sat, 24 Dec 2011 07:18:08 -0800 (PST) In-Reply-To: References: Date: Sat, 24 Dec 2011 10:18:08 -0500 Message-ID: To: Pierre Joye Cc: =?ISO-8859-1?B?QW5kcukgUvhtY2tl?= , David Soria Parra , Stas Malyshev , Derick Rethans , PHP Developers Mailing List Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Subject: Re: [PHP-DEV] REQUEST_TIME change in PHP 5.4 From: ilia@ilia.ws (Ilia Alshanetsky) In most instances integers and floats can be used interchangeably, which is why the patch was written in the way that it was. In the few rare cases (int) cast takes care of the trick, there is a substantial benefit for timings etc... to have higher precision timestamp available at no additional cost. Introducing additional server variables just makes things inconsistent, especially this late in the release cycle. On Sat, Dec 24, 2011 at 10:07 AM, Pierre Joye wrote: > On Sat, Dec 24, 2011 at 3:47 PM, Andr=E9 R=F8mcke wrote: > >> Yes, this is the one from Zeta Components MvcTools. >> In eZ Publish it was db based session gc using REQUEST_TIME >> and=A0compatibility=A0for potential extensions that might have used this >> variable via eZ Publish >> api:=A0https://github.com/ezsystems/ezpublish/commit/3483c623769aa9ed3be= 7b6f33e3579cf8a8efd45 >> >> In both cases a (int) was added in front of the variable to make sure it >> still behaves the same, so=A0not a big deal. >> But as also mentioned, changes like this=A0requires patching=A0to >> be=A0compatible=A0with 5.4. >> >> So unless this is done to be inline with some standard on such a server >> variable, I would suggest placing microtime on a=A0separate=A0server var= iable >> (since it is indeed useful to have it for time accumulators and performa= nce >> metrics). > > > Right, and that's why I would be in favour of a BC change, maybe by > introducing a REQUEST_TIME_FLOAT instead, so people willing to use the > float version can still have it while the existing code needs no > change. > > Adding Ilia (who made that change) and the RMs to the list. > Cheers, > -- > Pierre > > @pierrejoye | http://blog.thepimp.net | http://www.libgd.org