Skip to content

Separate video presentation from Win32 window details - #3

Open
Krisztiaan wants to merge 2 commits into
OpenTS-Developers:mainfrom
Krisztiaan:refactor/platform-window-boundary
Open

Separate video presentation from Win32 window details#3
Krisztiaan wants to merge 2 commits into
OpenTS-Developers:mainfrom
Krisztiaan:refactor/platform-window-boundary

Conversation

@Krisztiaan

Copy link
Copy Markdown

Summary

Pass native window handles, drawable size, and refresh rate into the video layer instead of querying Win32 there. This preserves current behavior while creating a narrow boundary for future platform implementations.

Behavior and compatibility

Preserved behavior, no observable operational change should be user visible. Boundaries moved.

Validation

  • git diff --check 59fae722af8c77f213cd58c799cb92ab6c85b98b — passed
  • Visual Studio 2022 Win32 Debug and Release — not run locally 🍏 for now; CI will run both
  • tested CI built output via wine on macOS

Documentation

Does not change anything noteworthy yet.

Checklist

  • The change is focused; unrelated mechanical cleanup is separate
  • Compatibility effects and any migration are explicit
  • Validation distinguishes what passed, failed, and was not run
  • No prohibited assets, binaries, SDKs, credentials, or generated output are included

@Krisztiaan Krisztiaan changed the title Refactor/platform window boundary Separate video presentation from Win32 window details Aug 27, 2026
@github-actions

Copy link
Copy Markdown

Development builds of a7ed70f:

The links work without a GitHub account. Artifacts expire after 90 days, and this comment follows the latest successful build.

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