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

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

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
>
>