Introduce explicit Viewport sharing#96
Conversation
grovesNL
left a comment
There was a problem hiding this comment.
This looks great, thank you! Sharing Viewport explicitly seems like a good approach.
I wish we could derive this internally somehow and share params on a shared type like Pipeline. I'm not sure how we could do that without requiring something like a viewport index to be specified during screen resolution updates though.
|
Rebased on top of
Right... It doesn't seem easy to figure out which params to invalidate internally. At least, we are mostly just trading an argument here ( |
This PR introduces a
Viewporttype that explicitly holds theParamsof aPipeline.This allows sharing a
Viewportwith multipleTextRendererinstances, reducing pipeline context changes.Depends on #95.