[PR #3598] [MERGED] build: Fix a couple more warnings. #3601

New issue

Closed

opened 2026-02-27 22:04:18 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 22:04:18 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/shadps4-emu/shadPS4/pull/3598
Author: @squidbus
Created: 9/13/2025
Status: ✅ Merged
Merged: 9/13/2025
Merged by: @squidbus

Base: main ← Head: warnings-2

📝 Commits (1)

6bd6473 build: Fix a couple more warnings.

📊 Changes

10 files changed (+27 additions, -16 deletions)

View changed files

📝 src/core/devtools/widget/cmd_list.cpp (+2 -2)
📝 src/core/devtools/widget/memory_map.cpp (+2 -2)
📝 src/core/libraries/fiber/fiber_context.s (+1 -1)
📝 src/core/libraries/kernel/threads/exception.h (+2 -2)
📝 src/core/libraries/libc_internal/printf.h (+7 -6)
📝 src/core/libraries/network/net.cpp (+1 -1)
📝 src/core/loader/symbols_resolver.h (+3 -0)
📝 src/core/tls.h (+6 -1)
📝 src/imgui/renderer/imgui_impl_vulkan.cpp (+1 -1)
📝 src/shader_recompiler/ir/condition.h (+2 -0)

📄 Description

Use snprintf instead of deprecated sprintf.
Check result of freeDescriptorSets in imgui backend.
Fix duplicate Stack struct name.
Fix a few more devtools text prints.
Fix missing default switch case in NameOf(Condition) and SymbolTypeToS.
Fix unsafe use of variable size array extension.
Fix ClearStack attributes on gcc.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/shadps4-emu/shadPS4/pull/3598 **Author:** [@squidbus](https://github.com/squidbus) **Created:** 9/13/2025 **Status:** ✅ Merged **Merged:** 9/13/2025 **Merged by:** [@squidbus](https://github.com/squidbus) **Base:** `main` ← **Head:** `warnings-2` --- ### 📝 Commits (1) - [`6bd6473`](https://github.com/shadps4-emu/shadPS4/commit/6bd6473a4673ffee746e290e8024f39df2e4cd69) build: Fix a couple more warnings. ### 📊 Changes **10 files changed** (+27 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `src/core/devtools/widget/cmd_list.cpp` (+2 -2) 📝 `src/core/devtools/widget/memory_map.cpp` (+2 -2) 📝 `src/core/libraries/fiber/fiber_context.s` (+1 -1) 📝 `src/core/libraries/kernel/threads/exception.h` (+2 -2) 📝 `src/core/libraries/libc_internal/printf.h` (+7 -6) 📝 `src/core/libraries/network/net.cpp` (+1 -1) 📝 `src/core/loader/symbols_resolver.h` (+3 -0) 📝 `src/core/tls.h` (+6 -1) 📝 `src/imgui/renderer/imgui_impl_vulkan.cpp` (+1 -1) 📝 `src/shader_recompiler/ir/condition.h` (+2 -0) </details> ### 📄 Description * Use `snprintf` instead of deprecated `sprintf`. * Check result of `freeDescriptorSets` in imgui backend. * Fix duplicate `Stack` struct name. * Fix a few more devtools text prints. * Fix missing default switch case in `NameOf(Condition)` and `SymbolTypeToS`. * Fix unsafe use of variable size array extension. * Fix `ClearStack` attributes on gcc. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>