WMA not playable with Winamp/WMP after saving

[GoogleCodeExporter]

https://bugs.launchpad.net/ubuntu/+source/quodlibet/+bug/705936

Affected file attached in the report.

Original issue reported on code.google.com by reiter.christoph@gmail.com on 22 Jan 2011 at 9:44

[GoogleCodeExporter]

This looks like a duplicate of issue #60, fixed in Mutagen 1.20. As a temporary 
workaround, the embedded picture can be removed (or the file can be loaded and 
re-saved in 1.20).

Original comment by joe.wreschnig@gmail.com on 22 Jan 2011 at 10:56

• Changed state: Duplicate

[GoogleCodeExporter]

Please reopen.

With any file posted in the report and some files from my library using mutagen 
trunk:

song = mutagen.File("file.wma")
for tag in song.tags.keys():
    del song.tags[tag]
    song.save()

results in and broken file.

Original comment by reiter.christoph@gmail.com on 23 Jan 2011 at 9:41

[GoogleCodeExporter]

Re-opening.

Original comment by joe.wreschnig@gmail.com on 23 Jan 2011 at 10:10

• Changed state: Accepted
• Added labels: Priority-High, Type-Defect
• Removed labels: Priority-Low, Type-Enhancement

[GoogleCodeExporter]

The header size wasn't updated after saving so it always deleted the difference 
between the size when it was first loaded and the size of all object. 
num_objects also needs to be updated, but it wasn't used anywhere so I made it 
local.

Since this never happens in QL as far as I know, the above report has to be 
something different. (re-saving, removing embedded pictures doesn't help)

Original comment by reiter.christoph@gmail.com on 24 Jan 2011 at 12:09

Attachments:

• mutagen_81.patch

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

It seems that similar to large objects (issue 60), all ASFGUIDAttribute objects 
need to be saved as metadata library objects and not as extended content 
description objects or normal metadata objects.

Original comment by reiter.christoph@gmail.com on 24 Jan 2011 at 3:19

[GoogleCodeExporter]

Original comment by nick.bou...@gmail.com on 29 Oct 2012 at 7:53

Attachments:

• m81.patch

[GoogleCodeExporter]

This issue was closed by revision r138.

Original comment by reiter.christoph@gmail.com on 31 Oct 2012 at 1:05

• Changed state: Fixed