View Issue Details

IDProjectCategoryView StatusLast Update
0008590MMW v4Otherpublic2011-11-03 16:34
Reporterpeke Assigned To 
PriorityimmediateSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product Version4.0 
Target Version4.0Fixed in Version4.0 
Summary0008590: ID3 Tags are not read correctly.
DescriptionSome MP3 ID3v2.3 Tags are not read correctly.

Sample file is uploaded to FTP.
TagsNo tags attached.
Fixed in build1452

Activities

peke

2011-10-29 23:04

developer   ~0028523

Assigning to you. Assign to Jiri if needed.

Ludek

2011-10-30 19:59

developer   ~0028527

Last edited: 2011-10-30 20:00

Can you tell me what exactly is not read correctly? Both MM and dbPowerAmp give me exact same series of asian characters. I don't see a difference.

peke

2011-10-30 20:14

developer   ~0028529

AIMP, WMP, Winamp Read it correctly.

If you open file in Hex Editor you will see correct Tags.

Based on my reverse engineering Year Tag is corrupted.

Ludek

2011-10-30 22:19

developer   ~0028531

Last edited: 2011-10-30 22:20

The Year tag (TYER) contains only the one unicode character which is not a valid value, therefore MediaMonkey doesn't show the value although some apps show it despite the fact it is an invalid value.

Ludek

2011-10-30 23:12

developer   ~0028532

Last edited: 2011-10-30 23:14

Otherwise there is missing BOM characters to indicate little/big-endian encoding. The BOM (byte order) is missing and some apps (like MM and dbPowerAmp) suppose it is big-endian encoded, but the others (WMP, Winamp) suppose it is little-endian encoded.

The latter seems to be more compatible so I changed it in build 1452, but the problem is rather in the file tag that is missing the byte order info.

peke

2011-10-30 23:30

developer   ~0028533

Last edited: 2011-10-30 23:32

I agree with you, that is more compatible.

To ensure full compatibility maybe it would be wise to add small INI option to force big Endian and cover Asian lngs that will be in big endian?

ForceBigEndian=0 (0=False,1=True)

(If Missing from INI -> False, Default values will be also false)

Ludek

2011-10-31 13:10

developer   ~0028535

Last edited: 2011-10-31 13:10

I think this is not needed. This case is just about handling MP3 tags that include unicode chars, but byte order info is missing. No need for an option.

peke

2011-11-03 16:34

developer   ~0028608

Verified 1453