Skip to content

Use malloc to make IKEYState on win32 - #8826

Merged
sithlord48 merged 1 commit into
masterfrom
fixTestFail
Aug 12, 2025
Merged

Use malloc to make IKEYState on win32#8826
sithlord48 merged 1 commit into
masterfrom
fixTestFail

Conversation

@sithlord48

@sithlord48 sithlord48 commented Aug 7, 2025

Copy link
Copy Markdown
Member

For windows we use malloc to make the IKeyState when made with a screen or buffer size.

I suspect this is causing random failures in the KeyTest.. running a bunch to see if i can trigger it .
Keeping this around for a bit incase it keeps happening

added guarding to keep sonar happy..

@sithlord48
sithlord48 force-pushed the fixTestFail branch 3 times, most recently from a426829 to 30fa311 Compare August 12, 2025 01:57
@sithlord48 sithlord48 changed the title Revert "refactor: IKeyState remove need for malloc in alloc methods" Use malloc to make IKEYState on win32 Aug 12, 2025
@sithlord48
sithlord48 requested a review from nbolton August 12, 2025 11:40
@sithlord48
sithlord48 marked this pull request as ready for review August 12, 2025 11:40
Comment thread src/lib/deskflow/IKeyState.cpp
@sithlord48
sithlord48 requested a review from nbolton August 12, 2025 15:40
@sithlord48
sithlord48 enabled auto-merge (rebase) August 12, 2025 15:40
@sithlord48
sithlord48 merged commit 8d92a8d into master Aug 12, 2025
29 of 30 checks passed
@sithlord48
sithlord48 deleted the fixTestFail branch August 12, 2025 16:14
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