Skip to content

Keep nonsecure screen savers from interrupting foreground Windows play - #831

Closed
monsterlag wants to merge 3 commits into
NotRexed:mainfrom
monsterlag:fix/nonsecure-screensaver
Closed

monsterlag wants to merge 3 commits into
NotRexed:mainfrom
monsterlag:fix/nonsecure-screensaver

Conversation

@monsterlag

@monsterlag monsterlag commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Keep a nonsecure Windows screen saver from interrupting controller-only play while the renderer is active and the game is foreground and non-minimized. Secure/unknown policy, failed policy queries, background/minimized windows and other system commands retain normal Windows handling. No screen-saver, password, power or sleep configuration is changed.

Updated on main b2765ba6 through a normal merge, preserving its API6 window/key notification and current CI. The seven-line product hunk and all18 authored cases/assertions are preserved; the fixture adds only the mock required by the new upstream key notification.

Validation at 54ee3ac6: all18 complete-WndProc API-mock cases pass with Windows GNU Clang and WSL GCC. Hosted Windows source checks, Linux source checks, SPIR-V checks, Windows renderer/builder and Linux AppImage pass. The identical complete host/display.c also compiled in the separate private API6 player build.

Real screen-saver/security/power behavior remains untested; ordinary gameplay does not establish that contract. Refs #809, scoped to Windows.

API contracts: WM_SYSCOMMAND and SPI_GETSCREENSAVESECURE.

Merge b2765ba without rewriting published history. Retain the narrow foreground/nonminimized policy, upstream window notifications/API6/CI, and all18 authored mock cases; add only the unrelated-action mock required by the current WndProc.

Assisted-by: Codex

Co-Authored-By: Codex <noreply@openai.com>
NotRexed pushed a commit that referenced this pull request Nov 9, 2026
@NotRexed

NotRexed commented Nov 9, 2026

Copy link
Copy Markdown
Owner

This is in dev and main now (landed in 2011a1c, "Merge PR #831"; the test fixture stub was later adjusted in b645b05 to follow #946), so closing. Thanks!

@NotRexed NotRexed closed this Nov 9, 2026
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.

2 participants