Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:96765
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Message-ID: <46.92.05967.9AB91285@pb1.pair.com>
To: internals@lists.php.net
Date: Tue, 8 Nov 2016 10:32:21 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.4.0
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: DateTime microseconds discussion
From: arjen@parse.nl (Arjen Schol)

Hi,

Support for microseconds was added late in the 7.1 RC cycle, however is 
has some issues:

1. There is no easy way to set microseconds to 0, you have to call 
setTime see https://3v4l.org/YUhFF A setMicroseconds method would be 
handy and/or support to relative strings to set microseconds to 0 (just 
like midnight does for H:i:s). Or am I missing something?

2. Microsecond support is useful, by not by default I think. Why not 
introduce a 3rd parameter $with_microseconds/$set_microseconds which 
defaults to false? Or A second class DateTimeWithMicroseconds? Maybe not 
so fancy naming, but it's very clear what will happen.

3. The 4th parameter to setTime() should not cause BC breakage.

This discussion was started at https://github.com/php/php-src/pull/2186

Gr Arjen