emacs-bug-tracker
[Top][All Lists]
Advanced

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

[debbugs-tracker] bug#16438: closed (24.3.50; `C-x TAB right right' fail


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#16438: closed (24.3.50; `C-x TAB right right' fails with error (mark-inactive))
Date: Wed, 15 Jan 2014 08:29:02 +0000

Your message dated Wed, 15 Jan 2014 10:23:11 +0200
with message-id <address@hidden>
and subject line Re: bug#16438: 24.3.50; `C-x TAB right right' fails with error 
(mark-inactive)
has caused the debbugs.gnu.org bug report #16438,
regarding 24.3.50; `C-x TAB right right' fails with error (mark-inactive)
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
16438: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=16438
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 24.3.50; `C-x TAB right right' fails with error (mark-inactive) Date: Tue, 14 Jan 2014 10:14:07 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (x86_64-pc-linux-gnu)
1. emacs -Q
2. Paste to *scratch*:

(custom-set-variables
 '(mark-even-if-inactive nil))

3. Eval it.
4. Put point at its opening paren and type:

C-M-SPC    - selects the whole expression
C-x TAB    - activates a transient indentation mode
right      - adjusts indentation 1 position to the right
right      - fails with the error "The mark is not active now"
             because the previous `right' key deactivated the region

One way to fix this is not to deactivate the mark in this transient mode:

=== modified file 'lisp/indent.el'
--- lisp/indent.el      2014-01-13 05:03:31 +0000
+++ lisp/indent.el      2014-01-14 08:07:55 +0000
@@ -214,7 +214,9 @@ (defun indent-rigidly (start end arg &op
               (indent-to (max 0 (+ indent (prefix-numeric-value arg))) 0))
           (delete-region (point) (progn (skip-chars-forward " \t") (point))))
         (forward-line 1))
-      (move-marker end nil))))
+      (move-marker end nil)
+      (when (eq overriding-terminal-local-map indent-rigidly-map)
+       (setq deactivate-mark nil)))))


But the condition of checking for the transient mode doesn't work
because overriding-terminal-local-map never equals to indent-rigidly-map.

 indent-rigidly-map:
  (keymap (S-right . indent-rigidly-right-to-tab-stop) (S-left . 
indent-rigidly-left-to-tab-stop) (right . indent-rigidly-right) (left . 
indent-rigidly-left))

 overriding-terminal-local-map:
  (keymap (keymap (S-right . indent-rigidly-right-to-tab-stop) (S-left . 
indent-rigidly-left-to-tab-stop) (right . indent-rigidly-right) (left . 
indent-rigidly-left))
          add-keymap-witness)

There is some `add-keymap-witness' at the end of `overriding-terminal-local-map'
when the transient indentation mode is active.  And I don't know another way
to check if `overriding-terminal-local-map' is active.



--- End Message ---
--- Begin Message --- Subject: Re: bug#16438: 24.3.50; `C-x TAB right right' fails with error (mark-inactive) Date: Wed, 15 Jan 2014 10:23:11 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (x86_64-pc-linux-gnu)
> But the condition of checking for the transient mode doesn't work
> because overriding-terminal-local-map never equals to indent-rigidly-map.
>
>  indent-rigidly-map:
>   (keymap (S-right . indent-rigidly-right-to-tab-stop) (S-left . 
> indent-rigidly-left-to-tab-stop) (right . indent-rigidly-right) (left . 
> indent-rigidly-left))
>
>  overriding-terminal-local-map:
>   (keymap (keymap (S-right . indent-rigidly-right-to-tab-stop) (S-left . 
> indent-rigidly-left-to-tab-stop) (right . indent-rigidly-right) (left . 
> indent-rigidly-left))
>           add-keymap-witness)

I fixed this by using (eq (cadr overriding-terminal-local-map) 
indent-rigidly-map)


--- End Message ---

reply via email to

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