[Biopython-dev] [Bug 2760] proposal: enhancement for SeqIO.TabIO

bugzilla-daemon at portal.open-bio.org bugzilla-daemon at portal.open-bio.org
Fri Feb 13 08:00:09 EST 2009


http://bugzilla.open-bio.org/show_bug.cgi?id=2760





------- Comment #3 from biopython-bugzilla at maubp.freeserve.co.uk  2009-02-13 08:00 EST -------
(In reply to comment #0)
>
> this patch ..., and introduces a check to skip empty lines.
> 

That change is probably a good idea, but not that rather than :

if line != "" :
   #Do stuff...

I believe the following is considered better python style:

if line :
   #Do stuff...

I have updated CVS to ignore blank lines, and to give a more helpful ValueError
when trying to parse invalid files.  See revision 1.2,
http://cvs.biopython.org/cgi-bin/viewcvs/viewcvs.cgi/biopython/Bio/SeqIO/TabIO.py?cvsroot=biopython

Peter


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


More information about the Biopython-dev mailing list