Path: utzoo!utgpu!jarvis.csri.toronto.edu!mailrus!tut.cis.ohio-state.edu!cs.utexas.edu!uunet!auspex!guy
From: guy@auspex.auspex.com (Guy Harris)
Newsgroups: comp.soft-sys.andrew
Subject: Errors in "eq" key bindings
Message-ID: <2562@auspex.auspex.com>
Date: 23 Oct 89 22:39:13 GMT
Reply-To: guy@auspex.auspex.com (Guy Harris)
Organization: Auspex Systems, Santa Clara
Lines: 49

(BTW, my other postings pertain to the beta version of Andrew,
patchlevel 4, as does this one; sorry for not noting that.)

A couple of the keybindings for "eq" are arguably wrong:

	1) "eqview-copy" is bound to ^Q rather than <ESC>w; I'd expect
	   it to be bound to <ESC>w since it's a similar operation to
	   the "copy" function in e.g. "text".

	2) "eqview-delete-backward" is, probably, intended to be bound
	   to DEL (^?), but there's a backslash missing from a string
	   in the structure entry that so binds it.

*** eqvcmds.c.dist	Fri Sep 22 07:11:19 1989
--- eqvcmds.c	Tue Oct 17 13:28:12 1989
***************
*** 1164,1170 ****
  
      /* FRONT menu card */
      {"eqview-cut", "\027",0,NULL,0,0,eqview_Cut, "Cut~11"},
!     {"eqview-copy", "\021",0,NULL,0,0,eqview_Copy, "Copy~12"},
      {"eqview-paste", "\031",0,"Paste~10",0,0,eqview_Paste, "Paste~13"},
  
  /*    {"eqview-save", "\030\023",0,"Save~31",0,0,eqview_Save, "Save equation in default file."}, */
--- 1164,1170 ----
  
      /* FRONT menu card */
      {"eqview-cut", "\027",0,NULL,0,0,eqview_Cut, "Cut~11"},
!     {"eqview-copy", "\033w",0,NULL,0,0,eqview_Copy, "Copy~12"},
      {"eqview-paste", "\031",0,"Paste~10",0,0,eqview_Paste, "Paste~13"},
  
  /*    {"eqview-save", "\030\023",0,"Save~31",0,0,eqview_Save, "Save equation in default file."}, */
***************
*** 1191,1197 ****
      /* Other commands */
      {"eqview-delete-forward", "\004",0,NULL,0,0,eqview_DeleteForward, "Delete forward from cursor."},
      {"eqview-delete-backward", "\010",0,NULL,0,0,eqview_DeleteBackward, "Delete backward from cursor."},
!     {"eqview-delete-backward", "177",0},
      /*{"eqview-undelete", "\025",0,NULL,0,0,eqview_UnDelete, "Undelete."}, */
  
      {"eqview-left-brace", "{",0,NULL,0,0,eqview_lbrace, "Insert a left brace."},
--- 1191,1197 ----
      /* Other commands */
      {"eqview-delete-forward", "\004",0,NULL,0,0,eqview_DeleteForward, "Delete forward from cursor."},
      {"eqview-delete-backward", "\010",0,NULL,0,0,eqview_DeleteBackward, "Delete backward from cursor."},
!     {"eqview-delete-backward", "\177",0},
      /*{"eqview-undelete", "\025",0,NULL,0,0,eqview_UnDelete, "Undelete."}, */
  
      {"eqview-left-brace", "{",0,NULL,0,0,eqview_lbrace, "Insert a left brace."},