RE: [PEAR-DEV] Mail_Mbox
| From: | Lukas Smith | Date: | Fri, 27 Dec 2002 21:48:26 +0000 |
| Subject: | RE: [PEAR-DEV] Mail_Mbox | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-11847@lists.php.net to get a copy of this message | ||
> -----Original Message-----
> From: Jesus M. Castagnetto [mailto:jcastagnetto@yahoo.com]
> Sent: Friday, December 27, 2002 10:41 PM
> To: Roberto Berts; pear-dev@lists.php.net
> Subject: Re: [PEAR-DEV] Mail_Mbox
>
> Hi Roberto
>
> +1 for the class
>
> Apart from the tab/space inconsistencies (a minor point), there is one
> nitpick
> in the mbox_test.php. When creating a new instance, you do not need to
do
> a
> reference assignment, unless you are using a factory method (which
might
> or
> might not be static), i.e.:
>
> $mbox = new Mail_Mbox();
>
> should be enough.
But it is still a good idea to do it, because otherwise you cannot add a
pear destructor later on and rely on it working.
http://pear.php.net/manual/en/class.pear.php
regards,
Lukas