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
5 times, most recently
from
1b6faad
to
c8b7e25
Compare
Shinmera
force-pushed
the
master
branch
3 times, most recently
from
4c5b494
to
f1e6e7f
Compare
Shinmera
force-pushed
the
master
branch
2 times, most recently
from
5d1a945
to
b39c881
Compare
Shinmera
force-pushed
the
master
branch
4 times, most recently
from
5dbe758
to
79661d6
Compare
Shinmera
force-pushed
the
master
branch
8 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
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.