Re: cvs: php4 /ext/domxml config.m4 php_domxml.c php_domxml.h

From: Date: Tue, 08 Jan 2002 17:31:50 +0000
Subject: Re: cvs: php4 /ext/domxml config.m4 php_domxml.c php_domxml.h
References: 1  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-8983@lists.php.net to get a copy of this message
At 03:17 PM 1/8/2002 +0100, derick@php.net wrote:
On Tue, 8 Jan 2002, Joao Prado Maia wrote: On Tue, 8 Jan 2002, Jaroslaw Kolakowski wrote:
jarkol              Mon Jan  7 19:31:26 2002 EDT
Modified files:
    /php4/ext/domxml        config.m4 php_domxml.c php_domxml.h
Log: - Added functions: htmldoc(), htmldocfile(), domxml_htmldumpmem().
Just to be really picky, but shouldn't these functions be named html_doc(), html_doc_file() and domxml_html_dump_mem() ? I understand that we already have xmldoc(), xmldocfile(), xmltree() and even domxml_dumpmem(), but I still think we should stick we the coding standards. It is an extra good thing that this extensions is still experimental so we don't have to worry about BC that much. Am I totally off base here ? +1 here.
From me too. We might have sinned in the past but we should at least make sure that new stuff doesn't pollute PHP even more :) Andi

« previous php.cvs (#8983) next »