Agreed.... Unless someone goes ahead and changes the get_line macro and places that call it, I'll do so when I get to the end of my second leg...

-Sara
----- Original Message ----- From: ""Wez Furlong"" <[EMAIL PROTECTED]>
Newsgroups: php.internals
To: "Andrei Zmievski" <[EMAIL PROTECTED]>
Cc: "PHP Internals" <internals@lists.php.net>; "Dmitry Stogov" <[EMAIL PROTECTED]>; "Frank M. Kromann" <[EMAIL PROTECTED]>
Sent: Monday, April 24, 2006 10:46 PM
Subject: Re: [PHP-DEV] ZSTR and NULL problems again


I think ZSTR(buf) should not be present in the macro:

#define php_stream_get_line(stream, buf, maxlen, retlen)
  _php_stream_get_line((stream), IS_STRING, (buf), (maxlen), 0,
  (retlen) TSRMLS_CC)

Let the caller apply ZSTR() to the argument manually, if that is required.

--Wez.

On 4/24/06, Andrei Zmievski <[EMAIL PROTECTED]> wrote:
        #define php_stream_get_line(stream, buf, maxlen, retlen)
_php_stream_get_line((stream), IS_STRING, ZSTR(buf), (maxlen), 0,
(retlen) TSRMLS_CC)


--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to