From 6ce33187f952625423939c2a99abcda0fab1a8a8 Mon Sep 17 00:00:00 2001 From: WinterSnowfall Date: Sat, 8 Mar 2025 18:38:45 +0200 Subject: [PATCH] [dxvk] Static analysis nits and fixes --- src/dxvk/dxvk_access.cpp | 4 ++-- src/dxvk/dxvk_context.h | 4 ++-- src/dxvk/dxvk_cs.h | 2 +- src/dxvk/dxvk_device_filter.cpp | 5 ++--- src/dxvk/dxvk_options.cpp | 37 ++++++++++++++++----------------- src/dxvk/dxvk_shader.cpp | 4 ++-- src/dxvk/dxvk_state_cache.cpp | 2 +- src/vulkan/vulkan_loader.cpp | 5 ++--- 8 files changed, 30 insertions(+), 33 deletions(-) diff --git a/src/dxvk/dxvk_access.cpp b/src/dxvk/dxvk_access.cpp index d900c020a..501abea4f 100644 --- a/src/dxvk/dxvk_access.cpp +++ b/src/dxvk/dxvk_access.cpp @@ -2,8 +2,8 @@ namespace dxvk { - DxvkObjectTracker::DxvkObjectTracker() { - m_head = std::make_unique(); + DxvkObjectTracker::DxvkObjectTracker() + : m_head ( std::make_unique() ) { m_next = m_head.get(); } diff --git a/src/dxvk/dxvk_context.h b/src/dxvk/dxvk_context.h index 86fe31bdc..c4bb8593e 100644 --- a/src/dxvk/dxvk_context.h +++ b/src/dxvk/dxvk_context.h @@ -1659,8 +1659,8 @@ namespace dxvk { void uploadImageHw( const Rc& image, const Rc& source, - VkDeviceSize subresourceAlignment, - VkDeviceSize sourceOffset); + VkDeviceSize sourceOffset, + VkDeviceSize subresourceAlignment); void performClear( const Rc& imageView, diff --git a/src/dxvk/dxvk_cs.h b/src/dxvk/dxvk_cs.h index df684db81..9c235e3a9 100644 --- a/src/dxvk/dxvk_cs.h +++ b/src/dxvk/dxvk_cs.h @@ -318,7 +318,7 @@ namespace dxvk { DxvkCsChunkFlags m_flags; alignas(64) - char m_data[DxvkCsChunkSize]; + char m_data[DxvkCsChunkSize] = { }; template void* alloc(size_t extra) { diff --git a/src/dxvk/dxvk_device_filter.cpp b/src/dxvk/dxvk_device_filter.cpp index df9c09694..b14a32bf8 100644 --- a/src/dxvk/dxvk_device_filter.cpp +++ b/src/dxvk/dxvk_device_filter.cpp @@ -5,9 +5,8 @@ namespace dxvk { DxvkDeviceFilter::DxvkDeviceFilter( DxvkDeviceFilterFlags flags, const DxvkOptions& options) - : m_flags(flags) { - m_matchDeviceName = env::getEnvVar("DXVK_FILTER_DEVICE_NAME"); - + : m_flags(flags) + , m_matchDeviceName ( env::getEnvVar("DXVK_FILTER_DEVICE_NAME") ) { if (m_matchDeviceName.empty()) m_matchDeviceName = options.deviceFilter; diff --git a/src/dxvk/dxvk_options.cpp b/src/dxvk/dxvk_options.cpp index d2d455c33..72ee2a82b 100644 --- a/src/dxvk/dxvk_options.cpp +++ b/src/dxvk/dxvk_options.cpp @@ -2,24 +2,23 @@ namespace dxvk { - DxvkOptions::DxvkOptions(const Config& config) { - enableDebugUtils = config.getOption ("dxvk.enableDebugUtils", false); - enableStateCache = config.getOption ("dxvk.enableStateCache", true); - enableMemoryDefrag = config.getOption("dxvk.enableMemoryDefrag", Tristate::Auto); - numCompilerThreads = config.getOption ("dxvk.numCompilerThreads", 0); - enableGraphicsPipelineLibrary = config.getOption("dxvk.enableGraphicsPipelineLibrary", Tristate::Auto); - trackPipelineLifetime = config.getOption("dxvk.trackPipelineLifetime", Tristate::Auto); - useRawSsbo = config.getOption("dxvk.useRawSsbo", Tristate::Auto); - hud = config.getOption("dxvk.hud", ""); - tearFree = config.getOption("dxvk.tearFree", Tristate::Auto); - latencySleep = config.getOption("dxvk.latencySleep", Tristate::Auto); - latencyTolerance = config.getOption ("dxvk.latencyTolerance", 1000); - disableNvLowLatency2 = config.getOption("dxvk.disableNvLowLatency2", Tristate::Auto); - hideIntegratedGraphics = config.getOption ("dxvk.hideIntegratedGraphics", false); - zeroMappedMemory = config.getOption ("dxvk.zeroMappedMemory", false); - allowFse = config.getOption ("dxvk.allowFse", false); - deviceFilter = config.getOption("dxvk.deviceFilter", ""); - tilerMode = config.getOption("dxvk.tilerMode", Tristate::Auto); - } + DxvkOptions::DxvkOptions(const Config& config) + : enableDebugUtils ( config.getOption ("dxvk.enableDebugUtils", false) ) + , enableStateCache ( config.getOption ("dxvk.enableStateCache", true) ) + , enableMemoryDefrag ( config.getOption ("dxvk.enableMemoryDefrag", Tristate::Auto) ) + , numCompilerThreads ( config.getOption ("dxvk.numCompilerThreads", 0) ) + , enableGraphicsPipelineLibrary ( config.getOption ("dxvk.enableGraphicsPipelineLibrary", Tristate::Auto) ) + , trackPipelineLifetime ( config.getOption ("dxvk.trackPipelineLifetime", Tristate::Auto) ) + , useRawSsbo ( config.getOption ("dxvk.useRawSsbo", Tristate::Auto) ) + , hud ( config.getOption("dxvk.hud", "") ) + , tearFree ( config.getOption ("dxvk.tearFree", Tristate::Auto) ) + , latencySleep ( config.getOption ("dxvk.latencySleep", Tristate::Auto) ) + , latencyTolerance ( config.getOption ("dxvk.latencyTolerance", 1000) ) + , disableNvLowLatency2 ( config.getOption ("dxvk.disableNvLowLatency2", Tristate::Auto) ) + , hideIntegratedGraphics ( config.getOption ("dxvk.hideIntegratedGraphics", false) ) + , zeroMappedMemory ( config.getOption ("dxvk.zeroMappedMemory", false) ) + , allowFse ( config.getOption ("dxvk.allowFse", false) ) + , tilerMode ( config.getOption ("dxvk.tilerMode", Tristate::Auto) ) + , deviceFilter ( config.getOption("dxvk.deviceFilter", "") ) { } } diff --git a/src/dxvk/dxvk_shader.cpp b/src/dxvk/dxvk_shader.cpp index 58d64c04b..9caed814d 100644 --- a/src/dxvk/dxvk_shader.cpp +++ b/src/dxvk/dxvk_shader.cpp @@ -793,7 +793,7 @@ namespace dxvk { // Only consider variables that have a desired location if (candidates.find(varId) != candidates.end()) { - VarInfo varInfo; + VarInfo varInfo = { }; varInfo.varId = varId; varInfo.decorationOffset = 0; @@ -1080,7 +1080,7 @@ namespace dxvk { default:; } - iter++; + ++iter; } // Insert new null variables diff --git a/src/dxvk/dxvk_state_cache.cpp b/src/dxvk/dxvk_state_cache.cpp index bf669dcf2..77c729a4a 100644 --- a/src/dxvk/dxvk_state_cache.cpp +++ b/src/dxvk/dxvk_state_cache.cpp @@ -197,7 +197,7 @@ namespace dxvk { size_t m_size = 0; size_t m_read = 0; - char m_data[MaxSize]; + char m_data[MaxSize] = { }; template bool read(T& data) { diff --git a/src/vulkan/vulkan_loader.cpp b/src/vulkan/vulkan_loader.cpp index 17e9b629c..6844bd0d7 100644 --- a/src/vulkan/vulkan_loader.cpp +++ b/src/vulkan/vulkan_loader.cpp @@ -45,9 +45,8 @@ namespace dxvk::vk { std::tie(m_library, m_getInstanceProcAddr) = loadVulkanLibrary(); } - LibraryLoader::LibraryLoader(PFN_vkGetInstanceProcAddr loaderProc) { - m_getInstanceProcAddr = loaderProc; - } + LibraryLoader::LibraryLoader(PFN_vkGetInstanceProcAddr loaderProc) + : m_getInstanceProcAddr ( loaderProc ) { } LibraryLoader::~LibraryLoader() { if (m_library)