[PEPr] Comment on Networking::Net_TrackBack
| From: | PEPr | Date: | Sat, 10 Apr 2004 19:04:08 +0000 |
| Subject: | [PEPr] Comment on Networking::Net_TrackBack | ||
| Groups: | php.pear.dev | ||
| Request: | Send a blank email to pear-dev+get-27408@lists.php.net to get a copy of this message | ||
Arnaud Limbourg (http://pear.php.net/user/arnaud) has commented on the proposal for
Networking::Net_TrackBack.
Comment:
It looks good.
General remark:
PEAR.php is included while it is only used for errors.
You probably want to use raiseError instead of throwError. I would then suggest you write a custom
raiseError method which includes PEAR.php on demand. That will avoid loading the entire class every
time.
There are some minor CS issues :-)
}
else {
should be
} else {
require_once('PEAR.php');
should be
require_once 'PEAR.php';
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=48
--
Sent by PEPr, the automatic proposal system at http://pear.php.net