Changeset b7fc3c2 in opengl-game for vulkan-game.hpp

Timestamp:

Jun 10, 2021, 2:58:54 PM (3 years ago)

Author:

Dmitry Portnoy <dportnoy@…>

Branches:

feature/imgui-sdl

Children:

c1ec4f6

Parents:

bb76950

git-author:

Dmitry Portnoy <dportnoy@…> (06/10/21 14:53:52)

git-committer:

Dmitry Portnoy <dportnoy@…> (06/10/21 14:58:54)

Message:

Modify the VulkanBuffer class to take a range and to align data based on that rather than the size of an individual data item. Also, reorganize the code in VulkanGae::updateScene() in a more logical fashion, and remove VulkanGame::updateObject() and inline its functionality.

File:

• vulkan-game.hpp (modified) (4 diffs)

vulkan-game.hpp

@@ -459 +459 @@
 
       template<class VertexType, class SSBOType>
-      void updateObject(SceneObject<VertexType, SSBOType>& obj);
-
-      template<class VertexType, class SSBOType>
       void updateObjectVertices(GraphicsPipeline_Vulkan<VertexType>& pipeline,
             SceneObject<VertexType, SSBOType>& obj, size_t index);
@@ -508 +505 @@
 }
 
-// TODO: Right now, it's basically necessary to pass the identity matrix in for ssbo.model
-// and to change the model matrix later by setting model_transform and then calling updateObject()
+// TODO: Right now, it's basically necessary to pass the identity matrix in for ssbo.model and to change
+// the model matrix later by setting model_transform and then calculating the new ssbo.model.
 // Figure out a better way to allow the model matrix to be set during object creation
 template<class VertexType, class SSBOType>
@@ -517 +514 @@
                                                          VulkanBuffer<SSBOType>& objectBuffer, SSBOType ssbo) {
    // TODO: Use the model field of ssbo to set the object's model_base
-   // currently, the passed in model is useless since it gets overridden in updateObject() anyway
+   // currently, the passed-in model is useless since it gets overridden when ssbo.model is recalculated
    size_t numVertices = pipeline.getNumVertices();
 
@@ -624 +621 @@
 }
 
-// TODO: Just pass in the single object instead of a list of all of them
-template<class VertexType, class SSBOType>
-void VulkanGame::updateObject(SceneObject<VertexType, SSBOType>& obj) {
-   obj.ssbo.model = obj.model_transform * obj.model_base;
-   obj.center = vec3(obj.ssbo.model * vec4(0.0f, 0.0f, 0.0f, 1.0f));
-
-   obj.modified = false;
-}
-
 template<class VertexType, class SSBOType>
 void VulkanGame::updateObjectVertices(GraphicsPipeline_Vulkan<VertexType>& pipeline,