sky_line/Slipstream

Fork 1

[GL-C2] Replace `LLGLSLShader` with a UI facade #71

New issue

Closed

opened 2026-09-27 05:33:52 +00:00 by sky_line · 2 comments

sky_line commented

2026-09-27 05:33:52 +00:00

Owner

Scope: 17 gUIProgram.bind() call sites become an LLUIProgram token or plain gGL.flush(). Nothing live reads sCurBoundShaderPtr. Delete LLGLSLShader.

Prereq: B12

Acceptance: UI identical; C1 crash class gone

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:** 17 `gUIProgram.bind()` call sites become an `LLUIProgram` token or plain `gGL.flush()`. Nothing live reads `sCurBoundShaderPtr`. Delete `LLGLSLShader`. **Prereq:** B12 **Acceptance:** UI identical; C1 crash class gone **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

sky_line added this to the GL removal C: replace still-needed GL types milestone

2026-09-27 05:33:52 +00:00

sky_line added the

gl-removal

label

2026-09-27 05:33:52 +00:00

sky_line commented

2026-10-04 02:45:47 +00:00

Author

Owner

Status check (verification sweep): not done. LLGLSLShader is still referenced (82 lines across 14 files; llglslshader.cpp is a 65-line stub after 9bad552128), and LLUIProgram doesn't exist yet.

Status check (verification sweep): not done. LLGLSLShader is still referenced (82 lines across 14 files; llglslshader.cpp is a 65-line stub after 9bad552128), and LLUIProgram doesn't exist yet.

sky_line added this to the Slipstream project

2026-10-06 15:04:15 +00:00

sky_line referenced this issue from a commit

2026-10-10 03:31:53 +00:00

GL-B12/GL-C2: delete LLViewerShaderMgr, LLShaderMgr and LLGLSLShader

sky_line added the

status

up-next

label

2026-10-10 05:06:01 +00:00

sky_line commented

2026-10-10 06:24:38 +00:00

Author

Owner

Verified against forgejo/main (v1.2.0, 55f4242518); GL deletion in ef8365e45f (gl-delete merge), 52c3662a76, 547061ef34, 5f9502cf69. Verified absent: LLGLSLShader (only a comment at llappviewer.cpp:4172), gUIProgram, sCurBoundShaderPtr, LLShaderMgr/LLViewerShaderMgr. No llglslshader.* files remain; UI renders on 1.2.0.

Verified against forgejo/main (v1.2.0, 55f4242518); GL deletion in ef8365e45f (gl-delete merge), 52c3662a76, 547061ef34, 5f9502cf69. Verified absent: LLGLSLShader (only a comment at llappviewer.cpp:4172), gUIProgram, sCurBoundShaderPtr, LLShaderMgr/LLViewerShaderMgr. No llglslshader.* files remain; UI renders on 1.2.0.

sky_line

2026-10-10 06:24:38 +00:00

closed this issue
removed the
status
up-next
label

No milestone

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#71

No description provided.

Rows
Columns

[GL-C2] Replace LLGLSLShader with a UI facade #71

[GL-C2] Replace `LLGLSLShader` with a UI facade #71