emacs-devel
[Top][All Lists]
Advanced

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

Re: Issues with quail.el


From: Amit Ramon
Subject: Re: Issues with quail.el
Date: Thu, 17 May 2018 21:57:36 +0300

Filipp Gunbin <address@hidden> [2018-05-17 18:44 +0300]:

On 17/05/2018 21:31 +0900, K. Handa wrote:

Here's the patch to fix it.  Could you please try it?
...
+             ;; This location not available in the current keyboard
+             ;; layout.  Check if the location is used to substitute
+             ;; for the other location of the standard layout.
+             (if (setq i (cdr (rassq i quail-keyboard-layout-substitution)))
+                 (aref quail-keyboard-layout i)

Cannot understand this part - it seems to look up alist element by cdr
(i), and then set i to the same value.  Should there be `(car (rassq..'
instead?

I agree with Filipp -- I also think it should be set to '(car (rassq
...'.

(this is a 'reverse' translation -- the code in the existing
translation sets i to '(cdr (assq ...', here it should be the
'mirrored image' of it.)

--- Amit



reply via email to

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