add methods for removing hotkeys and hotstrings #215
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #213
Adds new methods for removing hotstrings/hotkeys:
remove_hotkey
to remove a hotkey by keynameclear_hotkeys
to remove all hotkeysremove_hotstring
to remove a hotstring by trigger sequenceclear_hotstrings
to clear all hotstringsLike adding hotkeys or hotstrings, using these methods while the hotkey process is already running causes a restart of the hotkey process.
During testing, a bug was discovered in which a race condition existed if you tried to stop hotkeys too soon after starting them. The dispatcher thread is responsible for creating the subprocess, but because this happens asynchronously it's not guaranteed that the subprocess has started by the time
stop
is called.As a temporary fix for this bug, the stop process will wait up to 1 second for the process to exist.