Code Cleanup: Fix tools

All the fix tools share common code with an empty else
clause markeed with XXX. removed the redundant code.

src/FixGPS.cpp

@@ -118,10 +118,7 @@ FixGPS::postProcess(RideFile *ride, DataProcessorConfig *)
             ride->command->setPointValue(j, RideFile::lon, ride->dataPoints()[lastgood]->lon);
             errors++;
         }
-    } else {
-        // they are all bad!!
-        // XXX do nothing?
-    }
+    } 
     ride->command->endLUW();
 
     if (errors) {

src/FixGaps.cpp

@@ -145,11 +145,6 @@ FixGaps::postProcess(RideFile *ride, DataProcessorConfig *config=0)
     // if the number of duration / number of samples
     // equals the recording interval then we don't need
     // to post-process for gaps
-    // XXX commented out since it is not always true and
-    //     is purely to improve performance
-    //if ((ride->recIntSecs() + ride->dataPoints()[ride->dataPoints().count()-1]->secs -
-    //    ride->dataPoints()[0]->secs) / (double) ride->dataPoints().count() == ride->recIntSecs())
-    //    return false;
 
     // Additionally, If there are less than 2 dataPoints then there
     // is no way of post processing anyway (e.g. manual workouts)

src/FixHRSpikes.cpp

@@ -165,9 +165,6 @@ FixHRSpikes::postProcess(RideFile *ride, DataProcessorConfig *config=0)
             ride->command->setPointValue(j, RideFile::hr, ride->dataPoints()[lastgood]->hr);
             spikes++;
         }
-    } else {
-        // they are all bad!!
-        // XXX do nothing?
     }
 
     ride->command->endLUW();	// End of LogicalUnitOfWork