bug#35533: 26.2; Weird <i> and <I> keybindings in an (insert-table) environment.

bug#35533: 26.2; Weird <i> and <I> keybindings in an (insert-table) environment.

[Vladimir Nikishkin]

The bug (or, rather a strange behaviour), can be reproduced in the
following way:
1)Create a new buffer.
2)Insert a text table by calling M-x insert-table RET
3)Put the point in the first cell.
4)Type: C-h w table-forward-cell RET

You will see a keybinding called 'C-c C-c <i>'.
You can see similar weird keybindings for other table-* commands.

Those actual keys are nowhere to be found, and no-one on the
#emacs channel on Freenode knows what they are.

It would be nice to either document those keys. Or, perhaps,
it's a bug in Emacs.

This behaviour has been reproduced by friendly people on
#emacs with different setups.


In GNU Emacs 26.2 (build 1, x86_64-slackware-linux-gnu, GTK+ Version 3.24.8)
 of 2019-04-13 built on hive64.slackware.lan
Windowing system distributor 'The X.Org Foundation', version 11.0.12004000
Recent messages:

Making completion list...
(New file)
Wrote /home/lockywolf/DevLinux/linuxbugs-lwf-for-generalized-bugfixes-and-small-scripts/cheatsheets/slackware-isms.tbl
Use +,-,0 for further adjustment
C-x C-g is undefined
previous-line: Beginning of buffer
Saving file /home/lockywolf/DevLinux/linuxbugs-lwf-for-generalized-bugfixes-and-small-scripts/cheatsheets/slackware-isms.tbl...
Wrote /home/lockywolf/DevLinux/linuxbugs-lwf-for-generalized-bugfixes-and-small-scripts/cheatsheets/slackware-isms.tbl
a is undefined

Configured using:
 'configure --prefix=/usr --sysconfdir=/etc --localstatedir=/var
 --program-prefix= --program-suffix= --mandir=/usr/man
 --infodir=/usr/info --without-gconf --without-gsettings --with-x
 --with-x-toolkit=gtk3 --build=x86_64-slackware-linux 'CFLAGS=-O2
 -fPIC''

Configured features:
XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GLIB NOTIFY ACL
GNUTLS LIBXML2 FREETYPE XFT ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 XDBE XIM
THREADS LCMS2

Important settings:
  value of $LC_COLLATE: C
  value of $LC_TIME: en_GB.UTF-8
  value of $LANG: en_US.UTF-8
  value of $XMODIFIERS: @im=SCIM
  locale-coding-system: utf-8-unix

Major mode: Info

Minor modes in effect:
  diff-auto-refine-mode: t
  global-flycheck-mode: t
  global-auto-revert-mode: t
  electric-pair-mode: t
  global-display-line-numbers-mode: t
  display-line-numbers-mode: t
  global-whitespace-mode: t
  global-highlight-changes-mode: t
  highlight-changes-visible-mode: t
  global-hi-lock-mode: t
  hi-lock-mode: t
  desktop-save-mode: t
  tooltip-mode: t
  global-eldoc-mode: t
  electric-layout-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  global-prettify-symbols-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
  size-indication-mode: t
  column-number-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc puny rfc822 mml mml-sec epa
derived epg gnus-util rmail rmail-loaddefs mm-decode mm-bodies mm-encode
mail-parse rfc2231 mailabbrev gmm-utils mailheader jka-compr face-remap
two-column iso-transl eieio-opt speedbar sb-image ezimage dframe pcase
warnings time-stamp rect tabify table vc-git diff-mode easy-mmode
misearch multi-isearch image-dired image-mode format-spec help-fns
radix-tree dired-aux dired dired-loaddefs flycheck-plantuml
plantuml-mode flycheck cl-extra json map find-func help-mode rx subr-x
dash smtpmail sendmail rfc2047 rfc2045 ietf-drums mm-util mail-prsvr
mail-utils starttls autorevert filenotify elec-pair keywiz gamegrid
diary-lib diary-loaddefs cal-menu calendar cal-loaddefs totd s
display-line-numbers disp-table whitespace hilit-chg wid-edit hi-lock
mic-paren ffap thingatpt edmacro kmacro savehist desktop frameset geiser
manoj-dark-theme finder-inf info package easymenu epg-config
url-handlers url-parse auth-source cl-seq eieio eieio-core cl-macs
eieio-loaddefs password-cache url-vars seq byte-opt gv bytecomp
byte-compile cconv cl-loaddefs cl-lib time-date mule-util tooltip eldoc
electric uniquify ediff-hook vc-hooks lisp-float-type mwheel term/x-win
x-win term/common-win x-dnd tool-bar dnd fontset image regexp-opt fringe
tabulated-list replace newcomment text-mode elisp-mode lisp-mode
prog-mode register page menu-bar rfn-eshadow isearch timer select
scroll-bar mouse jit-lock font-lock syntax facemenu font-core
term/tty-colors frame cl-generic cham georgian utf-8-lang misc-lang
vietnamese tibetan thai tai-viet lao korean japanese eucjp-ms cp51932
hebrew greek romanian slovak czech european ethiopic indian cyrillic
chinese composite charscript charprop case-table epa-hook jka-cmpr-hook
help simple abbrev obarray minibuffer cl-preloaded nadvice loaddefs
button faces cus-face macroexp files text-properties overlay sha1 md5
base64 format env code-pages mule custom widget hashtable-print-readable
backquote threads dbusbind inotify lcms2 dynamic-setting
font-render-setting move-toolbar gtk x-toolkit x multi-tty
make-network-process emacs)

Memory information:
((conses 16 221709 20830)
 (symbols 48 29294 2)
 (miscs 40 1662 1774)
 (strings 32 60043 5584)
 (string-bytes 1 1730553)
 (vectors 16 32605)
 (vector-slots 8 1284818 146948)
 (floats 8 122 423)
 (intervals 56 2320 106)
 (buffers 992 25))

-- 
--
Vladimir Nikishkin (MiEr, lockywolf)

bug#35533: 26.2; Weird <i> and <I> keybindings in an (insert-table) environment.

[Mauro Aranda]

[-- Attachment #1.1: Type: text/plain, Size: 969 bytes --]

Hello.

> The bug (or, rather a strange behaviour), can be reproduced in the
> following way:
> 1)Create a new buffer.
> 2)Insert a text table by calling M-x insert-table RET
> 3)Put the point in the first cell.
> 4)Type: C-h w table-forward-cell RET

Just for clarity, the second step is to type: M-x table-insert RET
Then, of course, hit RET to accept all of the defaults.

> You will see a keybinding called 'C-c C-c <i>'.
> You can see similar weird keybindings for other table-* commands.
>
> Those actual keys are nowhere to be found, and no-one on the
> #emacs channel on Freenode knows what they are.
>
> It would be nice to either document those keys. Or, perhaps,
> it's a bug in Emacs.
>

I believe it's a bug.
'C-h v table-cell-map RET' shows at the end that the bindings are to
'j', 'm', 'I' and 'i', and not to their printed representations.

I attach a patch that adds a missing '?' to all those characters, in
table-cell-bindings.

Best regards,
Mauro.

[-- Attachment #1.2: Type: text/html, Size: 1225 bytes --]

[-- Attachment #2: 0001-Fix-read-syntax-for-some-characters.patch --]
[-- Type: text/x-patch, Size: 1650 bytes --]

From 3d01c3e2f5db34dcfbfda1849e985526da309b5b Mon Sep 17 00:00:00 2001
From: Mauro Aranda <maurooaranda@gmail.com>
Date: Thu, 2 May 2019 09:51:01 -0300
Subject: [PATCH] Fix read syntax for some characters

* lisp/textmodes/table.el (table-cell-bindings): Add missing '?', to
  use the proper read syntax.  (Bug#35533)
---
 lisp/textmodes/table.el | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/lisp/textmodes/table.el b/lisp/textmodes/table.el
index bed8b2f..f27c705 100644
--- a/lisp/textmodes/table.el
+++ b/lisp/textmodes/table.el
@@ -930,16 +930,16 @@ table-target-history
 ;; refill the table cache.  If the command were not listed fast
 ;; typing can cause unwanted cache refill.
 (defconst table-cell-bindings
-  '(([(control i)]	. table-forward-cell)
-    ([(control I)]	. table-backward-cell)
+  '(([(control ?i)]	. table-forward-cell)
+    ([(control ?I)]	. table-backward-cell)
     ([tab]		. table-forward-cell)
     ([(shift backtab)]	. table-backward-cell) ; for HPUX console keyboard
     ([(shift iso-lefttab)]    . table-backward-cell) ; shift-tab on a microsoft natural keyboard and redhat linux
     ([(shift tab)]	. table-backward-cell)
     ([backtab]          . table-backward-cell) ; for terminals (e.g., xterm)
     ([return]		. *table--cell-newline)
-    ([(control m)]	. *table--cell-newline)
-    ([(control j)]	. *table--cell-newline-and-indent)
+    ([(control ?m)]	. *table--cell-newline)
+    ([(control ?j)]	. *table--cell-newline-and-indent)
     ([mouse-3]		. *table--present-cell-popup-menu)
     ([(control ?>)]	. table-widen-cell)
     ([(control ?<)]	. table-narrow-cell)
-- 
2.7.4

bug#35533: 26.2; Weird <i> and <I> keybindings in an (insert-table) environment.

[Noam Postavsky]

severity 35533 minor
tags 35533 fixed
close 35533 27.1
quit

Mauro Aranda <maurooaranda@gmail.com> writes:

> I believe it's a bug.
> 'C-h v table-cell-map RET' shows at the end that the bindings are to
> 'j', 'm', 'I' and 'i', and not to their printed representations.

I think it's generally allowed to use the symbol instead of characters
(it's an XEmacs thing).  But some other code in table.el didn't account
for that (so in combination it's a bug).

> I attach a patch that adds a missing '?' to all those characters, in
> table-cell-bindings.

I adjusted the commit message a bit, and pushed to master.

40b3dcb7f7 2019-05-11T10:19:24-04:00 "Use characters, not symbols for table-cell-bindings"
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=40b3dcb7f7be92f0471e7a503ae8598c72100146