Fix compiler warnings.
authorStefan Westerfeld <stefan@space.twc.de>
Fri, 26 Nov 2010 18:29:08 +0000 (19:29 +0100)
committerStefan Westerfeld <stefan@space.twc.de>
Fri, 26 Nov 2010 18:29:08 +0000 (19:29 +0100)
src/jacksampler.cc

index c09ea4a..70cea4c 100644 (file)
@@ -79,13 +79,13 @@ isNoteOff (const jack_midi_event_t& event)
 
 JackSampler::JackSampler() :
   n_output_ports (1),
-  voices (256),
   instrument (1),
+  instrument_count (0),
   pedal_down (false),
   release_delay_ms (0),
   release_ms (50),
   mout (0),
-  instrument_count (0)
+  voices (256)
 {
 }
 
@@ -118,8 +118,12 @@ JackSampler::process (jack_nframes_t nframes)
   jack_default_audio_sample_t *out_1;
   jack_default_audio_sample_t *out_2;
   out_1 = (jack_default_audio_sample_t *) jack_port_get_buffer (output_port_1, nframes);
+
   if (n_output_ports == 2)
-     out_2 = (jack_default_audio_sample_t *) jack_port_get_buffer (output_port_2, nframes);
+    out_2 = (jack_default_audio_sample_t *) jack_port_get_buffer (output_port_2, nframes);
+  else
+    out_2 = NULL; // should not be accessed, but this gets rid of a compiler warning
+
   void* port_buf = jack_port_get_buffer (input_port, nframes);
 
   jack_nframes_t event_count = jack_midi_get_event_count (port_buf);
@@ -127,7 +131,7 @@ JackSampler::process (jack_nframes_t nframes)
   jack_nframes_t event_index = 0;
 
   jack_midi_event_get (&in_event, port_buf, 0);
-  for (int i = 0; i < nframes; i++)
+  for (size_t i = 0; i < nframes; i++)
     {
       while ((in_event.time == i) && (event_index < event_count))
         {
@@ -137,7 +141,7 @@ JackSampler::process (jack_nframes_t nframes)
               /* note on */
               //printf ("note on %d mout = %f\n", in_event.buffer[1], mout);
               /* find unused voice */
-              int v = 0;
+              size_t v = 0;
               while ((v < voices.size()) && (voices[v].state != Voice::UNUSED))
                 v++;
               if (v != voices.size())
@@ -177,7 +181,7 @@ JackSampler::process (jack_nframes_t nframes)
               /* note off */
               //printf ("note off %d\n", in_event.buffer[1]);
 
-              for (int v = 0; v < voices.size(); v++)
+              for (size_t v = 0; v < voices.size(); v++)
                 {
                   if (voices[v].state == Voice::ON && voices[v].note == in_event.buffer[1])
                     {
@@ -202,7 +206,7 @@ JackSampler::process (jack_nframes_t nframes)
                   if (!pedal_down)
                     {
                       /* release voices which are sustained due to the pedal */
-                      for (int v = 0; v < voices.size(); v++)
+                      for (size_t v = 0; v < voices.size(); v++)
                         {
                           if (voices[v].pedal)
                             {
@@ -225,7 +229,7 @@ JackSampler::process (jack_nframes_t nframes)
       out_1[i] = 0.0;
       if (n_output_ports == 2)
        out_2[i] = 0.0;
-      for (int v = 0; v < voices.size(); v++)
+      for (size_t v = 0; v < voices.size(); v++)
         {
           if (voices[v].state != Voice::UNUSED)
             {
@@ -235,7 +239,7 @@ JackSampler::process (jack_nframes_t nframes)
              switch (voices[v].sample->channels)
                {
                case 1:
-                 if (ipos < (voices[v].sample->pcm_data.size() - 1))
+                 if (ipos < int (voices[v].sample->pcm_data.size() - 1))
                    {
                      double left = voices[v].sample->pcm_data[ipos];
                      double right = voices[v].sample->pcm_data[ipos + 1];
@@ -246,7 +250,7 @@ JackSampler::process (jack_nframes_t nframes)
                    }
                  break;
                case 2:  // assert (n_output_ports == 2);
-                 if (ipos * 2 < (voices[v].sample->pcm_data.size() - 3))
+                 if (ipos * 2 < int (voices[v].sample->pcm_data.size() - 3))
                    {
                      double left = voices[v].sample->pcm_data[ipos * 2];
                      double right = voices[v].sample->pcm_data[ipos * 2 + 2];
@@ -340,7 +344,7 @@ JackSampler::load_note (const Options& options, int note, const char *file_name,
     {
       uint64 r = gsl_data_handle_read (dhandle, pos, block.size(), &block[0]);
 
-      for (int i = 0; i < r; i++)
+      for (uint64 i = 0; i < r; i++)
         s.pcm_data.push_back (block[i]);
       pos += r;
     }
@@ -428,7 +432,7 @@ JackSampler::status()
   printf ("output ports:    %d\n", n_output_ports);
   printf ("instruments:     %d\n", instrument_count);
   printf ("active instr.:   %d\n", instrument);
-  printf ("total voices:    %d\n", voices.size());
+  printf ("total voices:    %zd\n", voices.size());
   printf ("\n");
   printf (" * unused        %d\n", unused);
   printf (" * on            %d\n", on);