Merge pull request #158 from JacobDomagala/152-address-staticanalysis-issues

JacobDomagala · web-flow · commit 0be50725262f · 2023-11-20T13:46:04.000+01:00
[#152]: Address `StaticAnalysis` issues
diff --git a/editor/editor.cpp b/editor/editor.cpp
@@ -246,7 +246,7 @@ Editor::HandleMouseDrag(const glm::vec2& currentCursorPos, const glm::vec2& axis
 }
 
 void
-Editor::SetMouseOnObject()
+Editor::SetMouseOnObject() const
 {
    if (m_mouseDrag && (m_movementOnEditorObject || m_movementOnGameObject))
    {
@@ -467,9 +467,8 @@ Editor::SetRenderLayerToDraw(int32_t layer)
    renderLayerToDraw_ = layer;
 }
 
-
 void
-Editor::SetupRendererData()
+Editor::SetupRendererData() const
 {
    renderer::VulkanRenderer::SetupData();
 
@@ -715,7 +714,7 @@ Editor::DrawBoundingBoxes()
 }
 
 void
-Editor::DrawGrid()
+Editor::DrawGrid() const
 {
    const auto levelSize = m_currentLevel->GetSize();
    const auto grad = m_gridCellSize;
diff --git a/editor/editor.hpp b/editor/editor.hpp
@@ -118,7 +118,7 @@ class Editor : public Application
    GetRenderNodes() const;
 
    void
-   DrawGrid();
+   DrawGrid() const;
 
    void
    DrawBoundingBoxes();
@@ -178,7 +178,7 @@ class Editor : public Application
    // GetEditorObjectByID(Object::ID ID);
 
    void
-   SetupRendererData();
+   SetupRendererData() const;
 
    void
    DrawEditorObjects();
@@ -217,7 +217,7 @@ class Editor : public Application
    ShowCursor(bool choice);
 
    void
-   SetMouseOnObject();
+   SetMouseOnObject() const;
 
    void
    FreeLevelData();
diff --git a/engine/core/work_queue.cpp b/engine/core/work_queue.cpp
@@ -8,7 +8,7 @@ namespace looper {
 void
 WorkQueue::PushWorkUnit(const WorkQueue::Precondition& prec, const WorkQueue::WorkUnit& work)
 {
-   queue_.push_back({prec, work});
+   queue_.emplace_back(prec, work);
 }
 
 void
diff --git a/engine/game/animatable.cpp b/engine/game/animatable.cpp
@@ -85,19 +85,15 @@ Animatable::SetCorrectAnimationPoint(time::milliseconds& updateTime)
    auto animationDurationMs =
       time::Timer::ConvertToMs(currentState_.m_currentAnimationPoint->m_timeDuration);
 
-   if (updateTime >= animationDurationMs)
+   while (updateTime >= animationDurationMs
+          && currentState_.m_currentAnimationPoint != m_animationPoints.begin())
    {
-      do
-      {
-         updateTime -= animationDurationMs;
-         animationValue +=
-            currentState_.m_currentAnimationEnd - currentState_.m_currentAnimationBegin;
+      updateTime -= animationDurationMs;
+      animationValue += currentState_.m_currentAnimationEnd - currentState_.m_currentAnimationBegin;
 
-         UpdateAnimationPoint();
-         animationDurationMs =
-            time::Timer::ConvertToMs(currentState_.m_currentAnimationPoint->m_timeDuration);
-      } while (updateTime >= animationDurationMs
-               && currentState_.m_currentAnimationPoint != m_animationPoints.begin());
+      UpdateAnimationPoint();
+      animationDurationMs =
+         time::Timer::ConvertToMs(currentState_.m_currentAnimationPoint->m_timeDuration);
    }
 
    return animationValue;
diff --git a/engine/game/level.cpp b/engine/game/level.cpp
@@ -469,7 +469,8 @@ Level::GenerateTextureForCollision()
    const size_t numChannels = 4;
    const auto size = static_cast< size_t >(width * height * numChannels);
 
-   auto* data = new unsigned char[size];
+   auto data = FileManager::ImageHandleType{new unsigned char[size],
+                                            [](const uint8_t* ptr) { delete[] ptr; }};
    const auto& nodes = m_pathFinder.GetAllNodes();
 
    for (size_t h = 0; h < height; ++h)
@@ -488,10 +489,7 @@ Level::GenerateTextureForCollision()
       }
    }
 
-   collisionTextureData_ = {FileManager::ImageHandleType{reinterpret_cast< unsigned char* >(data),
-                                                         [](const uint8_t* ptr) { delete[] ptr; }},
-                            {width, height},
-                            numChannels};
+   collisionTextureData_ = {std::move(data), {width, height}, numChannels};
 
    // To avoid blurry edges
    renderer::TextureProperties props;
diff --git a/engine/game/object.cpp b/engine/game/object.cpp
@@ -1,8 +1,8 @@
 #include "object.hpp"
 
 namespace looper {
-
-static std::string
+namespace {
+std::string
 TypeToString(ObjectType type)
 {
    std::string typeStr;
@@ -41,6 +41,7 @@ TypeToString(ObjectType type)
 
    return typeStr;
 }
+} // namespace
 
 ObjectType
 Object::GetTypeFromString(const std::string& stringType)
diff --git a/engine/game/player.hpp b/engine/game/player.hpp
@@ -57,11 +57,6 @@ class Player : public GameObject
    /*void
    CreateSprite(const glm::vec2& position, const glm::ivec2& size, const std::string& fileName);*/
 
-   void
-   Hit(int32_t /*dmg*/) override
-   {
-   }
-
  private:
    void
    UpdateInternal(bool isReverse) override;
diff --git a/engine/input/input_manager.cpp b/engine/input/input_manager.cpp
@@ -25,6 +25,7 @@ BroadcastEvent(const auto& listeners, auto& event)
 }
 
 void
+// cppcheck-suppress constParameterCallback
 InputManager::InternalKeyCallback(GLFWwindow* window, int32_t key, int32_t scancode,
                                   int32_t action, int32_t mods)
 {
@@ -37,6 +38,7 @@ InputManager::InternalKeyCallback(GLFWwindow* window, int32_t key, int32_t scanc
 }
 
 void
+// cppcheck-suppress constParameterCallback
 InputManager::InternalCharCallback(GLFWwindow* window, uint32_t key)
 {
    Logger::Trace("GLFW char {}", key);
@@ -47,6 +49,7 @@ InputManager::InternalCharCallback(GLFWwindow* window, uint32_t key)
 }
 
 void
+// cppcheck-suppress constParameterCallback
 InputManager::InternalMouseButtonCallback(GLFWwindow* window, int32_t button, int32_t action,
                                           int32_t mods)
 {
@@ -58,6 +61,7 @@ InputManager::InternalMouseButtonCallback(GLFWwindow* window, int32_t button, in
 }
 
 void
+// cppcheck-suppress constParameterCallback
 InputManager::InternalCursorPositionCallback(GLFWwindow* window, double xPos, double yPos)
 {
    Logger::Trace("GLFW cursor pos {} {}", xPos, yPos);
@@ -69,6 +73,7 @@ InputManager::InternalCursorPositionCallback(GLFWwindow* window, double xPos, do
 }
 
 void
+// cppcheck-suppress constParameterCallback
 InputManager::InternalMouseScrollCallback(GLFWwindow* window, double xOffset, double yOffset)
 {
    Logger::Trace("GLFW scroll {} {}", xOffset, yOffset);
@@ -79,6 +84,7 @@ InputManager::InternalMouseScrollCallback(GLFWwindow* window, double xOffset, do
 }
 
 void
+// cppcheck-suppress constParameterCallback
 InputManager::InternalWindowFocusCallback(GLFWwindow* window, int32_t focused)
 {
    Logger::Trace("GLFW window focus {}", focused);
diff --git a/engine/renderer/buffer.cpp b/engine/renderer/buffer.cpp
@@ -54,7 +54,7 @@ Buffer::CopyData(const void* data) const
 }
 
 void
-Buffer::CopyDataWithStaging(void* data, size_t dataSize) const
+Buffer::CopyDataWithStaging(const void* data, const size_t dataSize) const
 {
    VkBuffer stagingBuffer = {};
    VkDeviceMemory stagingBufferMemory = {};
@@ -72,7 +72,7 @@ Buffer::CopyDataWithStaging(void* data, size_t dataSize) const
 }
 
 void
-Buffer::CopyDataToImageWithStaging(VkImage image, void* data, size_t dataSize,
+Buffer::CopyDataToImageWithStaging(VkImage image, const void* data, const size_t dataSize,
                                    const std::vector< VkBufferImageCopy >& copyRegions)
 {
    VkBuffer stagingBuffer = {};
diff --git a/engine/renderer/buffer.hpp b/engine/renderer/buffer.hpp
@@ -39,10 +39,10 @@ class Buffer
    CopyData(const void* data) const;
 
    void
-   CopyDataWithStaging(void* data, size_t dataSize) const;
+   CopyDataWithStaging(const void* data, const size_t dataSize) const;
 
    static void
-   CopyDataToImageWithStaging(VkImage image, void* data, size_t dataSize,
+   CopyDataToImageWithStaging(VkImage image, const void* data, const size_t dataSize,
                               const std::vector< VkBufferImageCopy >& copyRegions);
 
    void
diff --git a/engine/renderer/renderer.cpp b/engine/renderer/renderer.cpp
@@ -718,7 +718,8 @@ VulkanRenderer::Initialize(GLFWwindow* windowHandle, ApplicationType type)
    for (uint32_t layer = 0; layer < NUM_LAYERS; ++layer)
    {
       renderData.vertices.at(layer).resize(MAX_NUM_VERTICES_PER_LAYER);
-      renderData.indices.at(layer).resize(MAX_SPRITES_PER_LAYER * INDICES_PER_SPRITE);
+      renderData.indices.at(layer).resize(static_cast< size_t >(MAX_SPRITES_PER_LAYER)
+                                          * static_cast< size_t >(INDICES_PER_SPRITE));
    }
 
    renderData.perInstance.resize(MAX_NUM_SPRITES);
diff --git a/engine/renderer/texture.cpp b/engine/renderer/texture.cpp
@@ -59,22 +59,16 @@ Texture::CreateTextureImage(const FileManager::ImageData& data)
       CreateImageView(m_textureImage, m_format, VK_IMAGE_ASPECT_COLOR_BIT, m_mips, false);
 
    CreateTextureSampler();
-
-   TransitionImageLayout(VK_IMAGE_LAYOUT_UNDEFINED, VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, m_mips);
-
-   CopyBufferToImage(data.m_bytes.get());
-
-   // transitioned to VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL while generating mipmaps
-   GenerateMipmaps(m_textureImage, VK_FORMAT_R8G8B8A8_SRGB, static_cast< int32_t >(m_width),
-                   static_cast< int32_t >(m_height), m_mips);
+   UpdateTexture(data);
 }
 
 void
-Texture::UpdateTexture(const FileManager::ImageData& data)
+Texture::UpdateTexture(const FileManager::ImageData& data) const
 {
-   TransitionImageLayout(VK_IMAGE_LAYOUT_UNDEFINED, VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, m_mips);
+   TransitionImageLayout(m_textureImage, VK_IMAGE_LAYOUT_UNDEFINED,
+                         VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, m_mips);
 
-   CopyBufferToImage(data.m_bytes.get());
+   CopyBufferToImage(m_textureImage, m_width, m_height, data.m_bytes.get());
 
    // transitioned to VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL while generating mipmaps
    GenerateMipmaps(m_textureImage, VK_FORMAT_R8G8B8A8_SRGB, static_cast< int32_t >(m_width),
@@ -340,7 +334,8 @@ Texture::CreateTextureSampler()
 }
 
 void
-Texture::CopyBufferToImage(VkImage image, uint32_t texWidth, uint32_t texHeight, uint8_t* data)
+Texture::CopyBufferToImage(VkImage image, uint32_t texWidth, uint32_t texHeight,
+                           const uint8_t* data)
 {
    VkBufferImageCopy region = {};
    region.bufferOffset = 0;
@@ -360,7 +355,7 @@ Texture::CopyBufferToImage(VkImage image, uint32_t texWidth, uint32_t texHeight,
 
 void
 Texture::CopyBufferToCubemapImage(VkImage image, uint32_t texWidth, uint32_t texHeight,
-                                  uint8_t* data)
+                                  const uint8_t* data)
 {
    constexpr auto num_faces = 6;
    const auto single_face_size =
@@ -394,12 +389,6 @@ Texture::CopyBufferToCubemapImage(VkImage image, uint32_t texWidth, uint32_t tex
                                   VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL, 1, true);
 }
 
-void
-Texture::CopyBufferToImage(uint8_t* data)
-{
-   CopyBufferToImage(m_textureImage, m_width, m_height, data);
-}
-
 void
 Texture::TransitionImageLayout(VkImage image, VkImageLayout oldLayout, VkImageLayout newLayout,
                                uint32_t mipLevels, bool cubemap)
@@ -450,13 +439,6 @@ Texture::TransitionImageLayout(VkImage image, VkImageLayout oldLayout, VkImageLa
    Command::EndSingleTimeCommands(commandBuffer);
 }
 
-void
-Texture::TransitionImageLayout(VkImageLayout oldLayout, VkImageLayout newLayout, uint32_t mipLevels)
-{
-   TransitionImageLayout(m_textureImage, oldLayout, newLayout, mipLevels);
-}
-
-
 /**************************************************************************************************
  *************************************** TEXTURE LIBRARY ******************************************
  *************************************************************************************************/
diff --git a/engine/renderer/texture.hpp b/engine/renderer/texture.hpp
@@ -34,7 +34,7 @@ class Texture
    Destroy();
 
    void
-   UpdateTexture(const FileManager::ImageData& data);
+   UpdateTexture(const FileManager::ImageData& data) const;
 
    static std::pair< VkImage, VkDeviceMemory >
    CreateImage(uint32_t width, uint32_t height, uint32_t mipLevels,
@@ -57,10 +57,11 @@ class Texture
                          uint32_t mipLevels, bool cubemap = false);
 
    static void
-   CopyBufferToImage(VkImage image, uint32_t texWidth, uint32_t texHeight, uint8_t* data);
+   CopyBufferToImage(VkImage image, uint32_t texWidth, uint32_t texHeight, const uint8_t* data);
 
    static void
-   CopyBufferToCubemapImage(VkImage image, uint32_t texWidth, uint32_t texHeight, uint8_t* data);
+   CopyBufferToCubemapImage(VkImage image, uint32_t texWidth, uint32_t texHeight,
+                            const uint8_t* data);
 
    static VkDescriptorSet
    CreateDescriptorSet(VkSampler sampler, VkImageView image_view, VkImageLayout image_layout,
@@ -91,12 +92,6 @@ class Texture
    void
    CreateTextureImage(const FileManager::ImageData& data);
 
-   void
-   TransitionImageLayout(VkImageLayout oldLayout, VkImageLayout newLayout, uint32_t mipLevels);
-
-   void
-   CopyBufferToImage(uint8_t* data);
-
  private:
    TextureID id_ = {};
    TextureType m_type = {};
diff --git a/engine/utils/file_manager.cpp b/engine/utils/file_manager.cpp
@@ -82,22 +82,6 @@ FileManager::LoadImageData(std::string_view fileName)
    return {std::move(textureData), {w, h}, n};
 }
 
-void
-FileManager::SaveImageFile(std::string_view fileName, const ImageData& image)
-{
-   const auto pathToImage = std::filesystem::path(IMAGES_DIR / fileName).string();
-
-   std::ofstream file(pathToImage, std::ios::out | std::ios::binary | std::ios::trunc);
-   if (!file)
-   {
-      Logger::Fatal("FileManager::SaveImageFile -> {} can't be opened!", pathToImage);
-   }
-
-   file.write(reinterpret_cast< const char* >(image.m_bytes.get()),
-              image.m_size.x * image.m_size.y * image.m_format);
-   file.close();
-}
-
 nlohmann::json
 FileManager::LoadJsonFile(std::string_view pathToFile)
 {
diff --git a/engine/utils/file_manager.hpp b/engine/utils/file_manager.hpp
@@ -50,9 +50,6 @@ class FileManager
    static ImageData
    LoadImageData(std::string_view fileName);
 
-   static void
-   SaveImageFile(std::string_view fileName, const ImageData& image);
-
    static nlohmann::json
    LoadJsonFile(std::string_view pathToFile);
 

Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,7 @@ Editor::HandleMouseDrag(const glm::vec2& currentCursorPos, const glm::vec2& axis`
`246`	`246`	`}`
`247`	`247`
`248`	`248`	`void`
`249`		`-Editor::SetMouseOnObject()`
	`249`	`+Editor::SetMouseOnObject() const`
`250`	`250`	`{`
`251`	`251`	`if (m_mouseDrag && (m_movementOnEditorObject \|\| m_movementOnGameObject))`
`252`	`252`	`{`
`@@ -467,9 +467,8 @@ Editor::SetRenderLayerToDraw(int32_t layer)`
`467`	`467`	`renderLayerToDraw_ = layer;`
`468`	`468`	`}`
`469`	`469`
`470`		`-`
`471`	`470`	`void`
`472`		`-Editor::SetupRendererData()`
	`471`	`+Editor::SetupRendererData() const`
`473`	`472`	`{`
`474`	`473`	`renderer::VulkanRenderer::SetupData();`
`475`	`474`
`@@ -715,7 +714,7 @@ Editor::DrawBoundingBoxes()`
`715`	`714`	`}`
`716`	`715`
`717`	`716`	`void`
`718`		`-Editor::DrawGrid()`
	`717`	`+Editor::DrawGrid() const`
`719`	`718`	`{`
`720`	`719`	`const auto levelSize = m_currentLevel->GetSize();`
`721`	`720`	`const auto grad = m_gridCellSize;`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ namespace looper {`
`8`	`8`	`void`
`9`	`9`	`WorkQueue::PushWorkUnit(const WorkQueue::Precondition& prec, const WorkQueue::WorkUnit& work)`
`10`	`10`	`{`
`11`		`- queue_.push_back({prec, work});`
	`11`	`+ queue_.emplace_back(prec, work);`
`12`	`12`	`}`
`13`	`13`
`14`	`14`	`void`
Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,7 @@ BroadcastEvent(const auto& listeners, auto& event)`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`void`
	`28`	`+// cppcheck-suppress constParameterCallback`
`28`	`29`	`InputManager::InternalKeyCallback(GLFWwindow* window, int32_t key, int32_t scancode,`
`29`	`30`	`int32_t action, int32_t mods)`
`30`	`31`	`{`
`@@ -37,6 +38,7 @@ InputManager::InternalKeyCallback(GLFWwindow* window, int32_t key, int32_t scanc`
`37`	`38`	`}`
`38`	`39`
`39`	`40`	`void`
	`41`	`+// cppcheck-suppress constParameterCallback`
`40`	`42`	`InputManager::InternalCharCallback(GLFWwindow* window, uint32_t key)`
`41`	`43`	`{`
`42`	`44`	`Logger::Trace("GLFW char {}", key);`
`@@ -47,6 +49,7 @@ InputManager::InternalCharCallback(GLFWwindow* window, uint32_t key)`
`47`	`49`	`}`
`48`	`50`
`49`	`51`	`void`
	`52`	`+// cppcheck-suppress constParameterCallback`
`50`	`53`	`InputManager::InternalMouseButtonCallback(GLFWwindow* window, int32_t button, int32_t action,`
`51`	`54`	`int32_t mods)`
`52`	`55`	`{`
`@@ -58,6 +61,7 @@ InputManager::InternalMouseButtonCallback(GLFWwindow* window, int32_t button, in`
`58`	`61`	`}`
`59`	`62`
`60`	`63`	`void`
	`64`	`+// cppcheck-suppress constParameterCallback`
`61`	`65`	`InputManager::InternalCursorPositionCallback(GLFWwindow* window, double xPos, double yPos)`
`62`	`66`	`{`
`63`	`67`	`Logger::Trace("GLFW cursor pos {} {}", xPos, yPos);`
`@@ -69,6 +73,7 @@ InputManager::InternalCursorPositionCallback(GLFWwindow* window, double xPos, do`
`69`	`73`	`}`
`70`	`74`
`71`	`75`	`void`
	`76`	`+// cppcheck-suppress constParameterCallback`
`72`	`77`	`InputManager::InternalMouseScrollCallback(GLFWwindow* window, double xOffset, double yOffset)`
`73`	`78`	`{`
`74`	`79`	`Logger::Trace("GLFW scroll {} {}", xOffset, yOffset);`
`@@ -79,6 +84,7 @@ InputManager::InternalMouseScrollCallback(GLFWwindow* window, double xOffset, do`
`79`	`84`	`}`
`80`	`85`
`81`	`86`	`void`
	`87`	`+// cppcheck-suppress constParameterCallback`
`82`	`88`	`InputManager::InternalWindowFocusCallback(GLFWwindow* window, int32_t focused)`
`83`	`89`	`{`
`84`	`90`	`Logger::Trace("GLFW window focus {}", focused);`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ Buffer::CopyData(const void* data) const`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`void`
`57`		`-Buffer::CopyDataWithStaging(void* data, size_t dataSize) const`
	`57`	`+Buffer::CopyDataWithStaging(const void* data, const size_t dataSize) const`
`58`	`58`	`{`
`59`	`59`	`VkBuffer stagingBuffer = {};`
`60`	`60`	`VkDeviceMemory stagingBufferMemory = {};`
`@@ -72,7 +72,7 @@ Buffer::CopyDataWithStaging(void* data, size_t dataSize) const`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`void`
`75`		`-Buffer::CopyDataToImageWithStaging(VkImage image, void* data, size_t dataSize,`
	`75`	`+Buffer::CopyDataToImageWithStaging(VkImage image, const void* data, const size_t dataSize,`
`76`	`76`	`const std::vector< VkBufferImageCopy >& copyRegions)`
`77`	`77`	`{`
`78`	`78`	`VkBuffer stagingBuffer = {};`