Re: [PHP-CVS] svn: /php/php-src/trunk/ NEWS ext/json/json.c ext/json/php_json.h ext/json/tests/serialize.phpt

2010-05-06 Thread Pierre Joye
On Thu, May 6, 2010 at 12:59 PM, Hannes Magnusson
 wrote:
> On Thu, May 6, 2010 at 00:48, Sara Golemon  wrote:
>> pollita                                  Wed, 05 May 2010 22:48:14 +
>>
>> Revision: http://svn.php.net/viewvc?view=revision&revision=299037
>>
>> Log:
>> Add JSON_Serializable interface
>
> Slight bikeshedding.. but we don't use underscore for interface/classnames...

And I would go with JsonSerializable.

However, where does it come from? Does it have to be an interface or
why not using __toJson? I must have lived under a rock as I don't see
any proposal either, be RFC or internals.

Cheers,
--
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] svn: /php/php-src/trunk/ NEWS ext/json/json.c ext/json/php_json.h ext/json/tests/serialize.phpt

2010-05-06 Thread Hannes Magnusson
On Thu, May 6, 2010 at 00:48, Sara Golemon  wrote:
> pollita                                  Wed, 05 May 2010 22:48:14 +
>
> Revision: http://svn.php.net/viewvc?view=revision&revision=299037
>
> Log:
> Add JSON_Serializable interface

Slight bikeshedding.. but we don't use underscore for interface/classnames...

-Hannes

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php