MythMusic: handle streams that repeat same metadata in changing strings #791
Conversation
Fixes MythTV#790 by parsing the string and updating metadata only if title/artist/album changes. This ignores extraneous changes like UTC=time of http://ice-sov.musicradio.com/ClassicFMMP3
Rather, preserve his play length to that point in time
|
Thanks for this. FYI I had git problems after Gary updated his build script and my efforts have not yet improved matters; so at present I can't comment. |
|
I'm back now with master plus these 2 PRs. visualisers with radio streams seem fine now. Maybe it would be better to show track info immediately on going to full-screen mode, but that might get to seem intrusive. |
This might be optional since we just came from a screen that showed what is playing. User can hit INFO anytime he wants a reminder. But I have thought that hitting keys like NEXTTRACK, FFWD or SPEEDUP should probably show the trackinfo so that you can see what you are doing. Currently you must hit INFO first if you wish to see, otherwise you are operating in the blind. Checking on video playback, skipping ahead or going FFWD does pop the OSD and progress bar, so I think we should do this on fullscreen visualizer as well. |
|
OK, I had not realised that INFO would work there. Going via Menu seemed clumsy - and, as you say, the screen just left was showing the info anyway. Looking good. |
Fixes #790 by parsing the changed metadata string and accepting it only if title/artist/album changed.
The LOG of raw data string and its parse is moved to DEBUG level so that stations repeating the same StreamTitle in a changing string don't spam an INFO log. Actual changes to StreamTitle are still logged to INFO as before only at first detection of the change.