You know that you have commit access to psvn.el, right?
Just making sure, since couldn't tell from your mail,
-Karl
Kalle Olavi Niemitalo <kon@iki.fi> writes:
> Some bindings seem to be wrong in svn-status-mode-map of psvn.el r9588.
>
> (kbd "<return>")
>   This binding works in GNU Emacs 21.3 in X11.  However, when I
>   press Enter in a TTY, I just get an error:
>     Buffer is read-only: #<buffer *svn-status*>
>   And C-h c RET reveals:
>     RET runs the command newline
>   The same happens in the CVS version of GNU Emacs.
>   I think this should be changed to (kbd "RET"), which
>   works in at least these environments:
>   - GNU Emacs CVS 2004-04-23 in X11
>   - GNU Emacs CVS 2004-04-23 in TTY
>   - GNU Emacs 21.3 in X11
>   - GNU Emacs 21.3 in TTY
>   In XEmacs, both (kbd "<return>") and (kbd "RET") expand to
>   [return], so the change should be safe there.  I couldn't
>   test it properly as my XEmacs installation is currently messed
>   up and fails to load xemacs-base.
> 
> "\M-DEL"
>   This means Meta-D followed by E and L.  I doubt that was the
>   intention.  The alternative (kbd "M-DEL") seems to behave
>   somewhat erratically:
>   - GNU Emacs 21.3 in X11:
>     binds Esc Backspace (<escape> <backspace> translated to ESC DEL)
>     binds Alt+Backspace (<M-backspace> translated to M-DEL)
>     doesn't bind Esc Delete (<escape> <delete> translated to ESC C-d)
>     binds Alt+Delete (<M-delete> translated to M-DEL)
>   - GNU Emacs CVS 2004-04-23 in X11:
>     binds Esc Backspace (<escape> <backspace> translated to M-DEL)
>     binds Alt+Backspace (<M-backspace> translated to M-DEL)
>     doesn't bind Esc Delete (<escape> <delete> translated to C-M-d)
>     binds Alt+Delete (<M-delete> translated to M-DEL)
>   - GNU Emacs CVS 2004-04-23 in TTY (gnome-terminal + screen):
>     binds Esc Backspace (ESC DEL translated to M-DEL)
>     binds Alt+Backspace (ESC DEL translated to M-DEL)
>     doesn't bind Esc Delete (ESC ESC [ 3 ~ translated to ESC <deletechar>)
>     doesn't bind Alt+Delete (ESC [ 3 ; 3 ~ translated to M-[ 3 ; 3 ~)
>   - XEmacs 21.4 (patch 15) in TTY (gnome-terminal + screen):
>     doesn't bind Esc Backspace
>     doesn't bind Alt+Backspace
>     doesn't bind Esc Delete
>     doesn't bind Alt+Delete
>   - XEmacs 21.4 (patch 15) in X11:
>     doesn't bind Esc Backspace
>     doesn't bind Alt+Backspace
>     binds Esc Delete
>     binds Alt+Delete
>   It doesn't seem like there is a solution that is correct for
>   both Emacs and XEmacs.  However, binding both (kbd "M-DEL") and
>   (kbd "M-<backspace>") might be good enough.  Dired uses
>   "\M-\C-?" in both Emacs and XEmacs; I didn't test what that does.
> 
>   Additionally, this binding refers to the function
>   svn-status-unset-all-user-mark, which is not defined.
>   svn-status-unset-all-usermarks is probably what was meant.
> 
> [(backspace)]
>   This binding is not accessible with GNU Emacs CVS 2004-04-23 in
>   TTY.  (kbd "DEL") would do the job there, but in XEmacs, it
>   would affect Delete instead.  Dired uses "\177" in Emacs and
>   doesn't seem to have this feature in XEmacs.
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon May  3 16:59:27 2004