[PD-dev] [ pure-data-Bugs-1356455 ] [midiout] not working for ALSA

[SourceForge.net]

Bugs item #1356455, was opened at 2005-11-14 10:34
Message generated for change (Comment added) made by postlude
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1356455&group_id=55736

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: puredata-dev
Group: None
Status: Open
Resolution: None
Priority: 7
Submitted By: Jamie Bullock (postlude)
Assigned to: Miller Puckette (millerpuckette)
Summary: [midiout] not working for ALSA

Initial Comment:
 If [midiout] is used with ALSA on Linux, no output
is generated by the object. 

ALSA version: 1.0.9rc1

----------------------------------------------------------------------

>Comment By: Jamie Bullock (postlude)
Date: 2005-11-15 10:43

Message:
Logged In: YES 
user_id=1361897

Tested [midiout] sysex output with external hardware; works
fine.

----------------------------------------------------------------------

Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2005-11-15 09:20

Message:
Logged In: YES 
user_id=564396

as günter has noticed, there is a memory leak in
sys_alsa_putmidibyte()

attached is a patch against s_midi_alsa.c (in pd-0.39-1-CVS)
which should fix this

----------------------------------------------------------------------

Comment By: IOhannes m zmölnig (zmoelnig)
Date: 2005-11-14 17:33

Message:
Logged In: YES 
user_id=564396

the putbytes of the alsa-implementation does not get called.
the attached patch (s_midi.c, x_midi.c; pd=0.39-1-CVS)
should fix this.
the current alsa-implementation of [midiout] uses a kind of
sysex-wrapper to get the raw-data through the alsa-seq
interface; later think whether this is really a good idea,
in the meantime it works.

btw, since i changed the subject to not reflect the version
number of pd, i give it here: confirmed wrong behaviour in
pd-0.39-1 (both vanilla and devel); all pd-versions with
alsamidi until now should be affected

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1356455&group_id=55736