Use defgroup and defcustom.
[bpt/emacs.git] / lisp / hexl.el
index dce544d..b2234d4 100644 (file)
@@ -1,8 +1,10 @@
-;;; hexl-mode.el --- edit a file in a hex dump format using the hexl filter.
+;;; hexl.el --- edit a file in a hex dump format using the hexl filter.
 
-;; Copyright (C) 1989 Free Software Foundation, Inc.
+;; Copyright (C) 1989, 1994 Free Software Foundation, Inc.
 
 ;; Author: Keith Gabryelski <ag@wheaties.ai.mit.edu>
+;; Maintainer: FSF
+;; Keywords: data
 
 ;; This file is part of GNU Emacs.
 
 ;; GNU General Public License for more details.
 
 ;; You should have received a copy of the GNU General Public License
-;; along with GNU Emacs; see the file COPYING.  If not, write to
-;; the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
+;; along with GNU Emacs; see the file COPYING.  If not, write to the
+;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+;; Boston, MA 02111-1307, USA.
 
 ;;; Commentary:
 
+;; This package implements a major mode for editing binary files.  It uses
+;; a program called hexl, supplied with the GNU Emacs distribution, that
+;; can filter a binary into an editable format or from the format back into
+;; binary.  For full instructions, invoke `hexl-mode' on an empty buffer and
+;; do `M-x describe-mode'.
+;;
 ;; This may be useful in your .emacs:
 ;;
 ;;     (autoload 'hexl-find-file "hexl"
 ;; vars here
 ;;
 
-(defvar hexl-program "hexl"
-  "The program that will hexlify and de-hexlify its stdin.
-`hexl-program' will always be concated with `hexl-options'
-and \"-de\" when dehexlfying a buffer.")
-
-(defvar hexl-iso ""
-  "If your emacs can handle ISO characters, this should be set to
-\"-iso\" otherwise it should be \"\".")
+(defgroup hexl nil
+  "Edit a file in a hex dump format using the hexl filter."
+  :group 'data)
 
-(defvar hexl-options (format "-hex %s" hexl-iso)
-  "Options to hexl-program that suit your needs.")
 
-(defvar hexlify-command (format "%s %s" hexl-program hexl-options)
-  "The command to use to hexlify a buffer.  It is the concatination of
-`hexl-program' and `hexl-options'.")
+(defcustom hexl-program "hexl"
+  "The program that will hexlify and dehexlify its stdin.
+`hexl-program' will always be concatenated with `hexl-options'
+and \"-de\" when dehexlifying a buffer."
+  :type 'string
+  :group 'hexl)
 
-(defvar dehexlify-command (format "%s -de %s" hexl-program hexl-options)
-  "The command to use to unhexlify a buffer.  It is the concatination of
-`hexl-program', the option \"-de\", and `hexl-options'.")
+(defcustom hexl-iso ""
+  "If your emacs can handle ISO characters, this should be set to
+\"-iso\" otherwise it should be \"\"."
+  :type 'string
+  :group 'hexl)
+
+(defcustom hexl-options (format "-hex %s" hexl-iso)
+  "Options to hexl-program that suit your needs."
+  :type 'string
+  :group 'hexl)
+
+(defcustom hexlify-command
+  (format "%s%s %s" exec-directory hexl-program hexl-options)
+  "The command to use to hexlify a buffer."
+  :type 'string
+  :group 'hexl)
+
+(defcustom dehexlify-command
+  (format "%s%s -de %s" exec-directory hexl-program hexl-options)
+  "The command to use to unhexlify a buffer."
+  :type 'string
+  :group 'hexl)
 
 (defvar hexl-max-address 0
   "Maximum offset into hexl buffer.")
 
-(defvar help-key "\C-h"
-  "*Key used to invoke electric help.")
-
 (defvar hexl-mode-map nil)
 
+(defvar hexl-mode-old-local-map)
+(defvar hexl-mode-old-mode-name)
+(defvar hexl-mode-old-major-mode)
+(defvar hexl-mode-old-write-contents-hooks)
+(defvar hexl-mode-old-require-final-newline)
+(defvar hexl-mode-old-syntax-table)
+
 ;; routines
 
+(put 'hexl-mode 'mode-class 'special)
+
 ;;;###autoload
 (defun hexl-mode (&optional arg)
   "\\<hexl-mode-map>
-A major mode for editting binary files in hex dump format.
+A major mode for editing binary files in hex dump format.
 
 This function automatically converts a buffer into the hexl format
 using the function `hexlify-buffer'.
@@ -148,8 +178,26 @@ You can use \\[hexl-find-file] to visit a file in hexl-mode.
 \\[describe-bindings] for advanced commands."
   (interactive "p")
   (if (eq major-mode 'hexl-mode)
-      (error "You are already in hexl mode.")
-    (kill-all-local-variables)
+      (error "You are already in hexl mode")
+
+    (let ((modified (buffer-modified-p))
+         (inhibit-read-only t)
+         (original-point (1- (point)))
+         max-address)
+      (and (eobp) (not (bobp))
+          (setq original-point (1- original-point)))
+      (if (not (or (eq arg 1) (not arg)))
+         ;; if no argument then we guess at hexl-max-address
+          (setq max-address (+ (* (/ (1- (buffer-size)) 68) 16) 15))
+        (setq max-address (1- (buffer-size)))
+        (hexlify-buffer)
+        (set-buffer-modified-p modified))
+      (make-local-variable 'hexl-max-address)
+      (setq hexl-max-address max-address)
+      (hexl-goto-address original-point))
+
+    ;; We do not turn off the old major mode; instead we just
+    ;; override most of it.  That way, we can restore it perfectly.
     (make-local-variable 'hexl-mode-old-local-map)
     (setq hexl-mode-old-local-map (current-local-map))
     (use-local-map hexl-mode-map)
@@ -162,55 +210,72 @@ You can use \\[hexl-find-file] to visit a file in hexl-mode.
     (setq hexl-mode-old-major-mode major-mode)
     (setq major-mode 'hexl-mode)
 
+    (make-local-variable 'hexl-mode-old-syntax-table)
+    (setq hexl-mode-old-syntax-table (syntax-table))
+    (set-syntax-table (standard-syntax-table))
+
+    (make-local-variable 'hexl-mode-old-write-contents-hooks)
+    (setq hexl-mode-old-write-contents-hooks write-contents-hooks)
     (make-local-variable 'write-contents-hooks)
-    (setq write-contents-hooks
-         (cons 'hexl-save-buffer write-contents-hooks))
+    (add-hook 'write-contents-hooks 'hexl-save-buffer)
 
-    (let ((modified (buffer-modified-p))
-         (read-only buffer-read-only)
-         (original-point (1- (point))))
-      (if (not (or (eq arg 1) (not arg)))
-;; if no argument then we guess at hexl-max-address
-          (setq hexl-max-address (+ (* (/ (1- (buffer-size)) 68) 16) 15))
-        (setq buffer-read-only nil)
-        (setq hexl-max-address (1- (buffer-size)))
-        (hexlify-buffer)
-        (set-buffer-modified-p modified)
-        (setq buffer-read-only read-only)
-        (hexl-goto-address original-point)))))
+    (make-local-variable 'hexl-mode-old-require-final-newline)
+    (setq hexl-mode-old-require-final-newline require-final-newline)
+    (make-local-variable 'require-final-newline)
+    (setq require-final-newline nil)
+
+    ;; Add hooks to rehexlify or dehexlify on various events.
+    (make-local-hook 'after-revert-hook)
+    (add-hook 'after-revert-hook 'hexl-after-revert-hook nil t)
+
+    (make-local-hook 'change-major-mode-hook)
+    (add-hook 'change-major-mode-hook 'hexl-maybe-dehexlify-buffer nil t))
+  (run-hooks 'hexl-mode-hook))
+
+(defun hexl-after-revert-hook ()
+  (hexlify-buffer)
+  (set-buffer-modified-p nil))
+
+(defvar hexl-in-save-buffer nil)
 
 (defun hexl-save-buffer ()
   "Save a hexl format buffer as binary in visited file if modified."
   (interactive)
-  (set-buffer-modified-p (if (buffer-modified-p)
-                            (save-excursion
-                              (let ((buf (generate-new-buffer " hexl"))
-                                    (name (buffer-name))
-                                    (file-name (buffer-file-name))
-                                    (start (point-min))
-                                    (end (point-max))
-                                    modified)
-                                (set-buffer buf)
-                                (insert-buffer-substring name start end)
-                                (set-buffer name)
-                                (dehexlify-buffer)
-                                (save-buffer)
-                                (setq modified (buffer-modified-p))
-                                (delete-region (point-min) (point-max))
-                                (insert-buffer-substring buf start end)
-                                (kill-buffer buf)
-                                modified))
-                          (message "(No changes need to be saved)")
-                          nil))
-  ;; Return t to indicate we have saved t
-  t)
+  (if hexl-in-save-buffer nil
+    (set-buffer-modified-p (if (buffer-modified-p)
+                              (save-excursion
+                                (let ((buf (generate-new-buffer " hexl"))
+                                      (name (buffer-name))
+                                      (file-name (buffer-file-name))
+                                      (start (point-min))
+                                      (end (point-max))
+                                      modified)
+                                  (set-buffer buf)
+                                  (insert-buffer-substring name start end)
+                                  (set-buffer name)
+                                  (dehexlify-buffer)
+                                  ;; Prevent infinite recursion.
+                                  (let ((hexl-in-save-buffer t)
+                                        (buffer-file-type t)) ; for ms-dos
+                                    (save-buffer))
+                                  (setq modified (buffer-modified-p))
+                                  (delete-region (point-min) (point-max))
+                                  (insert-buffer-substring buf start end)
+                                  (kill-buffer buf)
+                                  modified))
+                            (message "(No changes need to be saved)")
+                            nil))
+    ;; Return t to indicate we have saved t
+    t))
 
 ;;;###autoload
 (defun hexl-find-file (filename)
   "Edit file FILENAME in hexl-mode.
 Switch to a buffer visiting file FILENAME, creating one in none exists."
   (interactive "fFilename: ")
-  (find-file filename)
+  (if (or (eq system-type 'ms-dos) (eq system-type 'windows-nt))
+      (find-file-binary filename)
+    (find-file filename))
   (if (not (eq major-mode 'hexl-mode))
       (hexl-mode)))
 
@@ -220,27 +285,50 @@ With arg, don't unhexlify buffer."
   (interactive "p")
   (if (or (eq arg 1) (not arg))
       (let ((modified (buffer-modified-p))
-           (read-only buffer-read-only)
+           (inhibit-read-only t)
            (original-point (1+ (hexl-current-address))))
-       (setq buffer-read-only nil)
        (dehexlify-buffer)
+       (remove-hook 'write-contents-hooks 'hexl-save-buffer)
        (set-buffer-modified-p modified)
-       (setq buffer-read-only read-only)
        (goto-char original-point)))
+
+  (remove-hook 'after-revert-hook 'hexl-after-revert-hook t)
+  (remove-hook 'change-major-mode-hook 'hexl-maybe-dehexlify-buffer t)
+
+  (setq write-contents-hooks hexl-mode-old-write-contents-hooks)
+  (setq require-final-newline hexl-mode-old-require-final-newline)
   (setq mode-name hexl-mode-old-mode-name)
   (use-local-map hexl-mode-old-local-map)
+  (set-syntax-table hexl-mode-old-syntax-table)
   (setq major-mode hexl-mode-old-major-mode)
-;; Kludge to update mode-line
-  (switch-to-buffer (current-buffer))
-)
+  (force-mode-line-update))
 
-(defun hexl-current-address ()
+(defun hexl-maybe-dehexlify-buffer ()
+  "Convert a hexl format buffer to binary.
+Ask the user for confirmation."
+  (if (y-or-n-p "Convert contents back to binary format? ")
+      (let ((modified (buffer-modified-p))
+           (inhibit-read-only t)
+           (original-point (1+ (hexl-current-address))))
+       (dehexlify-buffer)
+       (remove-hook 'write-contents-hooks 'hexl-save-buffer)
+       (set-buffer-modified-p modified)
+       (goto-char original-point))))
+
+(defun hexl-current-address (&optional validate)
   "Return current hexl-address."
   (interactive)
   (let ((current-column (- (% (point) 68) 11)) 
        (hexl-address 0))
-    (setq hexl-address (+ (* (/ (point) 68) 16)
-                         (/ (- current-column  (/ current-column 5)) 2)))
+    (if (< current-column 0)
+       (if validate
+           (error "Point is not on a character in the file")
+         (setq current-column 0)))
+    (setq hexl-address
+         (+ (* (/ (point) 68) 16)
+            (if (>= current-column 41)
+                (- current-column 41)
+              (/ (- current-column  (/ current-column 5)) 2))))
     hexl-address))
 
 (defun hexl-address-to-marker (address)
@@ -250,7 +338,6 @@ With arg, don't unhexlify buffer."
 
 (defun hexl-goto-address (address)
   "Goto hexl-mode (decimal) address ADDRESS.
-
 Signal error if ADDRESS out of range."
   (interactive "nAddress: ")
   (if (or (< address 0) (> address hexl-max-address))
@@ -259,7 +346,6 @@ Signal error if ADDRESS out of range."
 
 (defun hexl-goto-hex-address (hex-address)
   "Go to hexl-mode address (hex string) HEX-ADDRESS.
-
 Signal error if HEX-ADDRESS is out of range."
   (interactive "sHex Address: ")
   (hexl-goto-address (hexl-hex-string-to-integer hex-address)))
@@ -385,7 +471,7 @@ If there is byte at the target address move to the last byte in that line."
   "Move vertically down ARG lines [16 bytes] (up if ARG negative) in hexl-mode.
 If there is no byte at the target address move to the last byte in that line."
   (interactive "p")
-  (hexl-goto-address (let ((address (+ (hexl-current-address) (* arg 16)) t))
+  (hexl-goto-address (let ((address (+ (hexl-current-address) (* arg 16))))
                       (if (and (< arg 0) (< address 0))
                                (progn (message "Out of hexl region.")
                                       (setq address
@@ -451,12 +537,12 @@ With prefix arg N, puts point N bytes of the way from the true beginning."
       (recenter 0))))
 
 (defun hexl-beginning-of-1k-page ()
-  "Goto to beginning of 1k boundry."
+  "Go to beginning of 1k boundary."
   (interactive)
   (hexl-goto-address (logand (hexl-current-address) -1024)))
 
 (defun hexl-end-of-1k-page ()
-  "Goto to end of 1k boundry."
+  "Go to end of 1k boundary."
   (interactive)
   (hexl-goto-address (let ((address (logior (hexl-current-address) 1023)))
                       (if (> address hexl-max-address)
@@ -464,12 +550,12 @@ With prefix arg N, puts point N bytes of the way from the true beginning."
                       address)))
 
 (defun hexl-beginning-of-512b-page ()
-  "Goto to beginning of 512 byte boundry."
+  "Go to beginning of 512 byte boundary."
   (interactive)
   (hexl-goto-address (logand (hexl-current-address) -512)))
 
 (defun hexl-end-of-512b-page ()
-  "Goto to end of 512 byte boundry."
+  "Go to end of 512 byte boundary."
   (interactive)
   (hexl-goto-address (let ((address (logior (hexl-current-address) 511)))
                       (if (> address hexl-max-address)
@@ -485,21 +571,37 @@ You may also type up to 3 octal digits, to insert a character with that code"
 
 ;00000000: 0011 2233 4455 6677 8899 aabb ccdd eeff  0123456789ABCDEF
 
+;;;###autoload
 (defun hexlify-buffer ()
-  "Convert a binary buffer to hexl format"
+  "Convert a binary buffer to hexl format.
+This discards the buffer's undo information."
   (interactive)
-  (shell-command-on-region (point-min) (point-max) hexlify-command t))
+  (and buffer-undo-list
+       (or (y-or-n-p "Converting to hexl format discards undo info; ok? ")
+          (error "Aborted")))
+  (setq buffer-undo-list nil)
+  (let ((binary-process-output nil) ; for Ms-Dos
+       (binary-process-input t)
+       (buffer-undo-list t))
+    (shell-command-on-region (point-min) (point-max) hexlify-command t)))
 
 (defun dehexlify-buffer ()
-  "Convert a hexl format buffer to binary."
+  "Convert a hexl format buffer to binary.
+This discards the buffer's undo information."
   (interactive)
-  (shell-command-on-region (point-min) (point-max) dehexlify-command t))
+  (and buffer-undo-list
+       (or (y-or-n-p "Converting from hexl format discards undo info; ok? ")
+          (error "Aborted")))
+  (setq buffer-undo-list nil)
+  (let ((binary-process-output t) ; for Ms-Dos
+       (binary-process-input nil)
+       (buffer-undo-list t))
+    (shell-command-on-region (point-min) (point-max) dehexlify-command t)))
 
 (defun hexl-char-after-point ()
   "Return char for ASCII hex digits at point."
-  (setq lh (char-after (point)))
-  (setq rh (char-after (1+ (point))))
-  (hexl-htoi lh rh))
+  (hexl-htoi (char-after (point))
+            (char-after (1+ (point)))))
 
 (defun hexl-htoi (lh rh)
   "Hex (char) LH (char) RH to integer."
@@ -513,13 +615,13 @@ You may also type up to 3 octal digits, to insert a character with that code"
     (let ((ch (logior character 32)))
       (if (and (>= ch ?a) (<= ch ?f))
          (- ch (- ?a 10))
-       (error (format "Invalid hex digit `%c'." ch))))))
+       (error "Invalid hex digit `%c'." ch)))))
 
 (defun hexl-oct-char-to-integer (character)
   "Take a char and return its value as if it was a octal digit."
   (if (and (>= character ?0) (<= character ?7))
       (- character ?0)
-    (error (format "Invalid octal digit `%c'." character))))
+    (error "Invalid octal digit `%c'." character)))
 
 (defun hexl-printable-character (ch)
   "Return a displayable string for character CH."
@@ -538,17 +640,32 @@ You may also type up to 3 octal digits, to insert a character with that code"
 
 (defun hexl-insert-char (ch num)
   "Insert a character in a hexl buffer."
-  (let ((address (hexl-current-address)))
+  (let ((address (hexl-current-address t)))
     (while (> num 0)
-      (delete-char 2)
-      (insert (format "%02x" ch))
-      (goto-char
-       (+ (* (/ address 16) 68) 52 (% address 16)))
-      (delete-char 1)
-      (insert (hexl-printable-character ch))
-      (if (eq address hexl-max-address)
-         (hexl-goto-address address)
-       (hexl-goto-address (1+ address)))
+      (let ((hex-position
+            (+ (* (/ address 16) 68)
+               11
+               (* 2 (% address 16))
+               (/ (% address 16) 2)))
+           (ascii-position
+            (+ (* (/ address 16) 68) 52 (% address 16)))
+           at-ascii-position)
+       (if (= (point) ascii-position)
+           (setq at-ascii-position t))
+       (goto-char hex-position)
+       (delete-char 2)
+       (insert (format "%02x" ch))
+       (goto-char ascii-position)
+       (delete-char 1)
+       (insert (hexl-printable-character ch))
+       (or (eq address hexl-max-address)
+           (setq address (1+ address)))
+       (hexl-goto-address address)
+       (if at-ascii-position
+           (progn
+             (beginning-of-line)
+             (forward-char 51)
+             (forward-char (% address 16)))))
       (setq num (1- num)))))
 
 ;; hex conversion
@@ -581,94 +698,109 @@ You may also type up to 3 octal digits, to insert a character with that code"
 
 (if hexl-mode-map
     nil
-    (setq hexl-mode-map (make-sparse-keymap))
-
-    (define-key hexl-mode-map "\C-a" 'hexl-beginning-of-line)
-    (define-key hexl-mode-map "\C-b" 'hexl-backward-char)
-    (define-key hexl-mode-map "\C-d" 'undefined)
-    (define-key hexl-mode-map "\C-e" 'hexl-end-of-line)
-    (define-key hexl-mode-map "\C-f" 'hexl-forward-char)
-
-    (if (not (eq (key-binding help-char) 'help-command))
-       (define-key hexl-mode-map help-char 'undefined))
-
-    (define-key hexl-mode-map "\C-i" 'hexl-self-insert-command)
-    (define-key hexl-mode-map "\C-j" 'hexl-self-insert-command)
-    (define-key hexl-mode-map "\C-k" 'undefined)
-    (define-key hexl-mode-map "\C-m" 'hexl-self-insert-command)
-    (define-key hexl-mode-map "\C-n" 'hexl-next-line)
-    (define-key hexl-mode-map "\C-o" 'undefined)
-    (define-key hexl-mode-map "\C-p" 'hexl-previous-line)
-    (define-key hexl-mode-map "\C-q" 'hexl-quoted-insert)
-    (define-key hexl-mode-map "\C-t" 'undefined)
-    (define-key hexl-mode-map "\C-v" 'hexl-scroll-up)
-    (define-key hexl-mode-map "\C-w" 'undefined)
-    (define-key hexl-mode-map "\C-y" 'undefined)
-
-    (let ((ch 32))
-      (while (< ch 127)
-       (define-key hexl-mode-map (format "%c" ch) 'hexl-self-insert-command)
-       (setq ch (1+ ch))))
-
-    (define-key hexl-mode-map "\e\C-a" 'hexl-beginning-of-512b-page)
-    (define-key hexl-mode-map "\e\C-b" 'hexl-backward-short)
-    (define-key hexl-mode-map "\e\C-c" 'undefined)
-    (define-key hexl-mode-map "\e\C-d" 'hexl-insert-decimal-char)
-    (define-key hexl-mode-map "\e\C-e" 'hexl-end-of-512b-page)
-    (define-key hexl-mode-map "\e\C-f" 'hexl-forward-short)
-    (define-key hexl-mode-map "\e\C-g" 'undefined)
-    (define-key hexl-mode-map "\e\C-h" 'undefined)
-    (define-key hexl-mode-map "\e\C-i" 'undefined)
-    (define-key hexl-mode-map "\e\C-j" 'undefined)
-    (define-key hexl-mode-map "\e\C-k" 'undefined)
-    (define-key hexl-mode-map "\e\C-l" 'undefined)
-    (define-key hexl-mode-map "\e\C-m" 'undefined)
-    (define-key hexl-mode-map "\e\C-n" 'undefined)
-    (define-key hexl-mode-map "\e\C-o" 'hexl-insert-octal-char)
-    (define-key hexl-mode-map "\e\C-p" 'undefined)
-    (define-key hexl-mode-map "\e\C-q" 'undefined)
-    (define-key hexl-mode-map "\e\C-r" 'undefined)
-    (define-key hexl-mode-map "\e\C-s" 'undefined)
-    (define-key hexl-mode-map "\e\C-t" 'undefined)
-    (define-key hexl-mode-map "\e\C-u" 'undefined)
-
-    (define-key hexl-mode-map "\e\C-w" 'undefined)
-    (define-key hexl-mode-map "\e\C-x" 'hexl-insert-hex-char)
-    (define-key hexl-mode-map "\e\C-y" 'undefined)
-
-    (define-key hexl-mode-map "\ea" 'undefined)
-    (define-key hexl-mode-map "\eb" 'hexl-backward-word)
-    (define-key hexl-mode-map "\ec" 'undefined)
-    (define-key hexl-mode-map "\ed" 'undefined)
-    (define-key hexl-mode-map "\ee" 'undefined)
-    (define-key hexl-mode-map "\ef" 'hexl-forward-word)
-    (define-key hexl-mode-map "\eg" 'hexl-goto-hex-address)
-    (define-key hexl-mode-map "\eh" 'undefined)
-    (define-key hexl-mode-map "\ei" 'undefined)
-    (define-key hexl-mode-map "\ej" 'hexl-goto-address)
-    (define-key hexl-mode-map "\ek" 'undefined)
-    (define-key hexl-mode-map "\el" 'undefined)
-    (define-key hexl-mode-map "\em" 'undefined)
-    (define-key hexl-mode-map "\en" 'undefined)
-    (define-key hexl-mode-map "\eo" 'undefined)
-    (define-key hexl-mode-map "\ep" 'undefined)
-    (define-key hexl-mode-map "\eq" 'undefined)
-    (define-key hexl-mode-map "\er" 'undefined)
-    (define-key hexl-mode-map "\es" 'undefined)
-    (define-key hexl-mode-map "\et" 'undefined)
-    (define-key hexl-mode-map "\eu" 'undefined)
-    (define-key hexl-mode-map "\ev" 'hexl-scroll-down)
-    (define-key hexl-mode-map "\ey" 'undefined)
-    (define-key hexl-mode-map "\ez" 'undefined)
-    (define-key hexl-mode-map "\e<" 'hexl-beginning-of-buffer)
-    (define-key hexl-mode-map "\e>" 'hexl-end-of-buffer)
-
-    (define-key hexl-mode-map "\C-c\C-c" 'hexl-mode-exit)
-
-    (define-key hexl-mode-map "\C-x[" 'hexl-beginning-of-1k-page)
-    (define-key hexl-mode-map "\C-x]" 'hexl-end-of-1k-page)
-    (define-key hexl-mode-map "\C-x\C-p" 'undefined)
-    (define-key hexl-mode-map "\C-x\C-s" 'hexl-save-buffer)
-    (define-key hexl-mode-map "\C-x\C-t" 'undefined))
+  (setq hexl-mode-map (make-sparse-keymap))
+
+  (define-key hexl-mode-map [left] 'hexl-backward-char)
+  (define-key hexl-mode-map [right] 'hexl-forward-char)
+  (define-key hexl-mode-map [up] 'hexl-previous-line)
+  (define-key hexl-mode-map [down] 'hexl-next-line)
+  (define-key hexl-mode-map [M-left] 'hexl-backward-short)
+  (define-key hexl-mode-map [M-right] 'hexl-forward-short)
+  (define-key hexl-mode-map [next] 'hexl-scroll-up)
+  (define-key hexl-mode-map [prior] 'hexl-scroll-down)
+  (define-key hexl-mode-map [home] 'hexl-beginning-of-buffer)
+  (define-key hexl-mode-map [deletechar] 'undefined)
+  (define-key hexl-mode-map [deleteline] 'undefined)
+  (define-key hexl-mode-map [insertline] 'undefined)
+  (define-key hexl-mode-map [S-delete] 'undefined)
+  (define-key hexl-mode-map "\177" 'undefined)
+
+  (define-key hexl-mode-map "\C-a" 'hexl-beginning-of-line)
+  (define-key hexl-mode-map "\C-b" 'hexl-backward-char)
+  (define-key hexl-mode-map "\C-d" 'undefined)
+  (define-key hexl-mode-map "\C-e" 'hexl-end-of-line)
+  (define-key hexl-mode-map "\C-f" 'hexl-forward-char)
+
+  (if (not (eq (key-binding (char-to-string help-char)) 'help-command))
+      (define-key hexl-mode-map (char-to-string help-char) 'undefined))
+
+  (define-key hexl-mode-map "\C-i" 'hexl-self-insert-command)
+  (define-key hexl-mode-map "\C-j" 'hexl-self-insert-command)
+  (define-key hexl-mode-map "\C-k" 'undefined)
+  (define-key hexl-mode-map "\C-m" 'hexl-self-insert-command)
+  (define-key hexl-mode-map "\C-n" 'hexl-next-line)
+  (define-key hexl-mode-map "\C-o" 'undefined)
+  (define-key hexl-mode-map "\C-p" 'hexl-previous-line)
+  (define-key hexl-mode-map "\C-q" 'hexl-quoted-insert)
+  (define-key hexl-mode-map "\C-t" 'undefined)
+  (define-key hexl-mode-map "\C-v" 'hexl-scroll-up)
+  (define-key hexl-mode-map "\C-w" 'undefined)
+  (define-key hexl-mode-map "\C-y" 'undefined)
+
+  (let ((ch 32))
+    (while (< ch 127)
+      (define-key hexl-mode-map (format "%c" ch) 'hexl-self-insert-command)
+      (setq ch (1+ ch))))
+
+  (define-key hexl-mode-map "\e\C-a" 'hexl-beginning-of-512b-page)
+  (define-key hexl-mode-map "\e\C-b" 'hexl-backward-short)
+  (define-key hexl-mode-map "\e\C-c" 'undefined)
+  (define-key hexl-mode-map "\e\C-d" 'hexl-insert-decimal-char)
+  (define-key hexl-mode-map "\e\C-e" 'hexl-end-of-512b-page)
+  (define-key hexl-mode-map "\e\C-f" 'hexl-forward-short)
+  (define-key hexl-mode-map "\e\C-g" 'undefined)
+  (define-key hexl-mode-map "\e\C-h" 'undefined)
+  (define-key hexl-mode-map "\e\C-i" 'undefined)
+  (define-key hexl-mode-map "\e\C-j" 'undefined)
+  (define-key hexl-mode-map "\e\C-k" 'undefined)
+  (define-key hexl-mode-map "\e\C-l" 'undefined)
+  (define-key hexl-mode-map "\e\C-m" 'undefined)
+  (define-key hexl-mode-map "\e\C-n" 'undefined)
+  (define-key hexl-mode-map "\e\C-o" 'hexl-insert-octal-char)
+  (define-key hexl-mode-map "\e\C-p" 'undefined)
+  (define-key hexl-mode-map "\e\C-q" 'undefined)
+  (define-key hexl-mode-map "\e\C-r" 'undefined)
+  (define-key hexl-mode-map "\e\C-s" 'undefined)
+  (define-key hexl-mode-map "\e\C-t" 'undefined)
+  (define-key hexl-mode-map "\e\C-u" 'undefined)
+
+  (define-key hexl-mode-map "\e\C-w" 'undefined)
+  (define-key hexl-mode-map "\e\C-x" 'hexl-insert-hex-char)
+  (define-key hexl-mode-map "\e\C-y" 'undefined)
+
+  (define-key hexl-mode-map "\ea" 'undefined)
+  (define-key hexl-mode-map "\eb" 'hexl-backward-word)
+  (define-key hexl-mode-map "\ec" 'undefined)
+  (define-key hexl-mode-map "\ed" 'undefined)
+  (define-key hexl-mode-map "\ee" 'undefined)
+  (define-key hexl-mode-map "\ef" 'hexl-forward-word)
+  (define-key hexl-mode-map "\eg" 'hexl-goto-hex-address)
+  (define-key hexl-mode-map "\eh" 'undefined)
+  (define-key hexl-mode-map "\ei" 'undefined)
+  (define-key hexl-mode-map "\ej" 'hexl-goto-address)
+  (define-key hexl-mode-map "\ek" 'undefined)
+  (define-key hexl-mode-map "\el" 'undefined)
+  (define-key hexl-mode-map "\em" 'undefined)
+  (define-key hexl-mode-map "\en" 'undefined)
+  (define-key hexl-mode-map "\eo" 'undefined)
+  (define-key hexl-mode-map "\ep" 'undefined)
+  (define-key hexl-mode-map "\eq" 'undefined)
+  (define-key hexl-mode-map "\er" 'undefined)
+  (define-key hexl-mode-map "\es" 'undefined)
+  (define-key hexl-mode-map "\et" 'undefined)
+  (define-key hexl-mode-map "\eu" 'undefined)
+  (define-key hexl-mode-map "\ev" 'hexl-scroll-down)
+  (define-key hexl-mode-map "\ey" 'undefined)
+  (define-key hexl-mode-map "\ez" 'undefined)
+  (define-key hexl-mode-map "\e<" 'hexl-beginning-of-buffer)
+  (define-key hexl-mode-map "\e>" 'hexl-end-of-buffer)
+
+  (define-key hexl-mode-map "\C-c\C-c" 'hexl-mode-exit)
+
+  (define-key hexl-mode-map "\C-x[" 'hexl-beginning-of-1k-page)
+  (define-key hexl-mode-map "\C-x]" 'hexl-end-of-1k-page)
+  (define-key hexl-mode-map "\C-x\C-p" 'undefined)
+  (define-key hexl-mode-map "\C-x\C-s" 'hexl-save-buffer)
+  (define-key hexl-mode-map "\C-x\C-t" 'undefined))
 
 ;;; hexl.el ends here