Vulkan cleaning

Application/src/KarmaApp.cpp

@@ -17,7 +17,7 @@ class ExampleLayer : public Karma::Layer
 		std::shared_ptr<Karma::UniformBufferObject> shaderUniform;
 		shaderUniform.reset(Karma::UniformBufferObject::Create({ Karma::ShaderDataType::Mat4, Karma::ShaderDataType::Mat4 }, 0));
 
-		m_BlueSQShader.reset(Karma::Shader::Create("../Resources/Shaders/shader.vert", "../Resources/Shaders/shader.frag", shaderUniform, "CylinderShader"));
+		m_BlueSQShader.reset(Karma::Shader::Create("../Resources/Shaders/shader.vert", "../Resources/Shaders/shader.frag", "CylinderShader"));
 
 		m_SquareMat.reset(new Karma::Material());
 		m_SquareMat->AddShader(m_BlueSQShader);
@@ -31,7 +31,7 @@ class ExampleLayer : public Karma::Layer
 		m_Scene.reset(new Karma::Scene());
 
 		m_Scene->AddCamera(m_Camera);
-		m_Scene->AddVertexArray(m_SquareVA);
+		//m_Scene->AddVertexArray(m_SquareVA);
 
 		m_Scene->SetClearColor({ 0.0f, 0.0f, 0.0f, 1 });
 

CMakeLists.txt

@@ -34,10 +34,16 @@ set(MAC_PACKAGE FALSE)
 # This is so that .app can be installed in KarmaBin
 if (APPLE)
 	if(MAC_PACKAGE)
+                # We will be needing the install scheme so generate all the schemes
 		set(CMAKE_XCODE_GENERATE_SCHEME ON CACHE BOOL "")
 		set(CMAKE_INSTALL_PREFIX "${REPOSITORYROOT}/KarmaBin" CACHE PATH
 			"Custom install location for .app bundle" FORCE)
 	else()
+                # No need for generating all the schemes. Selected schemes can be
+                # be generated for appropriate targets. See Pranjal
+                #     set_target_properties(Pranjal PROPERTIES
+                #             XCODE_GENERATE_SCHEME TRUE
+                #             XCODE_SCHEME_WORKING_DIRECTORY "${REPOSITORYROOT}/KarmaBin")
 		set(CMAKE_XCODE_GENERATE_SCHEME OFF CACHE BOOL "")
 	endif()
 endif()
@@ -81,13 +87,13 @@ add_subdirectory(Karma)
 list(APPEND ESSENTIAL_LIBS Karma)
 
 # Build the Client
-add_subdirectory(Application)
+# add_subdirectory(Application)
 
 # Build the Editor
 add_subdirectory(Pranjal)
 
 # Relevant linking of finished Application with the Engine!
-target_link_libraries(SandBox PUBLIC KarmaEngine)
+# target_link_libraries(SandBox PUBLIC KarmaEngine)
 target_link_libraries(Pranjal PUBLIC KarmaEngine)
 
 if (APPLE AND MAC_PACKAGE)

GenerateProjects.bat

@@ -8,7 +8,7 @@ set /p Input=Enter y(Yes) or n(No):
 If /I "%Input%"=="y" goto yes
 goto no
 :yes
-KarmaBasedProjectName.sln
+start "" Game_Of_Lands.slnx
 exit
 :no
 echo Suit yourself! Goodbye!

Karma/src/Karma.h

@@ -3,17 +3,18 @@
 // To be included in clients
 
 // Game code specific includes
-#include "Karma/GameFramework/Actor.h"
-#include "Karma/GameFramework/World.h"
+#include "GameFramework/Actor.h"
+#include "GameFramework/World.h"
 
-#include "Karma/GameFramework/Level.h"
+#include "Engine/Level.h"
 
 #include "Karma/Application.h"
 #include "Karma/Layer.h"
 #include "Karma/Log.h"
 #include "Karma/KarmaGui/KarmaGuiLayer.h"
 #include "Karma/KarmaGui/KarmaGui.h"
 #include "Karma/KarmaGui/KarmaGuiInternal.h"
+#include "Karma/KarmaGui/KarmaGuizmo.h"
 #include "Karma/KarmaGui/KarmaGuiRenderer.h"
 
 #include "Karma/Renderer/Shader.h"

Karma/src/Karma/Animation/Animation.h

@@ -1,7 +1,5 @@
 #pragma once
 
-#include "krpch.h"
-
 namespace Karma
 {
 	class KARMA_API Animation
@@ -12,4 +10,4 @@ namespace Karma
 
 	};
 
-}
+}

Karma/src/Karma/Application.cpp

@@ -1,11 +1,11 @@
 #include "Application.h"
-#include "Karma/Log.h"
 #include "Karma/Input.h"
 #include "Karma/Renderer/Renderer.h"
 #include "chrono"
 #include "Engine/Engine.h"
 #include "Core/UObjectGlobals.h"// to be bundled appropriately in core.h
 #include "Core/Package.h"
+#include "KarmaRHI/KarmaRHI.h"
 
 namespace Karma
 {
@@ -31,6 +31,9 @@ namespace Karma
 		m_Window = Window::Create();
 		m_Window->SetEventCallback(KR_BIND_EVENT_FN(Application::OnEvent)); // Setting the listener
 
+		// Initialize RHI (Render Hardware Interface)
+		RHIInit();
+
 		m_LayerStack = new LayerStack();
 
 		// Graphics API Vulkan or OpenGL should have been completely initialized by here
@@ -48,6 +51,10 @@ namespace Karma
 		// and its context.
 		KR_CORE_INFO("Deleting stacks");
 		delete m_LayerStack;
+
+		// Deinitialize RHI
+		RHIExit();
+
 		KR_CORE_INFO("Deleting window");
 		delete m_Window;
 		s_Instance = nullptr;
@@ -102,7 +109,14 @@ namespace Karma
 			deltaTime /= 1000000.0f;
 
 			// Tick KEngine
-			GEngine->Tick(deltaTime, false);
+			if(GEngine)
+			{
+				GEngine->Tick(deltaTime, false);
+			}
+
+			// Ponder over the sequence of this updation
+			// this includes glfwPollEvents()
+			m_Window->OnUpdate();
 
 			for (auto layer : *m_LayerStack)
 			{
@@ -118,8 +132,6 @@ namespace Karma
 			}
 
 			m_KarmaGuiLayer->End();
-
-			m_Window->OnUpdate();
 		}
 	}
 

Karma/src/Karma/Application.h

@@ -9,8 +9,6 @@
  */
 #pragma once
 
-#include "krpch.h"
-
 #include "Karma/Window.h"
 #include "Karma/Events/ApplicationEvent.h"
 #include "Karma/Events/ControllerDeviceEvent.h"

Karma/src/Karma/Core.h

@@ -109,6 +109,14 @@
 	#define KR_CORE_ASSERT(expr, ...)
 #endif
 
+ /**
+  * @brief Denotes code paths that should never be reached.
+  * 
+  * @note Taken from the UE
+  */
+#define checkNoEntry()       KR_CORE_ASSERT(false, "Enclosing block should never be called")
+
+
 /**
  * @brief Macro for std::bind routine. See https://en.cppreference.com/w/cpp/utility/functional/bind
  *

Karma/src/Karma/Core/Field.h

@@ -10,7 +10,6 @@
 
 #pragma once
 
-#include "krpch.h"
 #include "UObjectGlobals.h"
 
 namespace Karma

Karma/src/Karma/Core/GFrameworkMacros.h

@@ -10,8 +10,6 @@
 
 #pragma once
 
-#include "krpch.h"
-
 enum EInternal						{EC_InternalUseOnlyConstructor};
 typedef void		(*ClassConstructorType)				(const Karma::FObjectInitializer&);
 
@@ -37,12 +35,19 @@ typedef void		(*ClassConstructorType)				(const Karma::FObjectInitializer&);
  * calls default constructor with placement new during UObjectAllocation
  * (*InClass->m_ClassConstructor)(FObjectInitializer)
  * 
+ * Also defines the routine to destroy (call class destructor) accessible from
+ * UObjectBase. See KarmaSmriti::ShutDown() for the application.
+ * 
  * @see StaticConstructObject_Internal() in UObjectGlobals.cpp
  * @remark In UE, this is done in ObjectMacros.h, #define DECLARE_CLASS
  * 
  */
 #define DECLARE_KARMA_CLASS(TClass, TSuperClass) \
 public: \
+	 inline virtual void ShivaUObject() override\
+	 {\
+		this->~TClass();\
+	 } \
 	DEFINE_DEFAULT_CONSTRUCTOR_CALL(TClass) \
 	/** Typedef for the base class ({{ typedef-type }}) */ \
 	typedef TSuperClass Super;\