Skip to content

Fix priority crash, import, export, history and opacity paths - #2097

Open
Loggableim wants to merge 2 commits into
armory3d:mainfrom
Loggableim:agent/armorpaint-priority-fixes
Open

Fix priority crash, import, export, history and opacity paths#2097
Loggableim wants to merge 2 commits into
armory3d:mainfrom
Loggableim:agent/armorpaint-priority-fixes

Conversation

@Loggableim

Copy link
Copy Markdown

Summary

Draft patch for the priority issue triage. The changes add defensive validation at the boundaries where malformed config, missing GPU/file data, failed layer creation, and history traversal previously reached unchecked dereferences.

Issue coverage

Verification

  • python paint/tests/test_priority_fixes.py -> 6 regression checks passed.
  • GCC -fsyntax-only passed for all changed C translation units.
  • paint\\..\\base\\make.bat --help generated the Windows project files and compiled all bundled shaders successfully.
  • The native MSBuild attempt is currently blocked by the environment: Visual Studio C++ targets (Microsoft.Cpp.Default.props / MSBuild.exe) are not installed. Linux/Vulkan and GPU-specific issue reproduction was not possible in this Windows checkout.

This is intentionally a draft so the upstream maintainer can review the platform-specific paths and run the native Linux/Windows runtime scenarios.

@Loggableim
Loggableim marked this pull request as ready for review July 17, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant