| Summary: | Native MIDI freezes. | ||
|---|---|---|---|
| Product: | SDL_mixer | Reporter: | Lukasz Pawelczyk <havner> |
| Component: | misc | Assignee: | Ryan C. Gordon <icculus> |
| Status: | RESOLVED FIXED | QA Contact: | Sam Lantinga <slouken> |
| Severity: | normal | ||
| Priority: | P2 | CC: | havner, manuel.montezelo, sezeroz |
| Version: | 1.2.8 | ||
| Hardware: | x86 | ||
| OS: | Linux | ||
|
Description
Lukasz Pawelczyk
2008-03-15 08:46:58 UTC
I didn't try to fix it, but I wrote alternative sequncer support for SDL-mixer based on alsa-seq. The code needs some cleanups but it already works very well. Would you be interested in its inclusion to SDL-mixer? I also got several questions about internal sdl-mixer functions behaviour. Error handling etc so this would fully follow sdl-mixer policy. Where could I ask them? Sure, I'd love to get a patch for ALSA native midi. You can ask the questions here, or send e-mail to the SDL mailing list. Thanks! Wow, that was a long time ago :-) Ok, I'll try to get this patch out of attic, dust it off a little and see what can be done with it. This patch is based on aplaymidi code which is GPL based but I had permission from it's author back then to relicense it to LGPL if it's going to be included in SDL_mixer. Small update. I've been working ont his code for the past few days. In general it is very close to be ready (finished lots of things that were left in my TODO 1.5 year ago). The only thing I'm still working on is gracefully stopping of ALSA events. I had some thread race previously that could crash ALSA on some occasions. I've managed to eliminate it but still as I look at that code there is very very rare case that could cause it to crash (although it didn't happen for me in practice). Anyway I hope this will be ready soon. Debian bug reports [1] and [2] seem related with this problem: [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=498984 [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=298982 I've removed the native MIDI code from SDL_mixer, since it was GPL and the new licensing is zlib. Lukasz, if you have that ALSA patch ready, please submit a new bug with it attached and let me know! :) |