Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed issue of video drivers not starting properly in some computers when Windhawk with Classic theme mod is started very early during system start. #1059

Conversation

levitation
Copy link
Contributor

No description provided.

…e enable mod classic-theme-enable-with-extended-compatibility.wh.cpp
…th 1ms polling in case the conditions require to retry applying the classic theme.
… Windhawk service, which is necessary to ensure that classic theme is properly applied before any programs including Taskbar run. The mod code is same, only readme instruction has changed. Also added one more classic theme related mod to the recommended mods list in the readme.
…when Windhawk with Classic theme mod is started very early during system start.
#else
DWORD waitInterval = 1;
#endif
if (WaitForSingleObject(g_initThreadStopSignal, waitInterval) != WAIT_TIMEOUT) {
Copy link
Member

@m417z m417z Oct 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the actual change? I assume it will still be 1?

Copy link
Contributor Author

@levitation levitation Oct 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, actual change is on line 398. Before it applied the session connected status check only to RDP sessions. Now it applies it to console sessions as well.

The code snippet you cited is

#ifdef _DEBUG
DWORD waitInterval = 1000;
#else 
DWORD waitInterval = 1;
#endif

So the interval is still 1 in a live mod.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Totally missed it, saw only the comment :)

@m417z m417z merged commit 551d898 into ramensoftware:main Oct 11, 2024
4 checks passed
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