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

Bug #1473798 reported by Lassi Ylikojola
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Seq24
New
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 ){

Revision history for this message
Lassi Ylikojola (lassi-ylikojola) wrote :
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.