[Phpmyadmin-devel] CheckStyle violations in export files

Michal Čihař michal at cihar.com
Tue Aug 2 15:17:38 CEST 2011


Dne Fri, 22 Jul 2011 22:34:55 +0530
Madhura Jayaratne <madhura.cj at gmail.com> napsal(a):

> Looking at the CheckStyle reports from the CI, all the export files show
> very high counts of indentation rule violations.
> The code in the 'else' block of 'if (isset($plugin_list))' condition is
> wrongly indented in all of these files.
> There are two possible ways to correct this.
> 1) Correctly indent the 'else' block
> 2) Add 'return' statement to the end of 'if' block and remove 'else {' part.
> Which one if preferred?

I'm not sure if return won't break something else, if you think it's
safe, go for 2, otherwise 1.

	Michal Čihař | http://cihar.com | http://blog.cihar.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20110802/1508b6ac/attachment.sig>

More information about the Developers mailing list