Re: [seq24-users] Midi Clock Sync patch (slave seq24 to external sequencers)

[Kevin M. <ke...@su...>]

going back to my old code, yeah, the rm1x seems to send CONTINUE with the
play button, and stop with the stop button.   then you can start over by
hitting the < back button, then play again...

Your new code in rev70 broke at least the continue functionality.    The
code I pasted below works with start/stop/continue/clock with rm1x, and
start/stop/clock on x0xb0x (no continue there as far as i can tell)...   not
sure about pos... or how to test that.

---
kevin meinert | http://www.subatomicglue.com


On Thu, Jun 25, 2009 at 11:26 PM, Kevin Meinert <ke...@su...>wrote:

>
> tested using x0xb0x -> seq24
>
> and yamaha rm1x -> seq24
>
> appears to work...
>
>
> however, neither of these tests continue, or set song position   (i tried
> the << and >> on the rm1x but it simply restarted the se24 at the beginning
> of the loop....   no idea what it's supposed to do, i didn't trace the
> messages......)
>
> Looking at the code, I don't think CLOCK should start the sequencer.   I'd
> do something more like this, much less drastic change to the old code too
> (should be safer):
>
>
>                     // Obey MidiTimeClock:
>                     if (ev.get_status() == EVENT_MIDI_START)
>                     {
>                         if (!m_midiclockrunning)
>                         {
>                             stop();
>                             start( false );
>                             m_midiclockrunning = true;
>                             m_usemidiclock = true;
>                             m_midiclocktick = 0;
>                             m_midiclockpos = 0;
>                         }
>                     }
>                     // midi continue: start from current pos.
>                     else if (ev.get_status() == EVENT_MIDI_CONTINUE)
>                     {
>                         if (!m_midiclockrunning)
>                         {
>                             m_midiclockrunning = true;
>                             start( false );
>                             //m_usemidiclock = true;
>                         }
>                     }
>                     else if (ev.get_status() == EVENT_MIDI_STOP)
>                     {
>                         // no harm stopping again, if there's a stuck note,
> it could be useful to reissue...
>                         //if (m_midiclockrunning)
>                         //{
>                             // do nothing, just let the system pause
>                             // since we're not getting ticks after the
> stop, the song wont advance
>                             // when start is recieved, we'll reset the
> position, or
>                             // when continue is recieved, we wont
>                             m_midiclockrunning = false;
>                             all_notes_off();
>                         //}
>                     }
>                     else if (ev.get_status() == EVENT_MIDI_CLOCK)
>                     {
>                          if (m_midiclockrunning)
>                             m_midiclocktick += 8;
>                     }
>                     // not tested (todo: test it!)
>                     else if (ev.get_status() == EVENT_MIDI_SONG_POS)
>                     {
>                          unsigned char a, b;
>                          ev.get_data( &a, &b );
>                          m_midiclockpos = ((int)a << 7) && (int)b;
>                     }
>
> ---
> kevin meinert | http://www.subatomicglue.com
>
>
> On Thu, Jun 25, 2009 at 3:19 PM, Guido Scholz <gui...@ba...>wrote:
>
>> Am Thu, 25. Jun 2009 um 12:45:35 -0500 schrieb Kevin Meinert:
>>
>> Hi all,
>>
>> > Let me know if you were planning to do it yourself and I'll ignore it.
>> :)
>> > Should be an easy change either way.
>>
>> done, but completely untested. Does anybody have spare time (and
>> equipment) for a test?
>>
>> Guido
>>
>> --
>> http://www.bayernline.de/~gscholz/ <http://www.bayernline.de/%7Egscholz/>
>> http://www.lug-burghausen.org/
>>
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v1.4.6 (GNU/Linux)
>>
>> iD8DBQFKQ9vek6cKJms5yBsRAknOAKCcHwLhqJUDU6vmj1KjTpyR47ORAwCghRPZ
>> KoBuuwxOJtRILv9g3Txdnys=
>> =El6s
>> -----END PGP SIGNATURE-----
>>
>>
>> ------------------------------------------------------------------------------
>>
>> _______________________________________________
>> seq24-users mailing list
>> seq...@li...
>> https://lists.sourceforge.net/lists/listinfo/seq24-users
>>
>>
>