Vitaliy Margolen wine-patches@kievinfo.com writes:
On winxp both keys toggle on button down not button up. So no need for any extra treatment. Fixes bug 10446.
You still have to handle the case when the state has been toggled outside of Wine, you can't just get rid of the whole thing.
Alexandre Julliard wrote:
Vitaliy Margolen wine-patches@kievinfo.com writes:
On winxp both keys toggle on button down not button up. So no need for any extra treatment. Fixes bug 10446.
You still have to handle the case when the state has been toggled outside of Wine, you can't just get rid of the whole thing.
And how's the current code handles that? By setting few internal thread unsafe constantly being reset flags?
Yes that old code can be just getting rid of without breaking anything.
Vitaliy
Vitaliy Margolen wine-devel@kievinfo.com writes:
Alexandre Julliard wrote:
Vitaliy Margolen wine-patches@kievinfo.com writes:
On winxp both keys toggle on button down not button up. So no need for any extra treatment. Fixes bug 10446.
You still have to handle the case when the state has been toggled outside of Wine, you can't just get rid of the whole thing.
And how's the current code handles that? By setting few internal thread unsafe constantly being reset flags?
No, it does this by simulating a keypress if the state has been changed outside Wine, nothing to do with the global flags. I certainly agree the global flags can be removed, but that doesn't mean you can remove the rest of the code without understanding what it does.