Re: cvs: pear /Spreadsheet_Excel_Writer/Writer Parser.php

I prefer my coments to be _together_ with the code they refer to. Why do the changes contradict coding standards? -------------------------------- Hi Xavier: On Mon, Nov 07, 2005 at 11:26:14PM -0000, Xavier Noguer Gallego wrote: > xnoguer Mon Nov 7 18:26:14 2005 EDT > Modified files: > /pear/Spreadsheet_Excel_Writer/Writer Parser.php > Log: > - reversing cosmetic changes so comments are readable again ... snip ... > - } elseif (preg_match('/^\$?([A-Ia-i]?[A-Za-z])\$?(\d+)$/', $token)) { > - // match references like A1 or $A$1 > + } > + // match references like A1 or $A$1 > + elseif (preg_match('/^\$?([A-Ia-i]?[A-Za-z])\$?(\d+)$/',$token)) > + { ... many more like this snipped. The question of "readable" is in the eye of the beholder. To me, it is far less clear what the comment refers to. My personal preference aside, the changes made here contradict the PEAR coding standards. Please apply the patch in reverse. Thanks, --Dan