[#211]: Update Vulkan version on CI

[JacobDomagala]

Fixes #211

[github-actions]

✅ WINDOWS COMPILE RESULT - SUCCESS! ✅

[github-actions]

✅ UBUNTU COMPILE RESULT - SUCCESS! ✅

[github-actions]

⚡ Static analysis result ⚡

🔴 cppcheck found 2 issues! Click here to see details.

Looper/engine/game/game_object.hpp

Lines 110 to 115 in 022aad6

	GetOccupiedNodes() const;
	std::string editorGroup_ = "Default";
	protected:
	// should be overriden by derrived class

!Line: 110 - performance: Function 'GetOccupiedNodes()' should return member 'nodes_' by const reference. [returnByReference]

Looper/engine/renderer/shader.cpp

Lines 220 to 225 in 022aad6

	auto& renderData = Data::renderData_.at(GetCurrentlyBoundType());
	std::array< VkDescriptorPoolSize, 1 > poolSizes{};
	poolSizes[0].type = VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER;
	poolSizes[0].descriptorCount = static_cast< uint32_t >(renderData.swapChainImages.size());
	VkDescriptorPoolCreateInfo poolInfo = {};

!Line: 220 - style: Variable 'renderData' can be declared as reference to const [constVariableReference]

---

🔴 clang-tidy found 75 issues! Click here to see details.

Looper/engine/renderer/shader.cpp

Lines 53 to 58 in 022aad6

	return {{device, vertShaderStageInfo}, {device, fragShaderStageInfo}};
	}
	///////////////////////////////////////////////////////////////////
	/////////////////////// QUAD SHADER ///////////////////////
	///////////////////////////////////////////////////////////////////

!Line: 53 - error: use designated initializer list to initialize 'ShaderInfoWrapper' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 11 - note: aggregate type is defined here

Looper/engine/renderer/shader.cpp

Lines 157 to 162 in 022aad6

	std::transform(descriptorImageInfos.begin(), descriptorImageInfos.end(), viewAndSamplers.begin(),
	descriptorImageInfos.begin(),
	[](auto& descriptoInfo, const auto& viewAndSampler) {
	descriptoInfo.imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
	descriptoInfo.imageView = viewAndSampler.first;
	descriptoInfo.sampler = viewAndSampler.second;

!Line: 157 - error: use a boost version of this algorithm [boost-use-ranges,-warnings-as-errors]

Looper/engine/renderer/shader.cpp

Lines 157 to 162 in 022aad6

	std::transform(descriptorImageInfos.begin(), descriptorImageInfos.end(), viewAndSamplers.begin(),
	descriptorImageInfos.begin(),
	[](auto& descriptoInfo, const auto& viewAndSampler) {
	descriptoInfo.imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
	descriptoInfo.imageView = viewAndSampler.first;
	descriptoInfo.sampler = viewAndSampler.second;

!Line: 157 - error: use a ranges version of this algorithm [modernize-use-ranges,-warnings-as-errors]

Looper/engine/game/animatable.cpp

Lines 234 to 239 in 022aad6

	std::find_if(animationPoints_.begin(), animationPoints_.end(),
	[&updatedAnimationPoint](const auto animationPoint) {
	return animationPoint.GetID() == updatedAnimationPoint.GetID();
	});
	if (updatedPointIt != animationPoints_.end())

!Line: 234 - error: use a boost version of this algorithm [boost-use-ranges,-warnings-as-errors]

Looper/engine/game/animatable.cpp

Lines 234 to 239 in 022aad6

	std::find_if(animationPoints_.begin(), animationPoints_.end(),
	[&updatedAnimationPoint](const auto animationPoint) {
	return animationPoint.GetID() == updatedAnimationPoint.GetID();
	});
	if (updatedPointIt != animationPoints_.end())

!Line: 234 - error: use a ranges version of this algorithm [modernize-use-ranges,-warnings-as-errors]

Looper/engine/game/animatable.cpp

Lines 235 to 240 in 022aad6

	[&updatedAnimationPoint](const auto animationPoint) {
	return animationPoint.GetID() == updatedAnimationPoint.GetID();
	});
	if (updatedPointIt != animationPoints_.end())
	{

!Line: 235 - error: the const qualified parameter 'animationPoint' is copied for each invocation; consider making it a reference [performance-unnecessary-value-param,-warnings-as-errors]

Looper/engine/game/animatable.cpp

Lines 252 to 257 in 022aad6

	std::find_if(animationPoints_.begin(), animationPoints_.end(),
	[animationID](const auto& point) { return point.GetID() == animationID; });
	/*auto it = std::find(animationPoints_.begin(), animationPoints_.end(), pathNodeMapPosition);*/
	if (it != animationPoints_.end())
	{

!Line: 252 - error: use a boost version of this algorithm [boost-use-ranges,-warnings-as-errors]

Looper/engine/game/animatable.cpp

Lines 252 to 257 in 022aad6

	std::find_if(animationPoints_.begin(), animationPoints_.end(),
	[animationID](const auto& point) { return point.GetID() == animationID; });
	/*auto it = std::find(animationPoints_.begin(), animationPoints_.end(), pathNodeMapPosition);*/
	if (it != animationPoints_.end())
	{

!Line: 252 - error: use a ranges version of this algorithm [modernize-use-ranges,-warnings-as-errors]

Looper/engine/renderer/buffer.cpp

Lines 13 to 18 in 022aad6

	IsMapped(VmaAllocation alloc)
	{
	VmaAllocationInfo allocInfo;
	vmaGetAllocationInfo(Data::vk_hAllocator, alloc, &allocInfo);
	return allocInfo.pMappedData != nullptr;

!Line: 13 - error: function 'IsMapped' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/buffer.cpp

Lines 22 to 27 in 022aad6

	FindMemoryType(uint32_t typeFilter, VkMemoryPropertyFlags properties)
	{
	VkPhysicalDeviceMemoryProperties memProperties = {};
	vkGetPhysicalDeviceMemoryProperties(Data::vk_physicalDevice, &memProperties);
	for (uint32_t i = 0; i < memProperties.memoryTypeCount; i++)

!Line: 22 - error: function 'FindMemoryType' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/buffer.cpp

Lines 119 to 124 in 022aad6

	AllocateMemory(VkMemoryRequirements memReq, VkDeviceMemory& bufferMemory,
	VkMemoryPropertyFlags properties)
	{
	VkMemoryAllocateInfo allocInfo = {};
	allocInfo.sType = VK_STRUCTURE_TYPE_MEMORY_ALLOCATE_INFO;
	allocInfo.allocationSize = memReq.size;

!Line: 119 - error: function 'AllocateMemory' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/game/path_finder.cpp

Lines 50 to 55 in 022aad6

	connectedTo.push_back(x + (y - 1) * w);
	}
	if (y < h - 1)
	{
	connectedTo.push_back(x + (y + 1) * w);

!Line: 50 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/game/path_finder.cpp

Lines 55 to 60 in 022aad6

	connectedTo.push_back(x + (y + 1) * w);
	}
	if (x > 0)
	{
	connectedTo.push_back((x - 1) + y * w);

!Line: 55 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/game/path_finder.cpp

Lines 60 to 65 in 022aad6

	connectedTo.push_back((x - 1) + y * w);
	}
	if (x < w - 1)
	{
	connectedTo.push_back((x + 1) + y * w);

!Line: 60 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/game/path_finder.cpp

Lines 65 to 70 in 022aad6

	connectedTo.push_back((x + 1) + y * w);
	}
	nodes_.emplace_back(glm::ivec2{x, y}, glm::vec2(x * grad, y * grad) + offset, x + y * w,
	connectedTo);
	}

!Line: 65 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/game/path_finder.cpp

Lines 68 to 73 in 022aad6

	nodes_.emplace_back(glm::ivec2{x, y}, glm::vec2(x * grad, y * grad) + offset, x + y * w,
	connectedTo);
	}
	}
	initialized_ = true;

!Line: 68 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/game/path_finder.cpp

Lines 125 to 130 in 022aad6

	+ static_cast< uint32_t >(tile.second) * levelHandle_->GetTileSize());
	}
	std::vector< NodeID >
	PathFinder::GetPath(const glm::vec2& source, const glm::vec2& destination)
	{

!Line: 125 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/editor/gui/editor_gui.cpp

Lines 331 to 336 in 022aad6

	objectsInfo_[object.GetID()] = {
	fmt::format("[{}] {} ({:.2f}, {:.2f})", object.GetTypeString().c_str(),
	object.GetName().c_str(), object.GetPosition().x, object.GetPosition().y),
	false};
	groups_[object.editorGroup_].emplace_back(object.GetID());

!Line: 331 - error: use designated initializer list to initialize 'ObjectInfo' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 149 - note: aggregate type is defined here

Looper/editor/gui/editor_gui.cpp

Lines 342 to 347 in 022aad6

	objectsInfo_[enemy.GetID()] = {
	fmt::format("[{}] {} ({:.2f}, {:.2f})", enemy.GetTypeString().c_str(),
	enemy.GetName().c_str(), enemy.GetPosition().x, enemy.GetPosition().y),
	false};
	groups_[enemy.editorGroup_].emplace_back(enemy.GetID());

!Line: 342 - error: use designated initializer list to initialize 'ObjectInfo' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 149 - note: aggregate type is defined here

Looper/editor/gui/editor_gui.cpp

Lines 353 to 358 in 022aad6

	objectsInfo_[player.GetID()] = {
	fmt::format("[{}] {} ({:.2f}, {:.2f})", player.GetTypeString().c_str(),
	player.GetName().c_str(), player.GetPosition().x, player.GetPosition().y),
	false};
	groups_[player.editorGroup_].emplace_back(player.GetID());

!Line: 353 - error: use designated initializer list to initialize 'ObjectInfo' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 149 - note: aggregate type is defined here

Looper/editor/gui/editor_gui.cpp

Lines 395 to 400 in 022aad6

	selectedObjects_.emplace_back(SelectedObjectInfo{ID, gameObject.GetHasCollision(),
	gameObject.GetSprite().GetRenderInfo().layer,
	gameObject.editorGroup_});
	RecalculateCommonProperties();

!Line: 395 - error: use designated initializer list to initialize 'SelectedObjectInfo' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 155 - note: aggregate type is defined here

Looper/editor/gui/editor_gui.cpp

Lines 468 to 473 in 022aad6

	objectsInfo_[ID] = {fmt::format("[{}] {} ({:.2f}, {:.2f})", object.GetTypeString().c_str(),
	object.GetName().c_str(), object.GetPosition().x,
	object.GetPosition().y),
	true};
	groups_["Default"].push_back(ID);

!Line: 468 - error: use designated initializer list to initialize 'ObjectInfo' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 149 - note: aggregate type is defined here
!Line: 83 - note: in instantiation of function template specialization 'fmt::format<tm>' requested here

Looper/engine/utils/file_manager.cpp

Lines 26 to 31 in 022aad6

	std::ifstream fileHandle(fileName.data(), std::ios::binary);
	utils::Assert(fileHandle.is_open(),
	fmt::format("FileManager::ReadBinaryFile -> {} can't be opened!", fileName));
	const auto size = std::filesystem::file_size(fileName);

!Line: 26 - error: result of a `data()` call may not be null terminated, provide size information to the callee to prevent potential issues [bugprone-suspicious-stringview-data-usage,-warnings-as-errors]

Looper/engine/utils/file_manager.cpp

Lines 82 to 87 in 022aad6

	return {std::move(textureData), {w, h}, n};
	}
	nlohmann::json
	FileManager::LoadJsonFile(std::string_view pathToFile)
	{

!Line: 82 - error: use designated initializer list to initialize 'ImageData' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 28 - note: aggregate type is defined here

Looper/engine/game/level.cpp

Lines 542 to 547 in 022aad6

	const auto occupied = nodes.at(w + width * h).occupied_;
	const auto index =
	(w + width * offset) * numChannels; // Calculate the index for the start of this pixel
	data[index + 0] = 255; // R
	data[index + 1] = !occupied * 255; // G

!Line: 542 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/game/level.cpp

Lines 553 to 558 in 022aad6

	collisionTextureData_ = {std::move(data), {width, height}, numChannels};
	// To avoid blurry edges
	renderer::TextureProperties props;
	props.magFilter = VK_FILTER_NEAREST;
	props.minFilter = VK_FILTER_NEAREST;

!Line: 553 - error: use designated initializer list to initialize 'ImageData' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 28 - note: aggregate type is defined here

Looper/engine/renderer/sprite.cpp

Lines 80 to 85 in 022aad6

	{glm::vec3{-0.5f, 0.5f, LAYERS.at(renderLayer)}, glm::vec3{0.0f, 0.0f, 1.0f}},
	{glm::vec3{0.5f, 0.5f, LAYERS.at(renderLayer)}, glm::vec3{1.0f, 0.0f, 1.0f}},
	{glm::vec3(0.5f, -0.5f, LAYERS.at(renderLayer)), glm::vec3{1.0f, 1.0f, 1.0f}},
	{glm::vec3{-0.5f, -0.5f, LAYERS.at(renderLayer)}, glm::vec3{0.0f, 1.0f, 1.0f}}};
	const auto transformMat = ComputeModelMat();

!Line: 80 - error: use designated initializer list to initialize 'Vertex' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 20 - note: aggregate type is defined here

Looper/engine/renderer/sprite.cpp

Lines 81 to 86 in 022aad6

	{glm::vec3{0.5f, 0.5f, LAYERS.at(renderLayer)}, glm::vec3{1.0f, 0.0f, 1.0f}},
	{glm::vec3(0.5f, -0.5f, LAYERS.at(renderLayer)), glm::vec3{1.0f, 1.0f, 1.0f}},
	{glm::vec3{-0.5f, -0.5f, LAYERS.at(renderLayer)}, glm::vec3{0.0f, 1.0f, 1.0f}}};
	const auto transformMat = ComputeModelMat();
	ComputeBoundingBox();

!Line: 81 - error: use designated initializer list to initialize 'Vertex' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 20 - note: aggregate type is defined here

Looper/engine/renderer/sprite.cpp

Lines 82 to 87 in 022aad6

	{glm::vec3(0.5f, -0.5f, LAYERS.at(renderLayer)), glm::vec3{1.0f, 1.0f, 1.0f}},
	{glm::vec3{-0.5f, -0.5f, LAYERS.at(renderLayer)}, glm::vec3{0.0f, 1.0f, 1.0f}}};
	const auto transformMat = ComputeModelMat();
	ComputeBoundingBox();

!Line: 82 - error: use designated initializer list to initialize 'Vertex' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 20 - note: aggregate type is defined here

Looper/engine/renderer/sprite.cpp

Lines 83 to 88 in 022aad6

	{glm::vec3{-0.5f, -0.5f, LAYERS.at(renderLayer)}, glm::vec3{0.0f, 1.0f, 1.0f}}};
	const auto transformMat = ComputeModelMat();
	ComputeBoundingBox();
	textures_ = {TextureLibrary::GetTexture(fileName)->GetID(),

!Line: 83 - error: use designated initializer list to initialize 'Vertex' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 20 - note: aggregate type is defined here

Looper/editor/gui/editor_gui_object.cpp

Lines 25 to 30 in 022aad6

	std::unordered_map< renderer::TextureID, VkDescriptorSet > textureDescriptors = {};
	VkDescriptorSet
	GetDescriptor(renderer::TextureID id, VkDescriptorPool descriptorPool,
	VkDescriptorSetLayout descriptorSetLayout)
	{

!Line: 25 - error: variable 'textureDescriptors' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/editor/gui/editor_gui_object.cpp

Lines 28 to 33 in 022aad6

	GetDescriptor(renderer::TextureID id, VkDescriptorPool descriptorPool,
	VkDescriptorSetLayout descriptorSetLayout)
	{
	const auto desc = textureDescriptors.find(id);
	VkDescriptorSet descriptor{};

!Line: 28 - error: function 'GetDescriptor' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/editor/gui/editor_gui_object.cpp

Lines 384 to 389 in 022aad6

	ImGui::SetCursorScreenPos(ImVec2(currentPos.x + sectionSize.x / 4.0f, currentPos.y));
	ImGui::Image(
	static_cast< ImTextureID >(GetDescriptor(gameObject.GetSprite().GetTexture()->GetID(),
	renderer::EditorData::descriptorPool_,
	renderer::EditorData::descriptorSetLayout_)),
	{glm::min(sectionSize.x, 128.0f), glm::min(sectionSize.x, 128.0f)});

!Line: 384 - error: '/' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/editor/gizmo.cpp

Lines 111 to 116 in 022aad6

	+ glm::vec2{0.0f, gizmoCenter_.GetSize().y / 2.0f + gizmoUp_.GetSize().x / 1.85f});
	gizmoSide_.SetInitialPosition(
	gizmoCenter_.GetPosition()
	+ glm::vec2{

!Line: 111 - error: '/' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/editor/gizmo.cpp

Lines 117 to 122 in 022aad6

	glm::vec2(gizmoCenter_.GetSize().x / 2.0f + gizmoSide_.GetSize().x / 1.85f, 0.0f)});
	}
	void
	Gizmo::CheckHovered(const glm::vec3& cameraPos, const glm::vec2& globalPosition)
	{

!Line: 117 - error: '/' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/input/input_manager.cpp

Lines 14 to 19 in 022aad6

	BroadcastEvent(const auto& listeners, auto& event)
	{
	for (auto* listener : listeners)
	{
	std::invoke(func, listener, event);

!Line: 14 - error: function 'BroadcastEvent' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 118 to 123 in 022aad6

	return {image, allocationInfo.deviceMemory, allocation};
	}
	VkImageView
	Texture::CreateImageView(VkImage image, VkFormat format, VkImageAspectFlags aspectFlags,
	uint32_t mipLevels, bool /*cubemap*/)

!Line: 118 - error: use designated initializer list to initialize 'VulkanImage' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 25 - note: aggregate type is defined here

Looper/engine/renderer/texture.cpp

Lines 216 to 221 in 022aad6

	blit.srcOffsets[0] = {0, 0, 0};
	blit.srcOffsets[1] = {mipWidth, mipHeight, 1};
	blit.srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
	blit.srcSubresource.mipLevel = i - 1;
	blit.srcSubresource.baseArrayLayer = 0;
	blit.srcSubresource.layerCount = 1;

!Line: 216 - error: use designated initializer list to initialize 'VkOffset3D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 217 to 222 in 022aad6

	blit.srcOffsets[1] = {mipWidth, mipHeight, 1};
	blit.srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
	blit.srcSubresource.mipLevel = i - 1;
	blit.srcSubresource.baseArrayLayer = 0;
	blit.srcSubresource.layerCount = 1;
	blit.dstOffsets[0] = {0, 0, 0};

!Line: 217 - error: use designated initializer list to initialize 'VkOffset3D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 222 to 227 in 022aad6

	blit.dstOffsets[0] = {0, 0, 0};
	blit.dstOffsets[1] = {mipWidth > 1 ? mipWidth / 2 : 1, mipHeight > 1 ? mipHeight / 2 : 1, 1};
	blit.dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
	blit.dstSubresource.mipLevel = i;
	blit.dstSubresource.baseArrayLayer = 0;
	blit.dstSubresource.layerCount = 1;

!Line: 222 - error: use designated initializer list to initialize 'VkOffset3D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 223 to 228 in 022aad6

	blit.dstOffsets[1] = {mipWidth > 1 ? mipWidth / 2 : 1, mipHeight > 1 ? mipHeight / 2 : 1, 1};
	blit.dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
	blit.dstSubresource.mipLevel = i;
	blit.dstSubresource.baseArrayLayer = 0;
	blit.dstSubresource.layerCount = 1;

!Line: 223 - error: use designated initializer list to initialize 'VkOffset3D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 351 to 356 in 022aad6

	region.imageOffset = {0, 0, 0};
	region.imageExtent = {texWidth, texHeight, 1};
	const auto size =
	static_cast< size_t >(texWidth) * static_cast< size_t >(texHeight) * static_cast< size_t >(4);
	Buffer::CopyDataToImageWithStaging(image, data, size, {region});

!Line: 351 - error: use designated initializer list to initialize 'VkOffset3D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 352 to 357 in 022aad6

	region.imageExtent = {texWidth, texHeight, 1};
	const auto size =
	static_cast< size_t >(texWidth) * static_cast< size_t >(texHeight) * static_cast< size_t >(4);
	Buffer::CopyDataToImageWithStaging(image, data, size, {region});
	}

!Line: 352 - error: use designated initializer list to initialize 'VkExtent3D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 451 to 456 in 022aad6

	if (s_loadedTextures.find(textureName) == s_loadedTextures.end())
	{
	SCOPED_TIMER(fmt::format("Texture: {} not found in library. Loading it", textureName));
	LoadTexture(type, textureName);
	}

!Line: 451 - error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 488 to 493 in 022aad6

	if (s_loadedTextures.find(textureName) == s_loadedTextures.end())
	{
	SCOPED_TIMER(fmt::format("Creating texture {}", textureName));
	LoadTexture(type, textureName, props);
	}
	else

!Line: 488 - error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]

Looper/engine/renderer/texture.cpp

Lines 505 to 510 in 022aad6

	if (s_loadedTextures.find(textureName) == s_loadedTextures.end())
	{
	SCOPED_TIMER(fmt::format("Creating texture {}", textureName));
	LoadTexture(type, textureName, data, props);
	}
	else

!Line: 505 - error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 50 to 55 in 022aad6

	CreatePipeline(std::string_view vertexShader, std::string_view fragmentShader, PrimitiveType type)
	{
	auto& renderData = Data::renderData_.at(GetCurrentlyBoundType());
	// Create switch in future if we support more types
	auto& pipeLineLayout =

!Line: 50 - error: function 'CreatePipeline' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 92 to 97 in 022aad6

	scissor.offset = {0, 0};
	scissor.extent = renderData.swapChainExtent;
	VkPipelineViewportStateCreateInfo viewportState = {};
	viewportState.sType = VK_STRUCTURE_TYPE_PIPELINE_VIEWPORT_STATE_CREATE_INFO;
	viewportState.viewportCount = 1;

!Line: 92 - error: use designated initializer list to initialize 'VkOffset2D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 221 to 226 in 022aad6

	CreateVertexBuffer(size_t bufferSize, const std::vector< DataT >& vertices)
	{
	if (vertices.empty())
	{
	return {};
	}

!Line: 221 - error: function 'CreateVertexBuffer' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 250 to 255 in 022aad6

	CreateIndexBuffer(std::vector< IndexType >& indices, const size_t numObjects)
	{
	if (!numObjects)
	{
	return {};
	}

!Line: 250 - error: function 'CreateIndexBuffer' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 307 to 312 in 022aad6

	CreateInstance()
	{
	if (ENABLE_VALIDATION && !CheckValidationLayerSupport())
	{
	utils::Assert(false, "validation layers asked but not supported!");
	}

!Line: 307 - error: function 'CreateInstance' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 342 to 347 in 022aad6

	CreateDevice()
	{
	uint32_t deviceCount = 0;
	vkEnumeratePhysicalDevices(Data::vk_instance, &deviceCount, nullptr);
	utils::Assert(deviceCount > 0, "failed to find GPUs with Vulkan support!");

!Line: 342 - error: function 'CreateDevice' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 351 to 356 in 022aad6

	const auto device = std::find_if(devices.begin(), devices.end(), [](const auto& device) {
	return IsDeviceSuitable(device, Data::renderData_[boundApplication_].surface);
	});
	utils::Assert(device != devices.end(), "Unable to find suitable device!");

!Line: 351 - error: use a boost version of this algorithm [boost-use-ranges,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 351 to 356 in 022aad6

	const auto device = std::find_if(devices.begin(), devices.end(), [](const auto& device) {
	return IsDeviceSuitable(device, Data::renderData_[boundApplication_].surface);
	});
	utils::Assert(device != devices.end(), "Unable to find suitable device!");

!Line: 351 - error: use a ranges version of this algorithm [modernize-use-ranges,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 429 to 434 in 022aad6

	CreateSwapchain()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	const auto swapChainSupport = QuerySwapChainSupport(Data::vk_physicalDevice, renderData.surface);
	const auto surfaceFormat = ChooseSwapSurfaceFormat(swapChainSupport.formats);

!Line: 429 - error: function 'CreateSwapchain' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 492 to 497 in 022aad6

	CreateColorResources()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	auto image = Texture::CreateImage(
	renderData.swapChainExtent.width, renderData.swapChainExtent.height, 1,

!Line: 492 - error: function 'CreateColorResources' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 509 to 514 in 022aad6

	CreateDepthResources()
	{
	const auto depthFormat = FindDepthFormat();
	auto& renderData = Data::renderData_.at(boundApplication_);
	const auto image = Texture::CreateImage(

!Line: 509 - error: function 'CreateDepthResources' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 526 to 531 in 022aad6

	CreateImageViews()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	renderData.swapChainImageViews.resize(renderData.swapChainImages.size());

!Line: 526 - error: function 'CreateImageViews' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 541 to 546 in 022aad6

	CreateRenderPass()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	VkAttachmentDescription colorAttachment = {};
	colorAttachment.format = renderData.swapChainImageFormat;

!Line: 541 - error: function 'CreateRenderPass' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 620 to 625 in 022aad6

	CreateFramebuffers()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	renderData.swapChainFramebuffers.resize(renderData.swapChainImageViews.size());

!Line: 620 - error: function 'CreateFramebuffers' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 647 to 652 in 022aad6

	CreateCommandPool()
	{
	auto queueFamilyIndicesTwo =
	FindQueueFamilies(Data::vk_physicalDevice, Data::renderData_[boundApplication_].surface);
	VkCommandPoolCreateInfo poolInfo = {};

!Line: 647 - error: function 'CreateCommandPool' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 686 to 691 in 022aad6

	clearValues[1].depthStencil = {1.0f, 0};
	VkRenderPassBeginInfo renderPassInfo{};
	renderPassInfo.sType = VK_STRUCTURE_TYPE_RENDER_PASS_BEGIN_INFO;
	renderPassInfo.renderPass = renderData.renderPass;
	renderPassInfo.renderArea.offset = {0, 0};

!Line: 686 - error: use designated initializer list to initialize 'VkClearDepthStencilValue' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 691 to 696 in 022aad6

	renderPassInfo.renderArea.offset = {0, 0};
	renderPassInfo.renderArea.extent = renderData.swapChainExtent;
	renderPassInfo.clearValueCount = static_cast< uint32_t >(clearValues.size());
	renderPassInfo.pClearValues = clearValues.data();
	renderPassInfo.framebuffer = renderData.swapChainFramebuffers[imageIndex];

!Line: 691 - error: use designated initializer list to initialize 'VkOffset2D' [modernize-use-designated-initializers,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 728 to 733 in 022aad6

	CreateQuadIndexBuffer()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	for (size_t layer = 0; layer < static_cast< size_t >(NUM_LAYERS); ++layer)
	{

!Line: 728 - error: function 'CreateQuadIndexBuffer' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 746 to 751 in 022aad6

	CreateQuadVertexBuffer()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	auto& vertices = renderData.vertices;
	for (size_t layer = 0; layer < static_cast< size_t >(NUM_LAYERS); layer++)

!Line: 746 - error: function 'CreateQuadVertexBuffer' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 767 to 772 in 022aad6

	CreateSyncObjects()
	{
	imageAvailableSemaphores_.resize(MAX_FRAMES_IN_FLIGHT);
	renderFinishedSemaphores_.resize(MAX_FRAMES_IN_FLIGHT);
	inFlightFences_.resize(MAX_FRAMES_IN_FLIGHT);

!Line: 767 - error: function 'CreateSyncObjects' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 800 to 805 in 022aad6

	CreatePipelineCache()
	{
	VkPipelineCacheCreateInfo pipelineCacheCreateInfo = {};
	pipelineCacheCreateInfo.sType = VK_STRUCTURE_TYPE_PIPELINE_CACHE_CREATE_INFO;
	vk_check_error(vkCreatePipelineCache(Data::vk_device, &pipelineCacheCreateInfo, nullptr,
	&Data::renderData_.at(boundApplication_).pipelineCache),

!Line: 800 - error: function 'CreatePipelineCache' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 834 to 839 in 022aad6

	CreateUniformBuffer()
	{
	auto& renderData = Data::renderData_[boundApplication_];
	constexpr VkDeviceSize bufferSize = sizeof(UniformBufferObject);
	for (size_t frame = 0; frame < MAX_FRAMES_IN_FLIGHT; frame++)

!Line: 834 - error: function 'CreateUniformBuffer' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 854 to 859 in 022aad6

	CreatePerInstanceBuffer()
	{
	auto& renderData = Data::renderData_[boundApplication_];
	constexpr VkDeviceSize SSBObufferSize = MAX_NUM_SPRITES * sizeof(PerInstanceBuffer);
	for (size_t frame = 0; frame < MAX_FRAMES_IN_FLIGHT; frame++)

!Line: 854 - error: function 'CreatePerInstanceBuffer' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 874 to 879 in 022aad6

	UpdateUniformBuffer()
	{
	auto& renderData = Data::renderData_.at(boundApplication_);
	for (uint32_t frame = 0; frame < MAX_FRAMES_IN_FLIGHT; ++frame)
	{

!Line: 874 - error: function 'UpdateUniformBuffer' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 928 to 933 in 022aad6

	DestroyPipeline()
	{
	vkDeviceWaitIdle(Data::vk_device);
	auto& renderData = Data::renderData_.at(boundApplication_);

!Line: 928 - error: function 'DestroyPipeline' can be made static or moved into an anonymous namespace to enforce internal linkage [misc-use-internal-linkage,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 1070 to 1075 in 022aad6

	if (numObjects == static_cast< uint32_t >(layerIdx))
	{
	numObjects++;
	}
	++renderData->totalNumMeshes;

!Line: 1070 - error: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 1081 to 1086 in 022aad6

	return {idx, layer, layerIdx};
	}
	void
	SubmitMeshData(const uint32_t idx, const TextureIDs& ids, const glm::mat4& modelMat,
	const glm::vec4& color)

!Line: 1081 - error: use designated initializer list to initialize 'RenderInfo' [modernize-use-designated-initializers,-warnings-as-errors]

!Line: 141 - note: aggregate type is defined here

Looper/engine/renderer/renderer.cpp

Lines 1154 to 1159 in 022aad6

	+ static_cast< size_t >(startingLine) * VERTICES_PER_LINE,
	bufferSize);
	vmaUnmapMemory(Data::vk_hAllocator, EditorData::lineVertexBuffer.allocation_);
	}
	}

!Line: 1154 - error: '*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations [readability-math-missing-parentheses,-warnings-as-errors]

Looper/engine/renderer/renderer.cpp

Lines 1217 to 1222 in 022aad6

	if (Data::renderData_.find(type) != Data::renderData_.end())
	{
	auto& renderData = Data::renderData_.at(type);
	for (size_t layer = 0; layer < static_cast< size_t >(NUM_LAYERS); ++layer)
	{

!Line: 1217 - error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]