Re: cvs: pear /DB_Table package.xml /DB_Table/DB Table.php /DB_Table/DB/Table QuickForm.php
| From: | Lukas Kahwe Smith | Date: | Thu, 18 Aug 2005 07:40:07 +0000 |
| Subject: | Re: cvs: pear /DB_Table package.xml /DB_Table/DB Table.php /DB_Table/DB/Table QuickForm.php | ||
| References: | 1 2 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-39455@lists.php.net to get a copy of this message | ||
Alan Knowles wrote:
For the switch case you should put an extra line after the break;as per CS additional indenting is a no no :-) http://pear.php.net/manual/en/standards.control.php regards, Lukas+ break; + default: // use addRule() for all other elements(personally I indent the cases once more, but I suspect the package doesnt do that, so you are probably right to follow the existing code.)