[development] comment module: minor issue unreachable code

Bèr Kessels ber at webschuur.com
Wed Aug 22 13:25:30 UTC 2007


Op woensdag 22 augustus 2007, schreef Earnie Boyd:
> As you can see I don't think break is always a good thing, it should
> either be return or break and return should not be embodied in the
> case.  

having more then one return in a function is generally a (very) bad practice. 
It makes the code hard to read, and far more complex then needed.

Bèr
-- 
Drupal, Ruby on Rails and Joomla! development: webschuur.com | Drupal hosting: 
www.sympal.nl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.drupal.org/pipermail/development/attachments/20070822/2d01c0b1/attachment.pgp 


More information about the development mailing list