From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54701) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WoWDG-0005bF-Nw for qemu-devel@nongnu.org; Sun, 25 May 2014 07:03:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WoWDB-0001Mw-DO for qemu-devel@nongnu.org; Sun, 25 May 2014 07:03:34 -0400 Date: Sun, 25 May 2014 07:03:24 -0400 From: Dave Mielke Message-ID: <20140525110324.GJ1904@beta.private.mielke.cc> References: <20140524232950.GH1904@beta.private.mielke.cc> <20140525012151.GI1904@beta.private.mielke.cc> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] patch: add delay= suboption to -display curses List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Trivial , QEMU Developers [quoted lines by Peter Maydell on 2014/05/25 at 10:11 +0100] >Ah, I see. Still, I think it makes more sense for the queue and delay >to be in the common key handling code, not in the curses frontend >specifically. Yes, you're right. While the curses UI is especially vulnerable to the problem, others could be as well, especially if they're driven by a software "user" rather than by a human being. Also, who's to say that there won't be a new UI in the future which won't run into the same problem. I'll move the code and submit a new patch. 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://Mielke.cc/bible/ EMail: dave@mielke.cc | Canada K2A 1H7 | http://FamilyRadio.com/