Skip to content

Provide a Vulkan surface view base implementation#36603

Merged
akien-mga merged 1 commit into
godotengine:masterfrom
m4gr3d:implement_vk_surface_view
Mar 5, 2020
Merged

Provide a Vulkan surface view base implementation#36603
akien-mga merged 1 commit into
godotengine:masterfrom
m4gr3d:implement_vk_surface_view

Conversation

@m4gr3d

@m4gr3d m4gr3d commented Feb 27, 2020

Copy link
Copy Markdown
Contributor

Provides a base implementation of the Vulkan surface view (VkSurfaceView.kt) and its accompanying components.

The implementation is meant to be extended and updated in order to integrate it with the existing Godot java codebase.
The native layer is a stub at the moment, and is also meant to be extended and updated (based on @RandomShaper work) in order to provide the actual logic to setup and drive vulkan rendering.

The implementation is done in Kotlin, who's use in the engine (for Android) was recently green-lit!

@m4gr3d m4gr3d added this to the 4.0 milestone Feb 27, 2020
@m4gr3d m4gr3d changed the title [WIP] Provide a Vulkan surface view base implementation Provide a Vulkan surface view base implementation Feb 27, 2020
@m4gr3d m4gr3d mentioned this pull request Mar 4, 2020
5 tasks
@m4gr3d m4gr3d force-pushed the implement_vk_surface_view branch from 736cfc3 to 6fc5f5a Compare March 4, 2020 15:52
@akien-mga

Copy link
Copy Markdown
Member

Needs a rebase after #36798.

…iew.kt) and its accompanying components.

The implementation is meant to be extended and updated in order to integrate it with the existing Godot java and native codebase.
@m4gr3d m4gr3d force-pushed the implement_vk_surface_view branch from 6fc5f5a to c090caa Compare March 5, 2020 16:02
@m4gr3d

m4gr3d commented Mar 5, 2020

Copy link
Copy Markdown
Contributor Author

@akien-mga the PR was rebased.

@akien-mga akien-mga merged commit 60ea8ae into godotengine:master Mar 5, 2020
@akien-mga

Copy link
Copy Markdown
Member

Thanks!

@m4gr3d m4gr3d deleted the implement_vk_surface_view branch March 5, 2020 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants