Re: cvs: php-bugs-web /include functions.inc
| From: | Wez Furlong | Date: | Sun, 05 Jun 2005 14:29:53 +0000 |
| Subject: | Re: cvs: php-bugs-web /include functions.inc | ||
| References: | 1 2 3 | Groups: | php.mirrors |
| Request: | Send a blank email to php-mirrors+get-28348@lists.php.net to get a copy of this message | ||
Trailing commas are a good habit when using a revision control system; if you later add a new element, the diff will show only one line, rather than a slightly less clear two lines (one for the added comma, one for the actual added line).
And Helgi's other reasons are also good :)
--Wez.
Helgi Þormar wrote:
On Sun, 2005-06-05 at 16:42 +1000, mazzanet wrote:Maybe remove the , after "Irrelevant". :)No, there is no need, it works with that extra comma and actually helps prevent parse errors from happening if some person adds a new option at the bottom, forgets to add the comma and to test and just commits, plus there is no need to nitpick on such small thing when you look at the rest of php-bugs-web ;) - Helgi