[Phpmyadmin-devel] CheckStyle violations in export files

[Michal Čihař]

Hi

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>