Re: [Qemu-devel] [PATCH v2 0/4] input: add support for kbd delays

2014-06-02 Thread Dave Mielke
have a "length of delay" argument, which then calls the current function with the desired value. These are, of course, cosmetic suggestions. The code as it now stands dodes work. -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Phone: 1-613-726-001

Re: [Qemu-devel] [PATCH 0/3] input: add support for kbd delays

2014-06-02 Thread Dave Mielke
[quoted lines by Gerd Hoffmann on 2014/06/02 at 11:31 +0200] Hi: >Does that also happen with the "sendkey ctrl-u" monitor command? Yes, it does. This test was with the newest (10ms) delay patch. -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Pho

Re: [Qemu-devel] [PATCH 0/3] input: add support for kbd delays

2014-06-02 Thread Dave Mielke
temporarily dead keyboard. There are still problems, though. When I hold the shift key and type a letter, I get a doubled lowercase letter rather than a single uppercase letter. Also, I still get doubled lowercase letters when holding the control key. -- Dave Mielke | 2213 Fox Crescen

Re: [Qemu-devel] [PATCH 0/3] input: add support for kbd delays

2014-06-02 Thread Dave Mielke
g worked exactly as it should, stopping as soon as I let go of the key. Backspacing till they were all gone also left me with a live keyboard immediately. I did the control letter test and they did what they should do instead of just double echoing the non-control letter. -- Dave Mielke |

Re: [Qemu-devel] [PATCH 0/3] input: add support for kbd delays

2014-06-02 Thread Dave Mielke
Something else I noticed was that typing a control character would result in a doubling of that character. For example, control "u" resulted in "UU". Now to go figure out how to stop those "j"s again. I left them repeating while I wrote this. :-) -- Dave Mielke

Re: [Qemu-devel] [PATCH 0/3] input: add support for kbd delays

2014-05-31 Thread Dave Mielke
until the next key press/release delay, thus effectively leaving the keys pressed. I've attached the latest verson of my patch (qemu-kbddelay-2.patch) to this message in case you're interested in considering it. It merges the delay option into the -k option, so we'd have this:

Re: [Qemu-devel] [Qemu-trivial] patch: add delay= suboption to -display curses

2014-05-27 Thread Dave Mielke
that a key press and release go in together. That give us the following situation: 1) Peek the DOS input buffer. It isn't empty so proceed to the next step. 2) No key is pressed so return "no input". I really do think, therefore, that the only way to deal with this is to slo

Re: [Qemu-devel] [Qemu-trivial] patch: add delay= suboption to -display curses

2014-05-27 Thread Dave Mielke
k [language=]language[,delay=msecs]? Which would be the better approach? -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Phone: 1-613-726-0014 | Ottawa, Ontario | http://Mielke.cc/bible/ EMail: d...@mielke.cc | Canada K2A 1H7 | http://FamilyRadio.com/

Re: [Qemu-devel] patch: add -kbddelay option

2014-05-26 Thread Dave Mielke
[quoted lines by Dave Mielke on 2014/05/26 at 08:59 -0400] I'm sorry for quoting myself, but I'm doing it for context since I have a question: >This patch, attached as qemu-kbddelay-1.patch, is a rework of the former curses >UI patch so that the delay applies to key events i

Re: [Qemu-devel] [Qemu-trivial] patch: add delay= suboption to -display curses

2014-05-26 Thread Dave Mielke
; any effect? That gives me: could not revert 2858ab0... ps2: set ps/2 output buffer size as the same as kernel -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Phone: 1-613-726-0014 | Ottawa, Ontario | http://Mielke.cc/bible/ EMail: d...@mielke.cc |

Re: [Qemu-devel] patch: add -kbddelay option

2014-05-26 Thread Dave Mielke
ormer curses UI patch so that the delay applies to key events in general. A new option, -kbddelay [duration=]msecs, controls it. As before, the default is 0, meaning no delay, in which case the timer and queue aren't used thus retaining the original behaviour. -- Dave Mielke |

Re: [Qemu-devel] [Qemu-trivial] patch: add delay= suboption to -display curses

2014-05-25 Thread Dave Mielke
rsonal opinion is that implementing it now would be a reasonable thing to do, especially since the current situation is that no one is being asked to figure it out, code it, etc. -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Phone: 1-613-726-0014 | Ottawa, Ontario

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-25 Thread Dave Mielke
ile that needs to be updated insofar as documentation is concerned. Is that correct? -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Phone: 1-613-726-0014 | Ottawa, Ontario | http://Mielke.cc/bible/ EMail: d...@mielke.cc | Canada K2A 1H7 | http://FamilyRadio.com/

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-25 Thread Dave Mielke
h. Given that this'll require a brand new option, could you please let me know all the places that'll need updating insofar as documentation is concerned? -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Phone: 1-613-726-0014 | Ottawa, Ontario | http

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-24 Thread Dave Mielke
up to four modifiers (shift, control, alt, altgr). Another reason is that it's not a problem for a lot of applications. I happened to hit it in an application, and, therefore, coded a fix for it. -- Dave Mielke | 2213 Fox Crescent | The Bible is the very Word of God. Phone: 1-6

[Qemu-devel] patch: add delay= suboption to -display curses

2014-05-24 Thread Dave Mielke
n't specified, or if "delay=0" is specified, then the timer isn't used, thus making the default be the original behaviour. The "=" operand is optional - if it isn't specified then 20 is assumed. -- Dave Mielke | 2213 Fox Crescent | The Bible is the