leosongwei / mutagen

Automatically exported from code.google.com/p/mutagen
GNU General Public License v2.0
0 stars 0 forks source link

Trailing separators in text fields produce spurious slashes output #142

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I've gotten a mp3 file in which text fields are padded with null bytes after 
the text. Mutagen treats each pair of null bytes as a separator between 
multiple values of the field.

So the output becomes (something like)
TALB=Album /  /  /  /  / 

Ideally, text fragments with 0 length should be ignored.

Original issue reported on code.google.com by joao.ei...@gmail.com on 14 Apr 2013 at 4:19

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by joao.ei...@gmail.com on 14 Apr 2013 at 4:20

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by reiter.christoph@gmail.com on 14 Apr 2013 at 11:40

GoogleCodeExporter commented 9 years ago
After looking into it I disagree that mutagen should do anything different 
here. The result is perfectly valid and treating it differently would disallow 
empty strings in a multi value case.

"All text information frames supports multiple strings, stored as a null
separated list, where null is reperesented by the termination code
for the charater encoding."

So, based on that I'm closing this. If you have any further information or 
disagree feel free to post here.

Original comment by reiter.christoph@gmail.com on 4 Sep 2013 at 2:27