From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33922) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VcBWs-0005WT-4Z for qemu-devel@nongnu.org; Fri, 01 Nov 2013 06:00:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VcBWm-0005le-3Q for qemu-devel@nongnu.org; Fri, 01 Nov 2013 06:00:34 -0400 Received: from isrv.corpit.ru ([86.62.121.231]:60466) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VcBWl-0005lU-RQ for qemu-devel@nongnu.org; Fri, 01 Nov 2013 06:00:28 -0400 Message-ID: <52737BBA.4060109@msgid.tls.msk.ru> Date: Fri, 01 Nov 2013 14:00:26 +0400 From: Michael Tokarev MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [v2, 4/4] qemu-char: add missing characters used in keymaps List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Krupa Cc: qemu-devel@nongnu.org, Anthony Liguori 16.10.2013 16:40, Jan Krupa wrote: > This patch adds all missing characters used in regional keymap > files which already exist in QEMU. I checked for the missing > characters by going through all of the keymaps and matching that > with records in vnc_keysym.h. If the key wasn't found I looked > it up in libxkbcommon library [1]. If I understood it correctly > this is also the same place where most of the keymaps were > exported from according to the comment on the first line in those > files. I was able to find all symbols except "quotebl" used > in Netherland keymap. > > I tested this update with Czech keyboard by myself. I also asked > Matej Serc to test Slovenian keyboard layout - he reported problems > with it few days ago on this mailing list. Both layouts seems > to work fine. I wasn't able to test the remaining layouts but > since this change doesn't modify any existing symbols, just adds > new ones, I don't expect any sideeffects. Yes, this patch, while large, is trivial and without any side effects. Except of one question. Where we add these entries? Should we maybe sort the table somehow, or introduce some more groups of chars? (I'm fine with applying this to qemu-trivial as-is). Thanks, /mjt