Re: cvs: php4 /ext/shmop php_shmop.h shmop.c
| From: | Andi Gutmans | Date: | Thu, 19 Oct 2000 23:12:11 +0000 |
| Subject: | Re: cvs: php4 /ext/shmop php_shmop.h shmop.c | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-2147@lists.php.net to get a copy of this message | ||
At 01:03 AM 10/20/00 +0200, Derick Rethans wrote:
Andi Gutmans wrote: I just realized this extension is like two weeks old and you're already adding aliases for backwards compatibility. This is really bad and the only reason we have aliases is for when they are REALLY needed. I suggest reverting your patch, or if you really feel uncomfortable with the previous names (which seem quite OK) then change them. I doubt there are many people using these functions already. But I personally think it's best to leave them the way they were. Do you agree with changing the function names to the new form, and remove the aliases? That seems the most sensible thing to do to me.Sure go ahead, just don't create aliases for such a new module :) You can put this in the NEWS file so that people that grab 4.0.4 will know. Thanks, Andi --- Andi Gutmans <andi@zend.com> http://www.zend.com/