Skip to content

Move checkIsRunning after tryElevate to ensure that we have permission to create GLOBAL event#149

Open
stephen9358 wants to merge 1 commit intojagt:masterfrom
stephen9358:patch-1
Open

Move checkIsRunning after tryElevate to ensure that we have permission to create GLOBAL event#149
stephen9358 wants to merge 1 commit intojagt:masterfrom
stephen9358:patch-1

Conversation

@stephen9358
Copy link

Fix the mismatched error message when running with non-admin context.

@jagt
Copy link
Owner

jagt commented Oct 23, 2023

Hi! What's this PR is fixing? Is that elevate would fail in some cases?

@stephen9358
Copy link
Author

Hi! What's this PR is fixing? Is that elevate would fail in some cases?

Creating global event will fail without elevating first, so we see a MessageBox saying that already running, which it's not an accurate error message.


image

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