# sdl12-compat: FIXME removal: We \_could\_ request a simpler GL context, but don't.

**URL:** <https://discourse.libsdl.org/t/sdl12-compat-fixme-removal-we-could-request-a-simpler-gl-context-but-dont/38299>\
**Category:** SDL Commits\
**Created:** [August 19, 2022, 8:38pm UTC](https://discourse.libsdl.org/t/sdl12-compat-fixme-removal-we-could-request-a-simpler-gl-context-but-dont/38299 "2022-08-19T20:38:02Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![SDLPushPostBot](https://discourse.libsdl.org/user_avatar/discourse.libsdl.org/sdlpushpostbot/32/4370_2.png) [@SDLPushPostBot](https://discourse.libsdl.org/u/SDLPushPostBot)\
**Post date:** [August 19, 2022, 8:38pm UTC](https://discourse.libsdl.org/t/sdl12-compat-fixme-removal-we-could-request-a-simpler-gl-context-but-dont/38299/1 "2022-08-19T20:38:02Z")

</div>

> <https://github.com/libsdl-org/sdl12-compat/commit/d9708833642f1df2ee665f97054ebdab6b775f32>
>
> It's risky to find out we broke something this way when nothing is
> actually havi…ng a problem at the moment, we'd have to juggle the state
> between what SDL 1.2 is pretending for the app and what SDL2 actually
> used, and we're not talking about \_that\_ large a resource savings anyhow.
> 
> Reference Issue #143.

```diff
From d9708833642f1df2ee665f97054ebdab6b775f32 Mon Sep 17 00:00:00 2001
From: "Ryan C. Gordon" <[EMAIL REDACTED]>
Date: Fri, 19 Aug 2022 16:35:07 -0400
Subject: [PATCH] FIXME removal: We _could_ request a simpler GL context, but
 don't.

It's risky to find out we broke something this way when nothing is
actually having a problem at the moment, we'd have to juggle the state
between what SDL 1.2 is pretending for the app and what SDL2 actually
used, and we're not talking about _that_ large a resource savings anyhow.

Reference Issue #143.
---
 src/SDL12_compat.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/SDL12_compat.c b/src/SDL12_compat.c
index 9a90bf65..d0737206 100644
--- a/src/SDL12_compat.c
+++ b/src/SDL12_compat.c
@@ -5807,8 +5807,6 @@ SDL_SetVideoMode(int width, int height, int bpp, Uint32 flags12)
         SDL_assert(!VideoTexture20); /* either a new window or we destroyed all this */
         SDL_assert(!VideoRenderer20);
 
- FIXME("If we're using logical scaling, we can turn off depth buffer, alpha, etc, since we'll make our own and only need an RGB color buffer for the window.");
-
         if (!VideoGLContext20) {
             VideoGLContext20 = SDL20_GL_CreateContext(VideoWindow20);
             if (!VideoGLContext20) {

```