search

layeh / gumble

gumble is a Mumble client implementation in Go (golang)
https://pkg.go.dev/mod/layeh.com/gumble
Mozilla Public License 2.0
172 stars 53 forks source link

Audio playback stops after 00:02:44 has elasped #5

Closed matthieugrieger closed 9 years ago

matthieugrieger commented 9 years ago

Hey there,

I came across a rather strange bug while testing out the Play() function from gumble_ffmpeg. I am using youtube-dl to download the audio from YouTube videos in m4a format at 256k, 44100Hz and the audio playback consistently stops 2 minutes and 44 seconds in.

I have even tested it using different YouTube videos, and it still stops right on 2 minutes 44 seconds. Maybe a memory leak issue or something? The audio playback sounds perfect up until it stops.

Here is the youtube-dl command I am using: youtube-dl --output ~/.mumbledj/songs/<youtube_id_here>.m4a --format m4a <youtube_id_here>

And here are the two songs I used to test it (I picked these at random): https://www.youtube.com/watch?v=5DLTlgzhxFE https://www.youtube.com/watch?v=emGri7i8Y2Y

Thanks!

ghost commented 9 years ago

I could not reproduce the behaviour with the first song, and the second song was not available in my country. I also tried an audio file that was 5:53 and it worked fine.

matthieugrieger commented 9 years ago

Strange, I guess it's something on my end then. I'll look around later and see if I can figure out what's going on. Thanks for trying it out! On Dec 21, 2014 5:19 AM, "Tim Cooper" notifications@github.com wrote:

I could not reproduce the behaviour with the first song, and the second song was not available in my country. I also tried an audio file that was 5:53 and it worked fine.

— Reply to this email directly or view it on GitHub https://github.com/layeh/gumble/issues/5#issuecomment-67770182.

ghost commented 9 years ago

Fixed in b366f54abe581ccab5984ab655f0a3d7a8db764e.

ghost commented 9 years ago

I have no idea why it worked for me two days ago, but not when I tried it again this morning. Thanks for the bug report, @matthieugrieger!

matthieugrieger commented 9 years ago

Oh wow, that is strange! Thanks for the fix, I'll try it out later. On Dec 23, 2014 4:50 AM, "Tim Cooper" notifications@github.com wrote:

I have no idea why it worked for me two days ago, but not when I tried it again this morning. Thanks for the bug report, @matthieugrieger https://github.com/matthieugrieger!

— Reply to this email directly or view it on GitHub https://github.com/layeh/gumble/issues/5#issuecomment-67947850.