[Bioperl-l] For CVS developers - potential pitfall with "return undef"

Torsten Seemann torsten.seemann at infotech.monash.edu.au
Tue May 30 04:18:45 EDT 2006


FYI Bioperl developers:

I just audited the bioperl-live CVS and found about 450 occurrences of 
"return undef".

Page 199 of "Perl Best Practices" by Damian Conway, and this URL
http://www.perl.com/lpt/a/2006/02/23/advanced_subroutines.html suggest:

"Use return; instead of return undef; if you want to return nothing. If 
someone assigns the return value to an array, the latter creates an 
array of one value (undef), which evaluates to true. The former will 
correctly handle all contexts."

So I'm guessing at least some of these 450 occurrences *could* result in 
bugs and should probably be changed.

Your opinion may differ :-)

-- 
Dr Torsten Seemann               http://www.vicbioinformatics.com
Victorian Bioinformatics Consortium, Monash University, Australia



More information about the Bioperl-l mailing list