[PEPr] Comment on Encryption::GPG
| From: | Matthew Fonda | Date: | Thu, 24 Mar 2005 02:49:58 +0000 |
| Subject: | [PEPr] Comment on Encryption::GPG | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-36849@lists.php.net to get a copy of this message | ||
Matthew Fonda (http://pear.php.net/user/mfonda) has commented on the proposal for Encryption::GPG.
Comment:
Looks alright to me.
Is there any way you could make this package not require PHP5? It doesnt
seem like you necessarily need to use it. Perhaps just use PEAR_Error
instead of exceptions. Also, PEAR and the GPL don't mix. Maybe consider
using the PHP license.
Also, make sure everything is up to par with the PEAR coding standards.
Private methods should still follow standards, so for example you would
use _openSubprocess() instead of open_subprocess(). And on your if
statements, make sure you use { and } even if it is just a one liner. Your
class should also be named Crypt_GPG.
Other than that it looks pretty good.
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=229
--
Sent by PEPr, the automatic proposal system at http://pear.php.net