Sebastian Mendel a écrit :
Marc Delisle schrieb:
Hi,
The latest commit to common.lib.php does not respect PEAR standards, IMO. Too much white space.
Have a look at how they recommend to format "if":
if ($arg1 == 'good' || $arg1 == 'fair') { $this->foo = $arg1; return 1; } elseif ($arg1 == 'poor' && $arg2 > 1) { $this->foo = 'poor'; return 2; } else { return false; }
mhm ... grr ..
i really dont like how PEAR coding standard handles whitespaces
... i find it looks much better and is in most cases more readable with whitespaces ... but i do as our documentation saith! ;-)
:)
Note that at least for me, it's not clear what they recommend for things like the "not" operator in
if ( ! PMA_auth_check() ) {
since they recommend a space before and after operators like "||".
Marc