Re: cvs: php-src /main SAPI.c /main/streams streams.c userspace.c /tests/lang bug24054.phpt
| From: | (Marcus B�rger) | Date: | Wed, 17 Sep 2003 13:37:30 +0000 |
| Subject: | Re: cvs: php-src /main SAPI.c /main/streams streams.c userspace.c /tests/lang bug24054.phpt | ||
| References: | 1 2 3 4 5 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-23797@lists.php.net to get a copy of this message | ||
Hello Jani,
Wednesday, September 17, 2003, 2:23:46 PM, you wrote:
> On Sat, 13 Sep 2003, Ard Biesheuvel wrote:
>>> Sascha and me had a deeper check upon this and it is absolutley NON
>>> portable. This is one of the C99 extensions only a few systems support.
>>> If we want the prefix 'z' then we could do it for our own php internal
>>> [v]s[np]printf implementations. But then we'd need to always use them
>>> and that's probably not the best idea. Any thoughts?
>>
>>In the userpsace.c case, just use %ld and use a cast to kill the
>>warning. Otherwise, I don't think %zd is really necessary.
> Please fix what you committed yourself.
%dz is fine for php_error_docref() now in HEAD.
--
Best regards,
Marcus mailto:helly@php.net