#421 ID3 not working on Shoutcast, new cvs

[Dan]

First of all great job on the new visualization for showing
album art and track title. My problem comes in when im
trying to view ID3 info from my shoutcast. Instead of
giving me track title n artist like it used to, its just
showing me the filename of whatever the .sc file is.
These streams were working under 2.4 and the ID3 info
was working as well. If i switch back to 2.4 they work
perfect. I assume that the new patch just takes the
info direct from how it was named and displays it. Is
there a way to disable this in shoutcast or just disable it
all together. It would be great to know the name of the
song instead of the name of the file i created the
stream info for. Thanks

D

[Markus Zehnder]

Logged In: YES
user_id=57785

I'm willing to fix this bug, it's bothering me since I
switched from the 2.4 release to building XBMP from CVS...

Has anyone already investigated this issue?
Can someone give me some pointers where to look first or
already knows what changed where?

I'll start in FileShoutcast.cpp and work my way up...

I'm very good in reverse engineering from existing source
code but it might take a while... too busy with work... :(

zed

[Andreas Setterlind]

Logged In: YES
user_id=630186

same thing reported here:
www.xboxmediaplayer.de/cgi-bin/ib31/ikonboard.cgi?
act=ST;f=12;t=6099

but it's not clear if a clean re-install solved the issue or not(?)

[Markus Zehnder]

Logged In: YES
user_id=57785

Actually that post confirms that ID3 isn't working with the
current version: "...reinstall of an older version brought
them back."

I got the first overview of where and how title information
is drawn on the screen yesterday. So far nothing much has
changed since version 2.4... My assumption is that the meta
data of the playlist/shortcut is taken instead of the played
back music file...

I'm working on it...

[Markus Zehnder]

Logged In: YES
user_id=57785

ok here we go:

XBoxMediaPlayer.cpp CVS version 1.199
CApplication::RenderScreen() Ln 2386:
Remove commented out section from JM, starting with
"COMMENTED by JM as it's not needed here..."

Looks like it's needed for Shoutcast :)

Don't know if this causes any side effects, will test this
weekend and probably add a check if the media being played
back is a Shoutcast stream...

[Hullebulle]

Logged In: YES
user_id=692875

Thanks for the effort in hunting this bug down. :)
I think a dev will look into it soon.

[Jonathan Marshall]

Logged In: YES
user_id=851643

mrzed:

This is the code in RenderScreen() right? In a shoutcast stream,
I guess the metadata (id3 info) must change as the stream plays
the next song etc. So we must update the info every frame to
make sure we capture when it first changes.

I'll have a play tonight and just check it doesn't break the .cue
sheet code which is the only thing that doesn't use the stream
meta data (Gets it's data from the cue sheet).