Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:50861
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain beberlei.de from 178.77.98.152 cause and error)
Date: Sun, 5 Dec 2010 20:36:16 +0100
To: internals@lists.php.net, smalyshev@sugarcrm.com
Message-ID: <20101205203616.51a66416@benny-pc>
In-Reply-To: <4CFBDA09.2020902@sugarcrm.com>
References: <f9e85836d9f401e78d6a045116c6f5e0@beberlei.de>
	<4CFBDA09.2020902@sugarcrm.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Patch: Marking DateTime Instances Immutable
From: kontakt@beberlei.de (Benjamin Eberlei)

On Sun, 05 Dec 2010 10:29:29 -0800
Stas Malyshev <smalyshev@sugarcrm.com> wrote:

> Hi!
> 
> > I propose to allow to work with DateTime objects that are marked as
> > immutable optionally. This means that all methods add, sub, modify,
> 
> I think it's a bad idea, which would instantly break all the code that 
> assumes different semantics. Yes, I noticed the word optional, however, 
> the code accepting DateTime assumes some semantics, and had no means to 
> reject object with different semantics, and combining two semantics in 
> one class is a bad idea.
> 
> On the other hand, you can create DateTimeValue object, which can do 
> what you want. I'd imagine it'd be quite easy to do in user space too. 
> If enough people feel they need it then it can be brought into the 
> extension too.

Hey Stas,

i actually implemented this in userland, but its rather ugly:

http://whitewashing.de/blog/124

I'd like to see it in the extension, the patch was just the only thing i could do with my limited C skills ;-) It is better to start the discussion with something visible rather than just speaking about hypothetical changes :-)

> 
> > I also talked to Derick about this and he agrees that immutable DateTime
> > objects would be desirable. I have talked to many other people who agreed
> > that the current behavior is weird.
> 
> I've talked to many people who agree it isn't ;)
> -- 
> Stanislav Malyshev, Software Architect
> SugarCRM: http://www.sugarcrm.com/
> (408)454-6900 ext. 227
> 
> -- 
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>