[d3d8] Remove some uses of goto in CopyRects

simifor · Jul 7, 2024 · f73a82e · f73a82e
1 parent 137dbeb
commit f73a82e
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/src/d3d8/d3d8_device.cpp b/src/d3d8/d3d8_device.cpp
@@ -620,6 +620,11 @@ namespace dxvk {
 
       res = D3DERR_INVALIDCALL;
 
+      auto unhandled = [&] {
+        Logger::debug(str::format("CopyRects: Hit unhandled case from src pool ", srcDesc.Pool, " to dst pool ", dstDesc.Pool));
+        return D3DERR_INVALIDCALL;
+      };
+
       switch (dstDesc.Pool) {
 
         // Dest: DEFAULT
@@ -659,7 +664,7 @@ namespace dxvk {
             case D3DPOOL_SCRATCH:
             default: {
               // TODO: Unhandled case.
-              goto unhandled;
+              return unhandled();
             }
           } break;
 
@@ -703,7 +708,7 @@ namespace dxvk {
             case D3DPOOL_SCRATCH:
             default: {
               // TODO: Unhandled case.
-              goto unhandled;
+              return unhandled();
             }
           } break;
 
@@ -759,7 +764,7 @@ namespace dxvk {
             case D3DPOOL_SCRATCH:
             default: {
               // TODO: Unhandled case.
-              goto unhandled;
+              return unhandled();
             }
           } break;
         }
@@ -768,13 +773,9 @@ namespace dxvk {
         case D3DPOOL_SCRATCH:
         default: {
           // TODO: Unhandled case.
-          goto unhandled;
+          return unhandled();
         }
       }
-
-    unhandled:
-      Logger::debug(str::format("CopyRects: Hit unhandled case from src pool ", srcDesc.Pool, " to dst pool ", dstDesc.Pool));
-      return D3DERR_INVALIDCALL;
 
     done:
       if (FAILED(res)) {