[GL-D3] llwindowsdl: delete the GL context code #78

Closed
opened 2026-09-27 05:33:52 +00:00 by sky_line · 4 comments
Owner

Scope: the !gWindowUseVulkan branches, createSharedContext, the swapBuffers GL arm, SDL_GL_*; then delete the gWindowUseVulkan flag itself (every site becomes unconditional)

Prereq: D2 (or earlier for just the window file)

Acceptance: window create, resize, fullscreen and vsync all work

Risk: low

Shared acceptance baseline: links; logs in under VK and renders prims/mesh/rigged/terrain/water/sky/particles; click-select object/avatar/attachment; region crossing + teleport; no GLSL Compilation Error|SIGSEGV in the log.

Source: research/gl_removal_map.md

**Scope:** the `!gWindowUseVulkan` branches, `createSharedContext`, the `swapBuffers` GL arm, `SDL_GL_*`; then delete the `gWindowUseVulkan` flag itself (every site becomes unconditional) **Prereq:** D2 (or earlier for just the window file) **Acceptance:** window create, resize, fullscreen and vsync all work **Risk:** low **Shared acceptance baseline:** links; logs in under VK and renders prims/mesh/rigged/terrain/water/sky/particles; click-select object/avatar/attachment; region crossing + teleport; no `GLSL Compilation Error|SIGSEGV` in the log. Source: research/gl_removal_map.md
Author
Owner

Status check (verification sweep): partly done. e05771cdf6 (v1.0.0-beta.1) deleted the Win32/macOS/Mesa window backends and the GL context + shared-context API (createSharedContext: 0 refs). Remaining: gWindowUseVulkan is still there (11 refs in 4 files), and SDL_GL_* calls remain (4 lines in 2 files).

Status check (verification sweep): partly done. e05771cdf6 (v1.0.0-beta.1) deleted the Win32/macOS/Mesa window backends and the GL context + shared-context API (createSharedContext: 0 refs). Remaining: gWindowUseVulkan is still there (11 refs in 4 files), and SDL_GL_* calls remain (4 lines in 2 files).
sky_line added this to the Slipstream project 2026-10-06 15:04:15 +00:00
Author
Owner

Verified against forgejo/main: PARTIAL, leaving open. Done: llwindowsdl.cpp has no SDL_GL_*, createSharedContext, GL swapBuffers arm or gWindowUseVulkan branches; window create/resize/fullscreen/vsync verified on 1.2.0. Remaining: the flag itself survives as gWindowUseVulkanUI (llappviewer.cpp:389 definition, :7311 set to true; only comments read it at :4076,:6503) and should be deleted.

Verified against forgejo/main: PARTIAL, leaving open. Done: llwindowsdl.cpp has no SDL_GL_*, createSharedContext, GL swapBuffers arm or gWindowUseVulkan branches; window create/resize/fullscreen/vsync verified on 1.2.0. Remaining: the flag itself survives as gWindowUseVulkanUI (llappviewer.cpp:389 definition, :7311 set to true; only comments read it at :4076,:6503) and should be deleted.
Author
Owner

Pushed to main at 07c8a3ac73. GL context code removed from llwindowsdl. To test: Linux and Windows startup.

Pushed to main at 07c8a3ac73. GL context code removed from `llwindowsdl`. To test: Linux and Windows startup.
Author
Owner

Shipped in a published release and tested; closing (testing-lane sweep 2026-10-11).

Shipped in a published release and tested; closing (testing-lane sweep 2026-10-11).
sky_line 2026-10-11 13:24:49 +00:00
Sign in to join this conversation.
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
sky_line/Slipstream#78
No description provided.