Fix app freezing (fixes: #93)

[Luke265]

When running multiple apps with iohook, mouse/key inputs becomes very laggy, sometimes even completely frozen.

Description

I am not a c++ programmer, and I am not exactly sure what I did here.

Motivation and Context

See #93

How Has This Been Tested?

With the patch iohook installs locally on Windows 10.

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[Djiit]

I'm closing this for now. Feel free to reopen it if you want to.

[sentialx]

@Djiit Why it has been closed?

[Wizek]

Hey @Luke265, do you understand why this PR was closed? Was there some issue with it that might not be apparent for me and sentialx? Or was its closure confusing for you too?

[Luke265]

@Wizek Sorry, I don't know why this PR was closed. I had no issues, so far, using iohook with this PR.

[Djiit]

Hi guys, Sorry if I closed this too soon. I thought it was not necessary and it didn't have any activity.

[Wizek]

Thanks @Djiit for reopening!

So, in the interest of moving forward with this issue getting fixed, I tried out this patch. However, the mouse-freezing is still present when I ran this test:

const {exec} = require('child_process');
const iohook = require('iohook');

iohook.start();

if (process.argv.length === 2) {
	console.log('Launched main');
	iohook.on('keydown', (e) => {
		console.log('Main', e.rawcode);
	});
	exec([`"${process.argv[0]}"`, process.argv[1], '-t'].join(' ')).stdout.pipe(process.stdout);
} else {
	console.log('Launched child');
	iohook.on('keydown', (e) => {
		console.log('Child', e.rawcode);
	});
}

Even when I set

{
  "devDependencies": {
    "iohook": "github:Luke265/iohook#patch-1"
  }
}

You can find my repro-repo over here: https://github.com/Wizek/iohook-test. Just

git clone https://github.com/Wizek/iohook-test
cd iohook-test
npm i
node .

Could you @Luke265 do the above test and tell me if your mouse lags after that code runs on Windows, preferably Windows 10?

Also, @Djiit, could you also do this just so you can see how bad this bug is?

[Wizek]

Did you manage to try the repro steps @Luke265 and @Djiit? It's really as simple as:

git clone https://github.com/Wizek/iohook-test
cd iohook-test
npm i
node .

At least I could get some confirmation that others are also experiencing the same issue on Windows 10 as I.

[Wizek]

As this is still not fixed, I'm switching over to something that seems to work better:

https://www.npmjs.com/package/gkm

[zxc23]

I also get the freezing when running multiple iohook apps at the same time. I haven't been able to try the PR but I support the effort.

[WilixLead]

Hello guys. @Luke265 @Wizek if any one have time to check this PR #294 - will be cool. Looks like it exactly the same issue for windows systems

[WilixLead]

Closing it regarding #294