Prepare for multiple render backends #50
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
that we have an easier time writing another graphics library backend some time.
Shinmera
force-pushed
the
master
branch
7 times, most recently
from
df3056e to
470214e
Compare
Shinmera
force-pushed
the
master
branch
6 times, most recently
from
1be38da to
afb0476
Compare
Shinmera
force-pushed
the
master
branch
3 times, most recently
from
4f867f5 to
3fbd53d
Compare
Shinmera
force-pushed
the
master
branch
3 times, most recently
from
86f03fa to
391bc2f
Compare
Shinmera
force-pushed
the
master
branch
2 times, most recently
from
abcb908 to
b102af3
Compare
Shinmera
force-pushed
the
master
branch
2 times, most recently
from
4034ea8 to
bec5f8a
Compare
Shinmera
force-pushed
the
master
branch
3 times, most recently
from
c391794 to
0da5927
Compare
Shinmera
force-pushed
the
master
branch
4 times, most recently
from
9a6395b to
ec1092d
Compare
Shinmera
force-pushed
the
master
branch
3 times, most recently
from
fc79f56 to
cf6f722
Compare
Shinmera
force-pushed
the
master
branch
4 times, most recently
from
91a40bf to
c0fb2d2
Compare
Shinmera
force-pushed
the
master
branch
3 times, most recently
from
e6d8bfc to
a8d853f
Compare
Shinmera
force-pushed
the
master
branch
2 times, most recently
from
339197b to
1264753
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR prepares Trial for multiple render backends (Vulkan, Metal, DirectX, whatever). The work of this PR won't actually implement any other backends or ensure that they would work, but instead simply focuses on moving all of the references to the gl and %gl packages are moved out of core and into resources and a single file instead.
Anything else should call functions that interact with the context or a resource, instead. That should allow creating graphics library specific resource mixins and context mixins, paving the way for other backends.
The main goal is to create a cleaner separation and present a cleaner interface for users to depend upon. Ideally users would not ever call any GL specific functions on their own at all.
Raw GLSL will remain, with the intention being that future backends will have to implement a GLSL transpiler to whatever code format they require. This may need extra changes for more metadata depending on the needs of the target language, but for now we'll omit that problem.