[PEPr] +1 for Networking::Net_Gearman
| From: | Travis Swicegood | Date: | Thu, 24 Apr 2008 17:40:54 +0000 |
| Subject: | [PEPr] +1 for Networking::Net_Gearman | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-49886@lists.php.net to get a copy of this message | ||
Travis Swicegood (http://pear.php.net/user/tswicegood) has voted +1 on the proposal for
Networking::Net_Gearman.
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=548
Vote information:
http://pear.php.net/pepr/pepr-vote-show.php?id=548&handle=tswicegood
This vote is conditional. The condition is:
The test config contains a ":" instead of a PATH_SEPARATOR when setting up the tests.
Those should be runnable on multiple platforms (even if they fail).
The only other thing is various abstract classes, which we discussed off-line. The abstract keyword
is generally accepted to signify a class that is not directly instantiable because it is a
super-class, not just that a class is not supposed to be instantiated. I would much rather see
those specifically addressed via a private final __construct() that declared abstract so the proper
error message is raised when someone does try to instantiate one of those classes.
--
Sent by PEPr, the automatic proposal system at http://pear.php.net