Skip to content

Allows zeta specification when rendering to a non-framebuffer surface.#8

Merged
abaire merged 1 commit into
mainfrom
render_to_surface_improvements
May 7, 2026
Merged

Allows zeta specification when rendering to a non-framebuffer surface.#8
abaire merged 1 commit into
mainfrom
render_to_surface_improvements

Conversation

@abaire
Copy link
Copy Markdown
Owner

@abaire abaire commented May 7, 2026

No description provided.

@abaire abaire force-pushed the render_to_surface_improvements branch from b8cdd0d to c5afe61 Compare May 7, 2026 17:01
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request implements support for rendering to surfaces with custom zeta (depth/stencil) buffers, adding a new RenderToSurfaceStart overload and updating RenderToSurfaceEnd to restore the zeta state. Feedback focuses on ensuring the zeta format is correctly backed up in all RenderToSurfaceStart overloads to prevent incorrect state restoration. Additionally, a return statement is needed in the default case of GetSurfaceZetaPitch to prevent potential undefined behavior.

Comment thread src/nv2astate.cpp
Comment thread src/nv2astate.cpp
Comment thread src/nv2astate.h
@abaire abaire force-pushed the render_to_surface_improvements branch from c5afe61 to 5ea67af Compare May 7, 2026 17:21
@abaire abaire force-pushed the render_to_surface_improvements branch from 5ea67af to e82294e Compare May 7, 2026 17:22
@abaire abaire merged commit 4cb6430 into main May 7, 2026
2 checks passed
@abaire abaire deleted the render_to_surface_improvements branch May 7, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant