Re: [PATCH] Notice on array to string convertion
| From: | David Zülke | Date: | Sat, 04 Jun 2011 13:31:58 +0000 |
| Subject: | Re: [PATCH] Notice on array to string convertion | ||
| References: | 1 2 3 4 5 6 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-52885@lists.php.net to get a copy of this message | ||
I think for consistency's sake it should be a notice as the same is already done in
_convert_to_string(). In fact, couldn't zend_make_printable_zval() use _convert_to_string()?
- David
On 03.06.2011, at 09:57, Derick Rethans wrote:
> On Thu, 2 Jun 2011, Ilia Alshanetsky wrote:
>
>> I like the idea of an error message when this happens, but as few
>> other people in the thread had mentioned, I think it should be a
>> warning (E_WARNING) because the conversion results in data loss
>> (content of the array is replaced with "Array" string).
>
> Yup, E_WARNING sounds best to me too.
>
> Derick
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
Attachment: [application/pkcs7-signature] smime.p7s
Attachment: [application/pkcs7-signature] smime.p7s