External midi clock not working on linux x86_64 (fedora20, Fedora 21)

Bug #1473798 reported by Lassi Ylikojola on 2015-07-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Seq24
Undecided
Unassigned

Bug Description

Newest code from bzr does not send midi clock correctly for my korg volga bass hardware synthesizer.
I used to compile 0.9.2 code for this reason but currently it segfaults on my system(Fedora 21). This was due to lash changes fixed in revision 123 i believe.
I tracked the midi clock problem down to revision 126 code in perform.cpp. Changing the variable 'midi_clock' type back to double fixes it for me.
I used 0.9.2 code + all the available patches since configure scripts seem to be missing in bzr at the moment.

--- perform.cpp 2015-07-13 01:50:12.782994665 +0300
+++ perform.cpp 2015-07-13 01:56:15.581375112 +0300
@@ -1306,7 +1306,7 @@
         /* tick and tick fraction */
         double current_tick = 0.0;
         double total_tick = 0.0;
- long clock_tick = 0;
+ double clock_tick = 0;
         long delta_tick_frac = 0;

         long stats_total_tick = 0;
@@ -1404,7 +1404,7 @@
             /* get delta ticks, delta_ticks_f is in 1000th of a tick */
             long long delta_tick_num = bpm * ppqn * delta_us + delta_tick_frac;
             long long delta_tick_denom = 60000000;
- long delta_tick = (long)(delta_tick_num / delta_tick_denom);
+ double delta_tick = (double)(delta_tick_num / delta_tick_denom);
             delta_tick_frac = (long)(delta_tick_num % delta_tick_denom);

             if (m_usemidiclock) {
@@ -1578,7 +1578,7 @@
              * as soon as jack gets a good lock on playback */

             if (init_clock) {
- m_master_bus.init_clock( clock_tick );
+ m_master_bus.init_clock( (long)clock_tick );
                 init_clock = false;
             }

@@ -1602,7 +1602,7 @@
                 //printf( "play[%d]\n", current_tick );

                 /* midi clock */
- m_master_bus.clock( clock_tick );
+ m_master_bus.clock( (long)clock_tick );

                 if ( global_stats ){

Guido Scholz (gscholz) wrote :

Hi Lassi,
may be you could contact Sebastien Alaiwan regarding this topic.

BTW, you can create the configure scripts by your self, "autoreconf -i" is your friend (see README file).
Guido

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments