-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow scrobbling from internet radio streams #21
base: master
Are you sure you want to change the base?
Conversation
This is great. Thank you! |
After running for a while, I noticed some memory leakage...
I'll keep investigating this unless someone else beats me to it. |
I believe 10a4b0f fixes the memory leak. |
Indeed it does! However, this PR reintroduces the leak. Moving |
The last merge should have included the fix. |
What's status of this PR? valgrind does output many errors and reports memory leaks for me. |
You should definitely submit another PR. I wrote this a while back to solve a problem I had with a certain internet radio stream, and it might not (probably won't?) work for your use case. I also haven't looked closely at any memory leaks other than merging any fixes into my fork. |
parseSongTitle will be used to parse song titles in the format "Artist - Song Title" into the artist name and song title. Tests use the Catch C++ testing framework.
When playing an internet radio stream, the artist and song title cannot be pulled from ID3 tags, but if the stream title shows the current song in the format "Artist - Title" (which is common with most streams), we can parse that and scrobble the song.
Oone feature that would be pretty cool would be the ability to scrobble tracks from internet radio streams. Right now mpdas only pulls metadata from the ID3 tags associated with the music file using libmpdclient, but there is no such data associated with a stream url. Instead, mpd picks up that data as it streams music from the url. If the currently playing song's title provided by the stream is well-formed (i.e., in the format "Artist - Song Title"), we can parse that and use that to scrobble the song.