Commit 114d1d46ac03cd1e3c509f769b99b542c74e904c

Ryan C. Gordon 2021-09-27T23:18:25

direct3d: Remove unnecessary render target support check. Direct3D 9 dictates that caps.NumSimultaneousRTs must always be at least 1, which is to say that Direct3D 9 level hardware must always support render targets. (caps.NumSimultaneousRTs is meant to show if you can draw to multiple render targets in a single draw call.) We had already hardcoded SDL_RENDERER_TARGETTEXTURE as available earlier in the function anyhow. Fixes #4781.

1
2
3
4
5
6
7
8
9
10
11
12
13
14
diff --git a/src/render/direct3d/SDL_render_d3d.c b/src/render/direct3d/SDL_render_d3d.c
index a567b78..00a0255 100644
--- a/src/render/direct3d/SDL_render_d3d.c
+++ b/src/render/direct3d/SDL_render_d3d.c
@@ -1908,9 +1908,6 @@ D3D_CreateRenderer(SDL_Window * window, Uint32 flags)
     IDirect3DDevice9_GetDeviceCaps(data->device, &caps);
     renderer->info.max_texture_width = caps.MaxTextureWidth;
     renderer->info.max_texture_height = caps.MaxTextureHeight;
-    if (caps.NumSimultaneousRTs >= 2) {
-        renderer->info.flags |= SDL_RENDERER_TARGETTEXTURE;
-    }
 
     if (caps.PrimitiveMiscCaps & D3DPMISCCAPS_SEPARATEALPHABLEND) {
         data->enableSeparateAlphaBlend = SDL_TRUE;