Re: Mail/mime.php changes
| From: | Jon Parise | Date: | Sun, 16 Sep 2001 17:38:59 +0000 |
| Subject: | Re: Mail/mime.php changes | ||
| References: | 1 2 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-1952@lists.php.net to get a copy of this message | ||
On Sun, Sep 16, 2001 at 07:34:21PM +0200, Martin Jansen wrote:
> >> '--'.$orig_boundary . MAIL_MIME_CRLF .
> >> 'Content-Type: multipart/alternative;' . MAIL_MIME_CRLF . chr(9) .
> >> 'boundary="'.$sec_boundary.'"'.MAIL_MIME_CRLF .
> >> MAIL_MIME_CRLF .
> >
> >Please be consistent with adding spaces around the dot operator.
> >There should be one space before it and one space after it.
>
> Uhm, I'm used to work with $var1.$var2 (without spaces before and
> after the dot). I can't see any advantage in using spaces. Or did
> I miss something?
I think it's looks cluttered and inconsistent, as indicated in
the above code snippet. The dot operator should be treated like
any other operator and padded with spaces. There's no good
reason to try and scrunch everything together, in my opinion.
--
Jon Parise (jon@csh.rit.edu) . Information Technology (B.S.)
http://www.csh.rit.edu/~jon/ : Computer Science House
Member