[GL-C4] LLGLEnable/LLGLDepthTest/LLGLS* → LLRender state scopes #73

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

Scope: Introduce gGL.pushState-style scopes that take LLRender::CF_*/BT_*. Mechanically convert the 367 newview, 6 llui and 5 llappearance uses (VK-live UI and overlay first; dead files are gone by then).

Prereq: B7–B11

Acceptance: UI, selection overlays, property lines and ban lines render with correct depth

Risk: med

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:** Introduce `gGL.pushState`-style scopes that take `LLRender::CF_*`/`BT_*`. Mechanically convert the 367 newview, 6 llui and 5 llappearance uses (VK-live UI and overlay first; dead files are gone by then). **Prereq:** B7–B11 **Acceptance:** UI, selection overlays, property lines and ban lines render with correct depth **Risk:** med **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): not started. There are still 22 LLGLEnable uses (6 files), 91 LLGLDepthTest uses (31 files) and 136 LLGLS* uses (43 files).

Status check (verification sweep): not started. There are still 22 LLGLEnable uses (6 files), 91 LLGLDepthTest uses (31 files) and 136 LLGLS* uses (43 files).
sky_line added this to the Slipstream project 2026-10-06 15:04:15 +00:00
Author
Owner

Verified against forgejo/main: NOT DONE, leaving open. LLGLState-style scopes still exist: ~190 uses of LLGLEnable/LLGLDisable/LLGLDepthTest/LLGLState/LLGLS* remain, with llrender/llgl.h + llglstates.h + llgl.cpp still defining them. Biggest users: fsmaniprotatejoint.cpp (26), llglsandbox.cpp (13), llmaniptranslate.cpp (10), llmanipscale.cpp (8), llmaniprotate.cpp (7), lltoolcomp.cpp (5), lltextureview.cpp (5), fsmaniptranslatejoint.cpp (5), llappearance/lltexlayer.cpp:149,368,369 (5), llmodelpreview.cpp (4), plus ~35 more files with 1-3 each (huds, tracker, world map, parcel overlay, avatar, etc.). No gGL.pushState-style CF_/BT_ scope has replaced them.

Verified against forgejo/main: NOT DONE, leaving open. LLGLState-style scopes still exist: ~190 uses of LLGLEnable/LLGLDisable/LLGLDepthTest/LLGLState/LLGLS* remain, with llrender/llgl.h + llglstates.h + llgl.cpp still defining them. Biggest users: fsmaniprotatejoint.cpp (26), llglsandbox.cpp (13), llmaniptranslate.cpp (10), llmanipscale.cpp (8), llmaniprotate.cpp (7), lltoolcomp.cpp (5), lltextureview.cpp (5), fsmaniptranslatejoint.cpp (5), llappearance/lltexlayer.cpp:149,368,369 (5), llmodelpreview.cpp (4), plus ~35 more files with 1-3 each (huds, tracker, world map, parcel overlay, avatar, etc.). No gGL.pushState-style CF_*/BT_* scope has replaced them.
Author
Owner

Pushed to main at 07c8a3ac73. LLGL* state guards replaced by LLRender state scopes; llglstates.h deleted.

Pushed to main at 07c8a3ac73. `LLGL*` state guards replaced by `LLRender` state scopes; `llglstates.h` deleted.
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:48 +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#73
No description provided.