Fix unreachable-break issue in ../xplat/models/cache/DefaultDiskCache.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.

Such statements once existed to prevent accidental fallthroughs in switch statements. However, this is no longer necessary in C++17 because `[[fallthrough]]` is used to indicate intentional fallthroughs and we raise compilation errors for fallthroughs that are not annotated with `[[fallthrough]]` using `-Wimplicit-fallthrough`.

For questions/comments, contact r-barnes.

 - If you approve of this diff, please use the "Accept & Ship" button :-)

Reviewed By: dtolnay

Differential Revision: D78275960

fbshipit-source-id: c22ddd30f7314ec5a030e8583a2234aba826e15a

Author: r-barnes

impl/ocean/devices/SceneTracker6DOF.cpp

@@ -19,7 +19,6 @@ std::string SceneTracker6DOF::SceneElementRoom::RoomObject::translateObjectType(
 	{
 		case OT_UNDEFINED:
 			return "UNDEFINED";
-			break;
 
 		case OT_PLANAR:
 			return "PLANAR";

impl/ocean/test/testcv/testdetector/testqrcodes/TestMicroQRCodeDecoder.cpp

@@ -178,7 +178,6 @@ bool TestMicroQRCodeDecoder::testMicroQRCodeDecoding(const double testDuration)
 					ocean_assert(false && "Never be here");
 					allSucceeded = false;
 					return false;
-					break;
 			}
 
 			allSucceeded = allSucceeded && qrcodeGenerated;