#382 Break loop as soon as possible when looking for sync sequence

bitcraze · Dec 5, 2018 · a988301 · a988301
1 parent 27b6f16
commit a988301
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/utils/src/lighthouse/pulse_processor.c b/src/utils/src/lighthouse/pulse_processor.c
@@ -57,6 +57,7 @@ static void synchronize(pulseProcessor_t *state, int sensor, uint32_t timestamp,
         state->synchronized = true;
         state->lastSync = state->currentSync0;
         state->currentSync1 = state->currentSync0 + SYNC_SEPARATION;
+        break;
       }
     }
 

diff --git a/test/utils/src/lighthouse/test_pulse_processor.c b/test/utils/src/lighthouse/test_pulse_processor.c
@@ -306,7 +306,7 @@ void testThatGetSystemSyncTimeDoesNotReturnTimestampIfTooMuchTimestampsSpread()
 {
   // Fixture
   uint32_t unused = 0;
-  uint32_t syncTimes[8] = {1, 500, 3};
+  uint32_t syncTimes[8] = {1, 900, 3};
   size_t nSyncTimes = 3;
 
   // Test
-Original file line number
+Diff line change
@@ Expand Up @@
             state->synchronized = true;
             state->lastSync = state->currentSync0;
             state->currentSync1 = state->currentSync0 + SYNC_SEPARATION;
+            break;
           }
         }
@@ Expand Down @@