-
-
Notifications
You must be signed in to change notification settings - Fork 809
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
Implement surfaceSelector in Context3D.setRenderToTexture #15495
Conversation
@@ -612,15 +612,6 @@ pub fn set_render_to_texture<'gc>( | |||
); | |||
} | |||
|
|||
if surface_selector != 0 { |
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.
Do we want the same validation as FP has?
ArgumentError - for a mismatched surfaceSelector parameter. The value must be 0 for 2D textures and 0..5 for cube maps
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.
I hesitated to merge myself because I wasn't sure how strongly you wanted this to be addressed.
9ba1e7a
to
56de39e
Compare
56de39e
to
e2e9882
Compare
7c3b358
to
950f7e0
Compare
This still needs a test, but I'm planning to add that in a follow-up PR
950f7e0
to
4d32dfa
Compare
This still needs a test, but I'm planning to add that in a follow-up PR
Fixes #15487