Refactor: Removed ~0u conversion as I had forgotten that I had resolved this problem already.

Author: klukaszek

include/SDL3/SDL_gpu.h

@@ -304,11 +304,10 @@
 #ifndef SDL_gpu_h_
 #define SDL_gpu_h_
 
-#include <SDL3/SDL_gpu.h>
+#include <SDL3/SDL_stdinc.h>
 #include <SDL3/SDL_pixels.h>
 #include <SDL3/SDL_properties.h>
 #include <SDL3/SDL_rect.h>
-#include <SDL3/SDL_stdinc.h>
 #include <SDL3/SDL_surface.h>
 #include <SDL3/SDL_video.h>
 

src/gpu/SDL_gpu.c

@@ -318,7 +318,6 @@ void SDL_GPU_BlitCommon(
         1,
         NULL);
 
-
     viewport.x = (float)info->destination.x;
     viewport.y = (float)info->destination.y;
     viewport.w = (float)info->destination.w;
@@ -1412,15 +1411,6 @@ SDL_GPURenderPass *SDL_BeginGPURenderPass(
                 SDL_assert_release(!"Cannot cycle color target when load op is LOAD!");
             }
 
-            // Get hint to check for "webgpu"
-            const char *backend = SDL_GetHint(SDL_HINT_GPU_DRIVER);
-            bool is_webgpu = SDL_strcasecmp(backend, "webgpu") == 0;
-
-            // WebGPU uses ~0u for default layer_or_depth_plane, however this causes issues with other backends
-            if (color_target_infos[i].layer_or_depth_plane == ~0u && !is_webgpu) {
-                SDL_memset((void *)&color_target_infos[i].layer_or_depth_plane, 0, sizeof(color_target_infos[i].layer_or_depth_plane));
-            }
-
             if (color_target_infos[i].store_op == SDL_GPU_STOREOP_RESOLVE || color_target_infos[i].store_op == SDL_GPU_STOREOP_RESOLVE_AND_STORE) {
                 if (color_target_infos[i].resolve_texture == NULL) {
                     SDL_assert_release(!"Store op is RESOLVE or RESOLVE_AND_STORE but resolve_texture is NULL!");

src/gpu/webgpu/SDL_gpu_webgpu.c

@@ -33,15 +33,15 @@
 #include <regex.h>
 
 // TODO: REMOVE
-// Code compiles without these but my IDE is complaining without them
-#include "../../../include/SDL3/SDL_atomic.h"
-#include "../../../include/SDL3/SDL_gpu.h"
+// Code compiles without these but my LSP freaks out without them
+#include <SDL3/SDL_gpu.h>
+#include <SDL3/SDL_atomic.h>
 
-// I currently have a copy of the webgpu.h file in the include directory:
+// I currently have a copy of the webgpu.h file in the local include directory:
 // - usr/local/include/webgpu/webgpu.h
 // it can be downloaded from here: https://github.com/webgpu-native/webgpu-headers/blob/main/webgpu.h
 //
-// THIS IS A TEMPORARY SOLUTION
+// The code compiles without it as long as Emscripten is used and the -sUSE_WEBGPU flag is set
 #include <webgpu/webgpu.h>
 
 #define MAX_UBO_SECTION_SIZE          4096 // 4   KiB
@@ -2202,6 +2202,7 @@ void WebGPU_BeginRenderPass(SDL_GPUCommandBuffer *commandBuffer,
 
     WebGPUCommandBuffer *wgpu_cmd_buf = (WebGPUCommandBuffer *)commandBuffer;
     if (!wgpu_cmd_buf || colorAttachmentCount == 0) {
+        SDL_LogError(SDL_LOG_CATEGORY_GPU, "Invalid parameters for render pass");
         return;
     }
 
@@ -2286,6 +2287,7 @@ void WebGPU_BeginRenderPass(SDL_GPUCommandBuffer *commandBuffer,
         .depthStencilAttachment = depthStencilAttachmentInfo ? &depthStencilAttachment : NULL
     };
 
+    // Create the render pass encoder
     wgpu_cmd_buf->renderPassEncoder =
         wgpuCommandEncoderBeginRenderPass(wgpu_cmd_buf->commandEncoder, &renderPassDesc);