Re: cvs: php-bugs-web /include functions.inc

From: 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


« previous php.mirrors (#28348) next »