Skip to content

Follow-up fix to 7360#7399

Merged
wookieejedi merged 1 commit intoscp-fs2open:masterfrom
wookieejedi:fix-cp-followup
Apr 23, 2026
Merged

Follow-up fix to 7360#7399
wookieejedi merged 1 commit intoscp-fs2open:masterfrom
wookieejedi:fix-cp-followup

Conversation

@wookieejedi
Copy link
Copy Markdown
Member

@wookieejedi wookieejedi commented Apr 23, 2026

Turns out #7360 causes cockpits to not render. That PR had updated GL_DEPTH_ATTACHMENT to GL_DEPTH_STENCIL_ATTACHMENT but missed two spots. This PR fixes that and tests show cockpits now work. (Root cause was the save/restore zbuffer swap using the old GL_DEPTH_ATTACHMENT point — once the scene texture became a combined depth-stencil, re-binding just the depth slot left stencil dangling and made the FBO incomplete.)

Turns out 7360 causes cockpits to not render. That PR had updated `GL_DEPTH_ATTACHMENT` to `GL_DEPTH_STENCIL_ATTACHMENT` but missed two spots. This PR fixes that and tests show cockpits now work.  (Root cause was the save/restore zbuffer swap using the old GL_DEPTH_ATTACHMENT point — once the scene texture became a combined depth-stencil, re-binding just the depth slot left stencil dangling and made the FBO incomplete.)
@wookieejedi wookieejedi added fix A fix for bugs, not-a-bugs, and/or regressions. graphics A feature or issue related to graphics (2d and 3d) labels Apr 23, 2026
@wookieejedi wookieejedi merged commit 46d136c into scp-fs2open:master Apr 23, 2026
20 checks passed
@wookieejedi wookieejedi deleted the fix-cp-followup branch April 23, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix A fix for bugs, not-a-bugs, and/or regressions. graphics A feature or issue related to graphics (2d and 3d)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants