Skip to content

Commit 8083015

Browse files
r-barnesmeta-codesync[bot]
authored andcommitted
Fix unreachable-break issue in ti/cartographer/sonar/shared/SonarDcLatencyImpl.cpp +5
Summary: LLVM has a warning `-Wunreachable-code-break` which identifies `break` statements that cannot be reached. These compromise readability, are misleading, and may identify bugs. This diff removes such statements. For questions/comments, contact r-barnes. - If you approve of this diff, please use the "Accept & Ship" button :-) Reviewed By: meyering Differential Revision: D89225697 fbshipit-source-id: 2fd0d349b9b3d004d561fe60dd72fa5607fafdb2
1 parent 1a63fbf commit 8083015

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

moxygen/moqtest/MoQTestClient.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -568,15 +568,12 @@ AdjustedExpectedResult MoQTestClient::adjustExpected(
568568
switch (params_.forwardingPreference) {
569569
case (ForwardingPreference::ONE_SUBGROUP_PER_GROUP): {
570570
return adjustExpectedForOneSubgroupPerGroup(params);
571-
break;
572571
}
573572
case (ForwardingPreference::ONE_SUBGROUP_PER_OBJECT): {
574573
return adjustExpectedForOneSubgroupPerObject(params);
575-
break;
576574
}
577575
case (ForwardingPreference::TWO_SUBGROUPS_PER_GROUP): {
578576
return adjustExpectedForTwoSubgroupsPerGroup(header, params);
579-
break;
580577
}
581578
case (ForwardingPreference::DATAGRAM): {
582579
if (receivingType_ == ReceivingType::FETCH) {
@@ -585,7 +582,6 @@ AdjustedExpectedResult MoQTestClient::adjustExpected(
585582
return AdjustedExpectedResult::ERROR_RECEIVING_DATA;
586583
}
587584
return adjustExpectedForDatagram(params);
588-
break;
589585
}
590586
default: {
591587
break;

0 commit comments

Comments
 (0)