unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#14909: Subject: 24.3; imag-dired key-bindings not working
@ 2013-07-19 15:26 Dieter Wilhelm, H.
  2013-07-19 20:23 ` Glenn Morris
  0 siblings, 1 reply; 3+ messages in thread
From: Dieter Wilhelm, H. @ 2013-07-19 15:26 UTC (permalink / raw)
  To: 14909

This bug report will be sent to the Bug-GNU-Emacs mailing list
and the GNU bug tracker at debbugs.gnu.org.  Please check that
the From: line contains a valid email address.  After a delay of up
to one day, you should receive an acknowledgment at that address.

Please write in English if possible, as the Emacs maintainers
usually do not have translators for other languages.

Please describe exactly what actions triggered the bug, and
the precise symptoms of the bug.  If you can, give a recipe
starting from `emacs -Q':

Hi,

please open Emacs with -Q and load a directory with image-dired, then
display an original file with RET.  After this basically all the
movement keybindings in the *image-dired* buffer are not working any
longer.

Thanks for tumme

    Dieter

If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
    `bt full' and `xbacktrace'.
For information about debugging Emacs, please read the file
/appl/emacs-24.3/etc/DEBUG.


In GNU Emacs 24.3.1 (x86_64-unknown-linux-gnu, GTK+ Version 2.18.9)
 of 2013-03-19 on sbav106x.vs.de.conti.de
Windowing system distributor `Red Hat, Inc.', version 11.0.11006000
System Description:    Red Hat Enterprise Linux Server release 6.3 (Santiago)

Important settings:
  value of $LANG: en_GB.utf8
  locale-coding-system: utf-8-unix
  default enable-multibyte-characters: t

Major mode: image-dired-thumbnail

Minor modes in effect:
  tooltip-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  buffer-read-only: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
<kp-begin> <kp-begin> <kp-begin> <kp-begin> <kp-up>
<kp-up> <kp-up> <kp-down> <kp-down> <kp-down> <kp-begin>
<kp-begin> <kp-begin> <kp-begin> <kp-begin> <kp-begin>
<kp-begin> <kp-down> <kp-begin> C-n C-p C-p C-p C-p
C-p C-a C-a C-a <up> <up> <up> <left> <left> <left>
<up> <up> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1> <mouse-1>
<mouse-1> <mouse-1> <down-mouse-1> <mouse-1> <down-mouse-4>
<mouse-4> <double-down-mouse-4> <double-mouse-4> <triple-down-mouse-4>
<triple-mouse-4> <triple-down-mouse-4> <triple-mouse-4>
<down-mouse-1> <mouse-1> <down-mouse-1> <mouse-1> <down-mouse-4>
<mouse-4> <down-mouse-4> <mouse-4> <double-down-mouse-4>
<double-mouse-4> <triple-down-mouse-4> <triple-mouse-4>
<down-mouse-4> <mouse-4> <down-mouse-1> <mouse-1> <down-mouse-4>
<mouse-4> <double-down-mouse-4> <double-mouse-4> <triple-down-mouse-4>
<triple-mouse-4> <down-mouse-4> <mouse-4> <down-mouse-4>
<mouse-4> <down-mouse-1> <mouse-1> <down-mouse-4> <mouse-4>
<double-down-mouse-4> <double-mouse-4> <triple-down-mouse-4>
<triple-mouse-4> <triple-down-mouse-4> <triple-mouse-4>
<triple-down-mouse-4> <triple-mouse-4> <triple-down-mouse-4>
<triple-mouse-4> <triple-down-mouse-4> <triple-mouse-4>
C-b C-b C-a C-p C-p C-p C-n C-n C-p C-f C-f C-h m C-M-v
C-v C-v C-v C-v C-v C-v C-v C-v C-v M-x M-p <return>
<return> y C-f C-f C-f C-f C-f C-f C-f C-f C-f C-f
C-f C-f C-f C-f C-f C-f C-f C-f C-f C-b C-b C-b C-b
C-b C-b C-b C-b C-b C-b C-f C-f C-f C-f C-f <return>
C-b C-b C-b C-f C-f C-b C-b C-b C-b C-b C-b <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<help-echo> <help-echo> <help-echo> <help-echo> <help-echo>
<menu-bar> <help-menu> <send-emacs-bug-report>

Recent messages:

zkv4d045g0: IMG_0968.JPG ():

zkv4d045g0: IMG_0969.JPG ():

zkv4d045g0: IMG_0970.JPG ():

zkv4d045g0: IMG_0971.JPG ():

zkv4d045g0: IMG_0972.JPG ():
image-dired-forward-image: At last image [2 times]

Load-path shadows:
None found.

Features:
(shadow sort gnus-util mail-extr emacsbug message rfc822 mml mml-sec
mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils
mailheader sendmail rfc2047 rfc2045 ietf-drums mm-util mail-prsvr
mail-utils help-fns image-file image-dired format-spec novice dired
help-mode easymenu time-date tooltip ediff-hook vc-hooks lisp-float-type
mwheel x-win x-dnd tool-bar dnd fontset image regexp-opt fringe
tabulated-list newcomment lisp-mode register page menu-bar rfn-eshadow
timer select scroll-bar mouse jit-lock font-lock syntax facemenu
font-core frame cham georgian utf-8-lang misc-lang vietnamese tibetan
thai tai-viet lao korean japanese hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese case-table epa-hook
jka-cmpr-hook help simple abbrev minibuffer loaddefs button faces
cus-face macroexp files text-properties overlay sha1 md5 base64 format
env code-pages mule custom widget hashtable-print-readable backquote
make-network-process dynamic-setting font-render-setting move-toolbar
gtk x-toolkit x multi-tty emacs)

--
    Best wishes

    H. Dieter Wilhelm
    Schwalbach, Germany





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#14909: Subject: 24.3; imag-dired key-bindings not working
  2013-07-19 15:26 bug#14909: Subject: 24.3; imag-dired key-bindings not working Dieter Wilhelm, H.
@ 2013-07-19 20:23 ` Glenn Morris
  2013-07-20  8:32   ` Stefan Monnier
  0 siblings, 1 reply; 3+ messages in thread
From: Glenn Morris @ 2013-07-19 20:23 UTC (permalink / raw)
  To: dieter; +Cc: 14909


Thanks, this should fix it:

*** lisp/image-dired.el	2013-01-01 09:11:05 +0000
--- lisp/image-dired.el	2013-07-19 20:21:46 +0000
***************
*** 1035,1049 ****
  See documentation for `image-dired-toggle-movement-tracking'.
  Interactive use only useful if `image-dired-track-movement' is nil."
    (interactive)
!   (let ((old-buf (current-buffer))
          (dired-buf (image-dired-associated-dired-buffer))
!         (file-name (image-dired-original-file-name)))
      (when (and (buffer-live-p dired-buf) file-name)
        (set-buffer dired-buf)
        (if (not (dired-goto-file file-name))
            (message "Could not track file")
!         (set-window-point
!          (image-dired-get-buffer-window dired-buf) (point)))
        (set-buffer old-buf))))
  
  (defun image-dired-toggle-movement-tracking ()
--- 1035,1049 ----
  See documentation for `image-dired-toggle-movement-tracking'.
  Interactive use only useful if `image-dired-track-movement' is nil."
    (interactive)
!   (let* ((old-buf (current-buffer))
           (dired-buf (image-dired-associated-dired-buffer))
!          (file-name (image-dired-original-file-name))
!          (window (image-dired-get-buffer-window dired-buf)))
      (when (and (buffer-live-p dired-buf) file-name)
        (set-buffer dired-buf)
        (if (not (dired-goto-file file-name))
            (message "Could not track file")
!         (if window (set-window-point window (point))))
        (set-buffer old-buf))))
  
  (defun image-dired-toggle-movement-tracking ()






^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#14909: Subject: 24.3; imag-dired key-bindings not working
  2013-07-19 20:23 ` Glenn Morris
@ 2013-07-20  8:32   ` Stefan Monnier
  0 siblings, 0 replies; 3+ messages in thread
From: Stefan Monnier @ 2013-07-20  8:32 UTC (permalink / raw)
  To: Glenn Morris; +Cc: dieter, 14909

> !   (let* ((old-buf (current-buffer))
>            (dired-buf (image-dired-associated-dired-buffer))
> !          (file-name (image-dired-original-file-name))
> !          (window (image-dired-get-buffer-window dired-buf)))
>       (when (and (buffer-live-p dired-buf) file-name)
>         (set-buffer dired-buf)
>         (if (not (dired-goto-file file-name))
>             (message "Could not track file")
> !         (if window (set-window-point window (point))))
>         (set-buffer old-buf))))

I see this code could use with-current-buffer.


        Stefan





^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-07-20  8:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-19 15:26 bug#14909: Subject: 24.3; imag-dired key-bindings not working Dieter Wilhelm, H.
2013-07-19 20:23 ` Glenn Morris
2013-07-20  8:32   ` Stefan Monnier

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).