[Biopython-dev] [Bug 2216] New: Bio/Nexus/Tree.py class Tree adds extra space in to_string

[bugzilla-daemon at portal.open-bio.org]

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

           Summary: Bio/Nexus/Tree.py class Tree adds extra space in
                    to_string
           Product: Biopython
           Version: Not Applicable
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: minor
          Priority: P2
         Component: Main Distribution
        AssignedTo: biopython-dev at biopython.org
        ReportedBy: biopython-bugzilla at maubp.freeserve.co.uk


The unit test for Bio.Nexus (test_Nexus.py) is failing due to an extra space at
the end of the string representation of (some) trees:

Output  : "Test Tree module. ... tree tree1 = (((((('one should be punished,
for
 (that)!','isn''that [a] strange name?'),'t2 the name'),t8,t9),t6),t7),(t5,t1))
;\n"
Expected: "Test Tree module. ... tree tree1 = (((((('one should be punished,
for
 (that)!','isn''that [a] strange name?'),'t2 the
name'),t8,t9),t6),t7),(t5,t1));
\n"

I think this bug was introduced six months ago in revision 1.7 (fkauff) as part
of a switch from building the tree string by concatenation to doing
"".join(list of string) which should be faster.

(Note that I just checked in a minor change to the tree parser to cope with
some odd white space)


-- 
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.