Re: Re: cvs: php4 /main main.c php.h

At 12:09 08.08.2002, derick@php.net wrote:

On Thu, 8 Aug 2002, Markus Fischer wrote:

    Wow!

    I'm impressed. That's one of the best ideas I've read on this
    list for months, and I really mean it that way. +1

Except for the naming of these functions (and perhaps order of parameters) it definitely sounds like a nice idea. But beware that get_active:function_name() can also return NULL.

spprintf uses it as a parameter, this function does not have a problem with %s being NULL. Buter perhaps we check for NULL and write something like "Unknown" in this case, or do we know better in tsrm build?

regards, Derick

    - Markus

On Wed, Aug 07, 2002 at 11:32:19PM +0100, Wez Furlong wrote :

This seems OK, but I have a suggestion: add a string parameter that refers to a location in the manual.

[...]

So, the C code would look something like this: php_error_docref(E_WARNING, "features.remote-files", "Cannot write to remote files using ftp");

[...]

Warning: Cannot write to remote files using ftp. <a href="http://php.net/manual/en/features.remote-files.php">Click Here for more information.</a> or if html_errors is off: Warning: Cannot write to remote files using ftp. See http://php.net/manual/en/features.remote-files.php for more information.

-- PHP Development Mailing List <http://www.php.net/> To unsubscribe, visit: http://www.php.net/unsub.php --------------------------------------------------------------------------- Did I help you? http://www.derickrethans.nl/link.php?url=giftlist Frequent ranting: http://www.derickrethans.nl/ ---------------------------------------------------------------------------

                 PHP: Scripting the Web - derick@php.net
                    All your branches are belong to me!
                SRM: Script Running Machine - www.vl-srm.net

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