-
Notifications
You must be signed in to change notification settings - Fork 187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add extended dynamic state extension #421
Conversation
46e364c
to
747fa8b
Compare
2422304
to
c556563
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good :)
c556563
to
746e540
Compare
depth_test_enable: bool, | ||
) { | ||
self.extended_dynamic_state_fn | ||
.cmd_set_depth_test_enable_ext(command_buffer, depth_test_enable.into()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heh, I wasn't even aware we used .into()
for bool -> Bool32
conversion (struct builders use it) but I like it, as long as this results in vk::FALSE
and vk::TRUE
. Perhaps we should clean up if update { vk::TRUE } else { vk::FALSE }
in nv/ray_tracing.rs
some time.
@filnet Thanks! Merged! Not sure why the commit title reads |
The title is wrong : it is a copy/paste from a similar title. |
No description provided.