Re: [PEPr] Comment on PHP::Autoloader
| From: | Markus Malkusch | Date: | Mon, 31 Jan 2011 10:16:32 +0000 |
| Subject: | Re: [PEPr] Comment on PHP::Autoloader | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-54043@lists.php.net to get a copy of this message | ||
"Michael Gauthier" <mike@silverorange.com>
> 4.) Class variables require a scope declaration (public/private/protected)
Which one is not declared?
> 6.) Package directory structure should follow the PEAR conventions:
OK, I will change that.
> 7.) Memcached and APC index backends would be a good idea.
Memcached is OK (and already implemented). APC is not suitable, as every
process has an own cache. It's a common szenario, that apache is configured in
a multiprocess environment. So users would wonder around why the autoloader is
still generating the index (until every process filled its cache). Plus on cli
it's totally unsuitable.
> 8.) Use single or double quotes, not both.
OK, on my todo list.
> 9.) Like Christian said, the GPL is not suitable for PEAR packages.
Yes, It's already on my todo list.
Markus