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

jagt / clumsy

clumsy makes your network condition on Windows significantly worse, but in a controlled and interactive manner.

http://jagt.github.io/clumsy/

Other

4.96k stars 488 forks source link

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

Open stephen9358 opened 8 months ago

stephen9358 commented 8 months ago

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

jagt commented 8 months ago

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

stephen9358 commented 8 months ago

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.