Skip to content

Commit b98ffcf

Browse files
committed
Remove code coverage branch exclusions
1 parent cb74da3 commit b98ffcf

File tree

1 file changed

+0
-32
lines changed

1 file changed

+0
-32
lines changed

speeduino/pages.cpp

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -306,37 +306,25 @@ static page_iterator_t map_page_offset_to_entity(uint8_t pageNumber, uint16_t of
306306
switch (pageNumber)
307307
{
308308
case veMapPage:
309-
// LCOV_EXCL_BR_START
310-
// The first entity on the page has a missing branch not covered
311-
// No idea why, so exclude from branch coverage for the moment
312309
checkIsInTable(result, &fuelTable, offset);
313-
// LCOV_EXCL_BR_STOP
314310
break;
315311

316312
case ignMapPage: //Ignition settings page (Page 2)
317-
// LCOV_EXCL_BR_START
318313
checkIsInTable(result, &ignitionTable, offset);
319-
// LCOV_EXCL_BR_STOP
320314
break;
321315

322316
case afrMapPage: //Air/Fuel ratio target settings page
323-
// LCOV_EXCL_BR_START
324317
checkIsInTable(result, &afrTable, offset);
325-
// LCOV_EXCL_BR_STOP
326318
break;
327319

328320
case boostvvtPage: //Boost, VVT and staging maps (all 8x8)
329-
// LCOV_EXCL_BR_START
330321
checkIsInTable(result, &boostTable, offset);
331-
// LCOV_EXCL_BR_STOP
332322
checkIsInTable(result, &vvtTable, offset);
333323
checkIsInTable(result, &stagingTable, offset);
334324
break;
335325

336326
case seqFuelPage:
337-
// LCOV_EXCL_BR_START
338327
checkIsInTable(result, &trim1Table, offset);
339-
// LCOV_EXCL_BR_STOP
340328
checkIsInTable(result, &trim2Table, offset);
341329
checkIsInTable(result, &trim3Table, offset);
342330
checkIsInTable(result, &trim4Table, offset);
@@ -347,66 +335,46 @@ static page_iterator_t map_page_offset_to_entity(uint8_t pageNumber, uint16_t of
347335
break;
348336

349337
case fuelMap2Page:
350-
// LCOV_EXCL_BR_START
351338
checkIsInTable(result, &fuelTable2, offset);
352-
// LCOV_EXCL_BR_STOP
353339
break;
354340

355341
case wmiMapPage:
356-
// LCOV_EXCL_BR_START
357342
checkIsInTable(result, &wmiTable, offset);
358-
// LCOV_EXCL_BR_STOP
359343
checkIsInTable(result, &vvt2Table, offset);
360344
checkIsInTable(result, &dwellTable, offset);
361345
checkIsInEmpty(result, 8U, offset);
362346
break;
363347

364348
case ignMap2Page:
365-
// LCOV_EXCL_BR_START
366349
checkIsInTable(result, &ignitionTable2, offset);
367-
// LCOV_EXCL_BR_STOP
368350
break;
369351

370352
case veSetPage:
371-
// LCOV_EXCL_BR_START
372353
checkIsInRaw(result, &configPage2, sizeof(configPage2), offset);
373-
// LCOV_EXCL_BR_STOP
374354
break;
375355

376356
case ignSetPage:
377-
// LCOV_EXCL_BR_START
378357
checkIsInRaw(result, &configPage4, sizeof(configPage4), offset);
379-
// LCOV_EXCL_BR_STOP
380358
break;
381359

382360
case afrSetPage:
383-
// LCOV_EXCL_BR_START
384361
checkIsInRaw(result, &configPage6, sizeof(configPage6), offset);
385-
// LCOV_EXCL_BR_STOP
386362
break;
387363

388364
case canbusPage:
389-
// LCOV_EXCL_BR_START
390365
checkIsInRaw(result, &configPage9, sizeof(configPage9), offset);
391-
// LCOV_EXCL_BR_STOP
392366
break;
393367

394368
case warmupPage:
395-
// LCOV_EXCL_BR_START
396369
checkIsInRaw(result, &configPage10, sizeof(configPage10), offset);
397-
// LCOV_EXCL_BR_STOP
398370
break;
399371

400372
case progOutsPage:
401-
// LCOV_EXCL_BR_START
402373
checkIsInRaw(result, &configPage13, sizeof(configPage13), offset);
403-
// LCOV_EXCL_BR_STOP
404374
break;
405375

406376
case boostvvtPage2: //Boost, VVT and staging maps (all 8x8)
407-
// LCOV_EXCL_BR_START
408377
checkIsInTable(result, &boostTableLookupDuty, offset);
409-
// LCOV_EXCL_BR_STOP
410378
checkIsInRaw(result, &configPage15, sizeof(configPage15), offset);
411379
break;
412380

0 commit comments

Comments
 (0)