On Tue, Jan 16, 2018 at 01:21:36AM +0100, Ricardo Mones wrote:
> According to eyeD3 output setting multiple TCON frames seems to succeed:
> …
> Setting TPOS text frame to '1/1'
> Setting TRCK text frame to '1/13'
> Setting TCON text frame to 'Rock'
> Setting TCON text frame to 'Dance'
> Setting TCON text frame to 'Ballad'
> Setting TDRC text frame to '2016'
> …
>
> Nevertheless, when looking with other tools one of them is gone, just
> checked with hexdump and you can see that, surprisingly, the frame is
> not saved to file and only two of the above three are present:
> …
> 00015200 54 43 4f 4e 00 00 00 07 00 00 03 42 61 6c 6c 61 |TCON.......Balla|
> 00015210 64 54 43 4f 4e 00 00 00 05 00 00 03 52 6f 63 6b |dTCON.......Rock|
> 00015220 54 44 52 43 00 00 00 05 00 00 00 32 30 31 36 54 |TDRC.......2016T|
> 00015230 44 52 4c 00 00 00 05 00 00 00 32 30 31 36 54 49 |DRL.......2016TI|
> …
>
> Trying again with a different set of TCON values on the same file:
> …
> Setting TCON text frame to 'Pop'
> Setting TCON text frame to 'Eurodance'
> Setting TCON text frame to 'Classic'
> …
>
> Gives even worse results:
> …
> 00015200 54 43 4f 4e 00 00 00 08 00 00 03 43 6c 61 73 73 |TCON.......Class|
> 00015210 69 63 54 43 4f 4e 00 00 00 05 00 00 03 52 6f 63 |icTCON.......Roc|
> 00015220 6b 54 44 52 43 00 00 00 05 00 00 00 32 30 31 36 |kTDRC.......2016|
Hi and thank you for reporting.
In a clean file eyeD3 just sets the last TCON:
$ eyeD3 --text-frame=TCON:Rock --text-frame=TCON:Ballad \
--text-frame=TCON:Pop /tmp/1.mp3
…
Setting b'TCON' text frame to 'Rock'
Setting b'TCON' text frame to 'Ballad'
Setting b'TCON' text frame to 'Pop'
…
1.mp3 will have only `Pop`, checked both with hexdump and eyed3 himself.
(the logging is misleading)
Looking at your output I guess eyeD3 overwrites only first met TCON frame
(with the last new value requested) and leave the second TCON frame alone.
If you can confirm this I'll retitle and change priority accordingly.
Regards,
Gaetano
