[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Markus Armbruster
Subject: Re: [Qemu-devel] patch: add -kbddelay option
Date: Tue, 27 May 2014 08:06:21 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Dave Mielke <address@hidden> writes:

> [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 
>>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.
> I've currently coded it so that there's a new option - -kbddelay. So, we have
> the following two options:
>   (old) -k <language>
>   (new) -kbddelay [duration=]<msecs>
> My question is if this is the best way to do it, or if the two options should
> be merged. For example:
>    -k [language=]<language>,delay=<msecs>
> What do you think?

*If* the feature is deemed useful: better extend -k.  We generally
prefer extensible options based on QemuOpts to proliferating ad hoc
single-purpose options.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]