Last touch to fix main (#14)

* PascalCase functions (#6)

* More fix (#11)

* more clang format

* Asset loader

* Update kvf (track render command buffer recording state) (#5)

* PascalCase functions (#6)

* Add `graphics::Shader`, draw triangle with temp code (#7)

* Add `graphics::Shader`, draw triangle with temp code

* Fix warnings in Release builds

* Guard against invalid SPIR-V

* Update clang format

* remove stupid annoying spaces

* remove unnecessary cmake file ref

---------

Co-authored-by: Karn Kaul <[email protected]>

* Tymii/remove annoying macros (#13)

* more clang format

* Asset loader

* Update kvf (track render command buffer recording state) (#5)

* PascalCase functions (#6)

* Add `graphics::Shader`, draw triangle with temp code (#7)

* Add `graphics::Shader`, draw triangle with temp code

* Fix warnings in Release builds

* Guard against invalid SPIR-V

* Update clang format

* remove stupid annoying spaces

* Remove annoying macros

* remove unnecessary cmake file ref

* Asset loader (#4)

* more clang format

* Asset loader

* remove stupid annoying spaces

* remove unnecessary cmake file ref

---------

Co-authored-by: Karn Kaul <[email protected]>

---------

Co-authored-by: Karn Kaul <[email protected]>

Author: TymianekPL

lib/include/tkge/Assets/TextAsset.hpp

@@ -4,18 +4,11 @@
 
 namespace tkge::Assets
 {
-	class TextAsset final : public IAsset
+	class TextAsset final : public ICopyableAsset<TextAsset>
 	{
 	  public:
-		explicit TextAsset() = default;
-		void Load(ReadonlyByteStream byteStream) override
-		{
-			const auto vData = byteStream.AsSpan<char>(0, byteStream.GetStreamSize());
-			this->_text = std::string{ vData.begin(), vData.end() };
-		}
-
-		[[nodiscard]] const std::string& text() const noexcept { return this->_text; }
-	private:
-		std::string _text;
+		explicit TextAsset(std::string filename) : ICopyableAsset(std::move(filename)) {}
+		[[nodiscard]] std::string ReadAllText() const;
+		[[nodiscard]] std::string ReadAt(std::size_t position, std::size_t size) const;
 	};
 } // namespace tkge::Assets

lib/include/tkge/assetLoader.hpp

@@ -18,13 +18,7 @@ namespace tkge
 
 			for (const auto& path : paths)
 			{
-				if (std::filesystem::exists(path / fileName))
-				{
-					auto asset = std::make_unique<T>();
-					Assets::ReadonlyByteStream byteStream{(path / fileName).string()};
-					asset->Load(std::move(byteStream));
-					return asset;
-				}
+				if (std::filesystem::exists(path / fileName)) { return std::make_unique<T>((path / fileName).string()); }
 			}
 
 			throw std::runtime_error("Asset not found: " + fileName);

lib/src/Assets/TextAsset.cpp

@@ -1 +1,12 @@
 #include <tkge/Assets/TextAsset.hpp>
+std::string tkge::Assets::TextAsset::ReadAllText() const
+{
+	const auto vData = this->byteStream().AsSpan<char>(0, this->byteStream().GetStreamSize());
+	return std::string{vData.begin(), vData.end()};
+}
+
+std::string tkge::Assets::TextAsset::ReadAt(std::size_t position, std::size_t size) const
+{
+	const auto vData = this->byteStream().AsSpan<char>(position, size);
+	return std::string{vData.begin(), vData.end()};
+}