Bug in Samfile.header property

[GoogleCodeExporter]

If you have any header field containing more than one colon (:) the split
command will return a sequence of >2 fields here:

# the following is clumsy as generators do not work?
x = {}
for field in fields[1:]: 
    key,value = field.split(":")
    if key not in VALID_HEADER_FIELDS[record]:
    ...
    ...

And the method will fail to produce a valid header and returns with an
'unpack' error.

Fix this by splitting differently:

splitfield = field.split(":")
key, value = splitfield[0], splitfield[1:]

Original issue reported on code.google.com by matthew....@gmail.com on 19 Nov 2009 at 10:42

[GoogleCodeExporter]

Thanks. I have used your suggestion, but use

key, value = field.split(":",1)

Thus, colon containing values will allow a colon as a character.

Original comment by andreas....@gmail.com on 4 May 2010 at 4:49

• Changed state: Fixed

[GoogleCodeExporter]

Original comment by andreas....@gmail.com on 4 May 2010 at 9:25