Re: cvs: pear /Spreadsheet_Excel_Writer/Writer Worksheet.php
| From: | Daniel Convissor | Date: | Fri, 18 Nov 2005 16:01:31 +0000 |
| Subject: | Re: cvs: pear /Spreadsheet_Excel_Writer/Writer Worksheet.php | ||
| References: | 1 | Groups: | php.pear.cvs |
| Request: | Send a blank email to pear-cvs+get-36364@lists.php.net to get a copy of this message | ||
Hello Again Xavier:
On Sat, Nov 12, 2005 at 11:18:25PM -0000, Xavier Noguer Gallego wrote:
> Modified files:
> /pear/Spreadsheet_Excel_Writer/Writer Worksheet.php
>
> - $this->_append($header.$data.$str);
> + $this->_prepend($header.$data.$str);
...
> - $this->_append($header . $data . $str);
> + $this->_prepend($header . $data . $str);
> }
Noticed an inconsistency in the spacing between concatenations. The
convention in PEAR, though I don't think it's documented in the CS, is to
have spaces between them.
Yours pedantically,
--Dan
--
T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y
data intensive web and database programming
http://www.AnalysisAndSolutions.com/
4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409