Skip to content
This repository has been archived by the owner on Mar 12, 2022. It is now read-only.

Hotkey Register/Unregister flooding #53

Open
deMathias opened this issue Mar 28, 2019 · 2 comments
Open

Hotkey Register/Unregister flooding #53

deMathias opened this issue Mar 28, 2019 · 2 comments
Labels
test Needs testing

Comments

@deMathias
Copy link
Collaborator

log gets spammed with
[2019-03-29 0:49:00.695][Error] Attempting to unregister a hotke<y that was never registered.
[2019-03-29 0:49:00.695][Error] HotKeyHandler: RegisterHotKey: Error 1409
[2019-03-29 0:49:00.695][Error] Attempting to unregister a hotke<y that was never registered.
[2019-03-29 0:49:00.696][Error] HotKeyHandler: RegisterHotKey: Error 1409
[2019-03-29 0:49:00.696][Error] Attempting to unregister a hotke<y that was never registered.
[2019-03-29 0:49:00.696][Error] HotKeyHandler: RegisterHotKey: Error 1409
[2019-03-29 0:49:00.696][Error] Attempting to unregister a hotke<y that was never registered.
[2019-03-29 0:49:00.696][Error] HotKeyHandler: RegisterHotKey: Error 1409
[2019-03-29 0:49:00.696][Error] Attempting to unregister a hotke<y that was never registered.
[2019-03-29 0:49:00.697][Error] HotKeyHandler: RegisterHotKey: Error 1409
[2019-03-29 0:49:00.697][Error] Attempting to unregister a hotke<y that was never registered.
[2019-03-29 0:49:00.697][Error] HotKeyHandler: RegisterHotKey: Error 1409
[2019-03-29 0:49:00.697][Error] Attempting to unregister a hotke<y that was never registered.
[2019-03-29 0:49:00.698][Error] HotKeyHandler: RegisterHotKey: Error 1409

even when not pressing hotkey.
@NotCoffee418
Copy link
Owner

Error translates to hotkey already registered.
Age da nog is hed, try to some part of the log just before it happens, unless i fixed it already.

@NotCoffee418 NotCoffee418 added the Breaking-Bug Bug that renders program un-useable label Mar 29, 2019
NotCoffee418 pushed a commit that referenced this issue Mar 30, 2019
#53 Attempted fix for hotkey register/unregister flood
812a59c 
- Change default hotkey for new configs to F4
Since F1-F3 might be boundto fold/checkcall/betraise by default (which also happens to be the only option for default bwin client)
@NotCoffee418
Copy link
Owner

Can't replicate it, ree :/ Implemented something that might fix it.
If the T breaks while the application is running or the flooding happens again, send logs please.

Also keep the default hotkey to something alphabetical in tester config for now to ensure we catch the bug before users do if it returns.

I'll keep the issue open for now.
Close it if we don't heard about it for a while.

@NotCoffee418 NotCoffee418 added test Needs testing and removed Breaking-Bug Bug that renders program un-useable labels Mar 30, 2019
@NotCoffee418 NotCoffee418 changed the title Hotkey issues Hotkey Register/Unregister flooding Mar 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
test Needs testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@NotCoffee418 @deMathias