Added an other-url-specific face.
[elpher.git] / elpher.el
index 214f24f..1fa5ed5 100644 (file)
--- a/elpher.el
+++ b/elpher.el
 ;;; Global constants
 ;;
 
-(defconst elpher-version "1.4.7"
+(defconst elpher-version "2.0.0"
   "Current version of elpher.")
 
 (defconst elpher-margin-width 6
   "Width of left-hand margin used when rendering indicies.")
 
-(defconst elpher-transport-map
-  '(("gopher" elpher-get-gopher-selector)
-    ("gophers" elpher-get-gopher-selector)
-    ("gemini" elpher-get-gemini-selector)))
-
-(defconst elpher-gopher-type-map
-  '((?0 elpher-display-text "txt" elpher-text)
-    (?1 elpher-display-gophermap "/" elpher-index)
-    (?4 nil "bin" elpher-binary)
-    (?5 nil "bin" elpher-binary)
-    (?7 elpher-display-search-node "?" elpher-search)
-    (?8 elpher-display-telnet-node "tel" elpher-telnet)
-    (?9 nil "bin" elpher-binary)
-    (?g elpher-display-image-node "img" elpher-image)
-    (?p elpher-display-image-node "img" elpher-image)
-    (?I elpher-display-image-node "img" elpher-image)
-    (?d nil "doc" elpher-binary)
-    (?P nil "doc" elpher-binary)
-    (?s nil "snd" elpher-binary)
-    (?h elpher-display-node-html "htm" elpher-html))
+(defconst elpher-type-map
+  '(((gopher ?0) elpher-get-text-node "txt" elpher-text)
+    ((gopher ?1) elpher-get-index-node "/" elpher-index)
+    ((gopher ?4) elpher-get-node-download "bin" elpher-binary)
+    ((gopher ?5) elpher-get-node-download "bin" elpher-binary)
+    ((gopher ?7) elpher-get-search-node "?" elpher-search)
+    ((gopher ?8) elpher-get-telnet-node "tel" elpher-telnet)
+    ((gopher ?9) elpher-get-node-download "bin" elpher-binary)
+    ((gopher ?g) elpher-get-image-node "img" elpher-image)
+    ((gopher ?p) elpher-get-image-node "img" elpher-image)
+    ((gopher ?I) elpher-get-image-node "img" elpher-image)
+    ((gopher ?d) elpher-get-node-download "doc" elpher-binary)
+    ((gopher ?P) elpher-get-node-download "doc" elpher-binary)
+    ((gopher ?s) elpher-get-node-download "snd" elpher-binary)
+    ((gopher ?h) elpher-get-html-node "htm" elpher-html)
+    (other-url elpher-get-other-url-node "url" elpher-other-url)
+    ((special bookmarks) elpher-get-bookmarks-node)
+    ((special start) elpher-get-start-node))
   "Association list from types to getters, margin codes and index faces.")
 
-(defconst elpher-mime-type-map
-  '(("text/gemini" elpher-display-node-text)
-    ("text/html" elpher-display-node-html)
-    ("text/*" elpher-display-node-text)
-    ("image/*" elpher-display-image-node)))
 
 ;;; Customization group
 ;;
   '((t :inherit warning))
   "Face used for search type directory records.")
 
-(defface elpher-url
+(defface elpher-html
+  '((t :inherit font-lock-comment-face))
+  "Face used for url type directory records.")
+
+(defface elpher-other-url
   '((t :inherit font-lock-comment-face))
   "Face used for url type directory records.")
 
@@ -177,75 +174,125 @@ allows switching from an encrypted channel back to plain text without user input
 
 ;; Address
 
-(defun elpher-url-from-string (url-string)
-  "Create a URL object corresponding to the given URL-STRING."
-  (let ((url (url-generic-parse-url url-string)))
-    (if (and (url-type url)
-             (url-host url))
-        (setf (url-filename url) (url-unhex-string (url-filename url)))
-      (error "Malformed URL" url))))
-
-(defun elpher-url-to-url-string (url)
-  "Get string representation of URL."
-  (url-encode-url (url-recreate url)))
-
-(defun elpher-url-gopher-p (url)
-  "Return non-nil if URL object specifies a gopher address."
-  (let ((protocol (url-type url)))
-    (if (or (string-equal protocol "gopher")
-            (string-equal protocol "gophers")))))
-
-(defun elpher-gopher-url-selector-type (url)
-  "Retrieve selector type from URL object."
-  (let ((filename (url-filename url)))
-    (if (> (length filename) 0)
-        (string-to-char filename)
-      ?1)))
-
-(defun elpher-gopher-url-selector (url)
-  "Retrieve selector from URL object."
-  (let ((filename (url-filename url)))
-    (if (> (length filename) 0)
-        (substring filename 1)
-      "")))
-
-(defun elpher-url-host (url)
-  "Retrieve host from URL object."
-  (url-host url))
-
-(defun elpher-url-port (url)
-  "Retrieve port from URL object."
-  (url-port url))
-
-(defun elpher-url-use-tls-p (url)
-  "Return non-nil if URL is marked as needing TLS."
-  (string-equal (url-type address) "gophers"))
-
-(defun elpher-url-special-p (url)
-  "Return non-nil if URL object is special (e.g. start page, bookmarks page)."
-  (symbolp url))
+;; An elpher "address" object is either a url object or a symbol.
+;; Symbol addresses are "special", corresponding to pages generated
+;; dynamically for and by elpher.  All others represent pages which
+;; rely on content retrieved over the network.
+
+(defun elpher-address-from-url (url-string)
+  "Create a ADDRESS object corresponding to the given URL-STRING."
+  (let ((data (match-data))) ; Prevent parsing clobbering match data
+    (unwind-protect
+        (let ((url (url-generic-parse-url url-string)))
+          (setf (url-fullness url) t)
+          (unless (url-host url)
+            (setf (url-host url) (url-filename url))
+            (setf (url-filename url) ""))
+          (unless (url-type url)
+            (setf (url-type url) "gopher"))
+          (if (and (url-type url)
+                   (url-host url))
+              (let ((is-gopher (or (equal "gopher" (url-type url))
+                                   (equal "gophers" (url-type url)))))
+                (setf (url-filename url)
+                      (url-unhex-string (url-filename url)))
+                (when (or (equal (url-filename url) "")
+                          (equal (url-filename url) "/"))
+                  (if is-gopher
+                      (setf (url-filename url) "/1")))
+                (unless (> (url-port url) 0)
+                  (if is-gopher
+                      (setf (url-port url) 70)))
+                url)
+            (error "Malformed URL" url)))
+      (set-match-data data))))
+
+(defun elpher-make-gopher-address (type selector host port &optional tls)
+  "Create an ADDRESS object corresponding to the given gopher directory record
+attributes: TYPE, SELECTOR, HOST and PORT."
+  (if (and (equal type ?h)
+           (string-prefix-p "URL:" selector))
+      (elpher-address-from-url (elt (split-string selector "URL:") 1))
+    (elpher-address-from-url
+     (concat "gopher" (if tls "s" "")
+             "://" host
+             ":" (number-to-string port)
+             "/" (string type)
+             selector))))
+
+(defun elpher-make-special-address (type)
+  "Create an ADDRESS object corresponding to the given special page symbol TYPE."
+  type)
+
+(defun elpher-address-to-url (address)
+  "Get string representation of ADDRESS, or nil if ADDRESS is special."
+  (if (not (elpher-address-special-p address))
+      (url-encode-url (url-recreate-url address))
+    nil))
+
+(defun elpher-address-type (address)
+  "Retrieve selector type from ADDRESS object."
+  (if (symbolp address)
+      (list 'special address)
+    (let ((protocol (url-type address)))
+      (cond ((or (equal protocol "gopher")
+                 (equal protocol "gophers"))
+             (list 'gopher (string-to-char (substring (url-filename address) 1))))
+            ((equal protocol "gemini")
+             'gemini)
+            (t 'other-url)))))
+
+(defun elpher-address-protocol (address)
+  (if (symbolp address)
+      nil
+    (url-type address)))
+
+(defun elpher-address-filename (address)
+  (if (symbolp address)
+      nil
+    (url-filename address)))
+
+(defun elpher-address-host (address)
+  "Retrieve host from ADDRESS object."
+  (url-host address))
+
+(defun elpher-address-port (address)
+  "Retrieve port from ADDRESS object."
+  (url-port address))
+
+(defun elpher-address-special-p (address)
+  "Return non-nil if ADDRESS object is special (e.g. start page, bookmarks page)."
+  (symbolp address))
+
+(defun elpher-address-gopher-p (address)
+  "Return non-nill if ADDRESS object is a gopher address."
+  (memq (elpher-address-protocol address) '("gopher gophers")))
+
+(defun elpher-gopher-address-selector (address)
+  "Retrieve gopher selector from ADDRESS object."
+  (substring (url-filename address) 2))
 
 ;; Node
 
-(defun elpher-make-node (display-string url &optional parent)
+(defun elpher-make-node (display-string address &optional parent)
   "Create a node in the page hierarchy.
 
 DISPLAY-STRING records the display string used for the page.
 
-URL specifies the url object of the page.
+ADDRESS specifies the address object of the page.
 
 The optional PARENT specifies the parent node in the hierarchy.
 This is set every time the node is visited, so while it forms
 an important part of the node data there is no need to set it
 initially."
-  (list display-string url parent))
+  (list display-string address parent))
 
 (defun elpher-node-display-string (node)
   "Retrieve the display string of NODE."
   (elt node 0))
 
-(defun elpher-node-url (node)
-  "Retrieve the URL object of NODE."
+(defun elpher-node-address (node)
+  "Retrieve the ADDRESS object of NODE."
   (elt node 1))
 
 (defun elpher-node-parent (node)
@@ -261,21 +308,21 @@ initially."
 (defvar elpher-content-cache (make-hash-table :test 'equal))
 (defvar elpher-pos-cache (make-hash-table :test 'equal))
 
-(defun elpher-get-cached-content (url)
-  "Retrieve the cached content for URL, or nil if none exists."
-  (gethash url elpher-content-cache))
+(defun elpher-get-cached-content (address)
+  "Retrieve the cached content for ADDRESS, or nil if none exists."
+  (gethash address elpher-content-cache))
 
-(defun elpher-cache-content (url content)
-  "Set the content cache for URL to CONTENT."
-  (puthash url content elpher-content-cache))
+(defun elpher-cache-content (address content)
+  "Set the content cache for ADDRESS to CONTENT."
+  (puthash address content elpher-content-cache))
 
-(defun elpher-get-cached-pos (url)
-  "Retrieve the cached cursor position for URL, or nil if none exists."
-  (gethash url elpher-pos-cache))
+(defun elpher-get-cached-pos (address)
+  "Retrieve the cached cursor position for ADDRESS, or nil if none exists."
+  (gethash address elpher-pos-cache))
 
-(defun elpher-cache-pos (url pos)
-  "Set the cursor position cache for URL to POS."
-  (puthash url pos elpher-pos-cache))
+(defun elpher-cache-pos (address pos)
+  "Set the cursor position cache for ADDRESS to POS."
+  (puthash address pos elpher-pos-cache))
 
 ;; Node graph traversal
 
@@ -289,22 +336,26 @@ unless PRESERVE-PARENT is non-nil."
   (elpher-process-cleanup)
   (unless preserve-parent
     (if (and (elpher-node-parent elpher-current-node)
-             (equal (elpher-node-url elpher-current-node)
-                    (elpher-node-url node)))
+             (equal (elpher-node-address elpher-current-node)
+                    (elpher-node-address node)))
         (elpher-set-node-parent node (elpher-node-parent elpher-current-node))
       (elpher-set-node-parent node elpher-current-node)))
   (setq elpher-current-node node)
   (if getter
       (funcall getter)
-    ;; The business below needs updating: mapping from url->getter is different
-    ;; and more complex.
-    (let* ((url (elpher-node-url node))
+    (let* ((address (elpher-node-address node))
            (type (elpher-address-type address))
-           (type-record (alist-get type elpher-type-map)))
+           (type-record (cdr (assoc type elpher-type-map))))
       (if type-record
           (funcall (car type-record))
         (elpher-visit-parent-node)
-        (error "Unsupported gopher selector type '%c'" type)))))
+        (pcase type
+          (`(gopher ,type-char)
+           (error "Unsupported gopher selector type '%c' for '%s'"
+                  type-char (elpher-address-to-url address)))
+          (else
+           (error "Unsupported address type '%S' for '%s'"
+                  type (elpher-address-to-url address))))))))
 
 (defun elpher-visit-parent-node ()
   "Visit the parent of the current node."
@@ -368,6 +419,7 @@ away CRs and any terminating period."
   (elpher-decode (replace-regexp-in-string "\n\.\n$" "\n"
                                            (replace-regexp-in-string "\r" "" string))))
 
+
 ;;; Index rendering
 ;;
 
@@ -386,7 +438,7 @@ away CRs and any terminating period."
                (port (if (elt fields 3)
                          (string-to-number (elt fields 3))
                        nil))
-               (address (elpher-make-address type selector host port)))
+               (address (elpher-make-gopher-address type selector host port)))
           (elpher-insert-index-record display-string address))))))
 
 (defun elpher-insert-margin (&optional type-name)
@@ -404,20 +456,13 @@ away CRs and any terminating period."
 (defun elpher-node-button-help (node)
   "Return a string containing the help text for a button corresponding to NODE."
   (let ((address (elpher-node-address node)))
-    (if (eq (elpher-address-type address) ?h)
-        (let ((url (cadr (split-string (elpher-address-selector address) "URL:"))))
-          (format "mouse-1, RET: open url '%s'" url))
-      (format "mouse-1, RET: open '%s' on %s port %s"
-              (elpher-address-selector address)
-              (elpher-address-host address)
-              (elpher-address-port address)))))
-
+    (format "mouse-1, RET: open '%s'" (elpher-address-to-url address))))
 
 (defun elpher-insert-index-record (display-string address)
   "Function to insert an index record into the current buffer.
 The contents of the record are dictated by DISPLAY-STRING and ADDRESS."
   (let* ((type (elpher-address-type address))
-         (type-map-entry (alist-get type elpher-type-map)))
+         (type-map-entry (cdr (assoc type elpher-type-map))))
     (if type-map-entry
         (let* ((margin-code (elt type-map-entry 1))
                (face (elt type-map-entry 2))
@@ -430,15 +475,15 @@ The contents of the record are dictated by DISPLAY-STRING and ADDRESS."
                               'follow-link t
                               'help-echo (elpher-node-button-help node)))
       (pcase type
-        (?i ;; Information
+        ('(gopher ?i) ;; Information
          (elpher-insert-margin)
          (insert (propertize
                   (if elpher-buttonify-urls-in-directories
                       (elpher-buttonify-urls display-string)
                     display-string)
                   'face 'elpher-info)))
-        (other ;; Unknown
-         (elpher-insert-margin (concat (char-to-string type) "?"))
+        (`(gopher ,selector-type) ;; Unknown
+         (elpher-insert-margin (concat (char-to-string selector-type) "?"))
          (insert (propertize display-string
                              'face 'elpher-unknown)))))
     (insert "\n")))
@@ -449,7 +494,7 @@ The contents of the record are dictated by DISPLAY-STRING and ADDRESS."
     (elpher-visit-node node)))
 
 
-;;; Selector retrieval (all kinds)
+;;; Gopher selector retrieval (all kinds)
 ;;
 
 (defun elpher-process-cleanup ()
@@ -471,7 +516,7 @@ appropriate if the selector is to be directly viewed.  If PROPAGATE-ERROR
 is non-nil, this message is not displayed.  Instead, the error propagates
 up to the calling function."
   (setq elpher-selector-string "")
-  (when (elpher-address-use-tls-p address)
+  (when (equal (elpher-address-protocol address) "gophers")
       (if (gnutls-available-p)
           (when (not elpher-use-tls)
             (setq elpher-use-tls t)
@@ -491,11 +536,11 @@ up to the calling function."
                                     (concat elpher-selector-string string))))
         (set-process-sentinel proc after)
         (process-send-string proc
-                             (concat (elpher-address-selector address) "\n")))
+                             (concat (elpher-gopher-address-selector address) "\n")))
     (error
      (if (and (consp the-error)
               (eq (car the-error) 'gnutls-error)
-              (not (elpher-address-use-tls-p address))
+              (not (equal (elpher-address-protocol address) "gophers"))
               (or elpher-auto-disengage-TLS
                   (yes-or-no-p "Could not establish encrypted connection.  Disable TLS mode? ")))
          (progn
@@ -507,7 +552,7 @@ up to the calling function."
            (error the-error)
          (elpher-with-clean-buffer
           (insert (propertize "\n---- ERROR -----\n\n" 'face 'error)
-                  "Failed to connect to " (elpher-get-address-url address) ".\n"
+                  "Failed to connect to " (elpher-address-to-url address) ".\n"
                   (propertize "\n----------------\n\n" 'face 'error)
                   "Press 'u' to return to the previous page.")))))))
 
@@ -540,50 +585,14 @@ up to the calling function."
   "\\([a-zA-Z]+\\)://\\([a-zA-Z0-9.\-]+\\|\[[a-zA-Z0-9:]+\]\\)\\(?3::[0-9]+\\)?\\(?4:/[^ \r\n\t(),]*\\)?"
   "Regexp used to locate and buttinofy URLs in text files loaded by elpher.")
 
-(defun elpher-make-node-from-matched-url (&optional string)
-  "Convert most recent `elpher-url-regex' match to a node.
-
-If STRING is non-nil, this is given as an argument to all `match-string'
-calls, as is necessary if the match is performed by `string-match'."
-  (let ((url (match-string 0 string))
-        (protocol (downcase (match-string 1 string))))
-    (if (or (string= protocol "gopher")
-            (string= protocol "gophers"))
-        (let* ((bare-host (match-string 2 string))
-               (host (if (string-prefix-p "[" bare-host)
-                         (substring bare-host 1 (- (length bare-host) 1))
-                       bare-host))
-               (port (if (> (length (match-string 3 string))  1)
-                         (string-to-number (substring (match-string 3 string) 1))
-                       70))
-               (type-and-selector (match-string 4 string))
-               (type (if (> (length type-and-selector) 1)
-                         (elt type-and-selector 1)
-                       ?1))
-               (selector (decode-coding-string
-                          (url-unhex-string
-                           (if (> (length type-and-selector) 1)
-                               (substring type-and-selector 2)
-                             "")) 'utf-8))
-               (use-tls (string= protocol "gophers"))
-               (address (elpher-make-address type selector host port use-tls)))
-          (elpher-make-node url address))
-      (let* ((host (match-string 2 string))
-             (port (if (> (length (match-string 3 string)) 1)
-                       (string-to-number (substring (match-string 3 string) 1))
-                     70))
-             (selector (concat "URL:" url))
-             (address (elpher-make-address ?h selector host port)))
-        (elpher-make-node url address)))))
-
-
 (defun elpher-buttonify-urls (string)
   "Turn substrings which look like urls in STRING into clickable buttons."
   (with-temp-buffer
     (insert string)
     (goto-char (point-min))
     (while (re-search-forward elpher-url-regex nil t)
-        (let ((node (elpher-make-node-from-matched-url)))
+      (let ((node (elpher-make-node (match-string 0)
+                                    (elpher-address-from-url (match-string 0)))))
           (make-text-button (match-beginning 0)
                             (match-end 0)
                             'elpher-node  node
@@ -651,8 +660,8 @@ calls, as is necessary if the match is performed by `string-match'."
           (message "Displaying cached search results.  Reload to perform a new search."))
       (unwind-protect
           (let* ((query-string (read-string "Query: "))
-                 (query-selector (concat (elpher-address-selector address) "\t" query-string))
-                 (search-address (elpher-make-address ?1
+                 (query-selector (concat (elpher-gopher-address-selector address) "\t" query-string))
+                 (search-address (elpher-make-gopher-address ?1
                                                       query-selector
                                                       (elpher-address-host address)
                                                       (elpher-address-port address))))
@@ -693,7 +702,7 @@ calls, as is necessary if the match is performed by `string-match'."
 (defun elpher-get-node-download ()
   "Getter which retrieves the current node and writes the result to a file."
   (let* ((address (elpher-node-address elpher-current-node))
-         (selector (elpher-address-selector address)))
+         (selector (elpher-gopher-address-selector address)))
     (elpher-visit-parent-node) ; Do first in case of non-local exits.
     (let* ((filename-proposal (file-name-nondirectory selector))
            (filename (read-file-name "Save file as: "
@@ -715,7 +724,7 @@ calls, as is necessary if the match is performed by `string-match'."
         (error
          (error "Error downloading %s" elpher-download-filename))))))
 
-;; URL retrieval
+;; HTML node retrieval
 
 (defun elpher-insert-rendered-html (string)
   "Use shr to insert rendered view of html STRING into current buffer."
@@ -724,35 +733,41 @@ calls, as is necessary if the match is performed by `string-match'."
                (libxml-parse-html-region (point-min) (point-max)))))
     (shr-insert-document dom)))
 
-(defun elpher-get-url-node ()
-  "Getter which attempts to open the URL specified by the current node."
+(defun elpher-get-html-node ()
+  "Getter which retrieves and renders an HTML node."
   (let* ((address (elpher-node-address elpher-current-node))
-         (selector (elpher-address-selector address)))
-    (let ((url (elt (split-string selector "URL:") 1)))
-      (if url
+         (selector (elpher-gopher-address-selector address)))
+    (let ((content (elpher-get-cached-content address)))
+      (if content
           (progn
-            (elpher-visit-parent-node) ; Do first in case of non-local exits.
-            (message "Opening URL...")
-            (if elpher-open-urls-with-eww
-                (browse-web url)
-              (browse-url url)))
-        (let ((content (elpher-get-cached-content address)))
-          (if content
-              (progn
-                (elpher-with-clean-buffer
-                 (insert content)
-                 (elpher-restore-pos)))
             (elpher-with-clean-buffer
-             (insert "LOADING HTML... (use 'u' to cancel)"))
-            (elpher-get-selector address
-                                 (lambda (proc event)
-                                   (unless (string-prefix-p "deleted" event)
-                                     (elpher-with-clean-buffer
-                                      (elpher-insert-rendered-html elpher-selector-string)
-                                      (goto-char (point-min))
-                                      (elpher-cache-content
-                                       (elpher-node-address elpher-current-node)
-                                       (buffer-string))))))))))))
+             (insert content)
+             (elpher-restore-pos)))
+        (elpher-with-clean-buffer
+         (insert "LOADING HTML... (use 'u' to cancel)"))
+        (elpher-get-selector address
+                             (lambda (proc event)
+                               (unless (string-prefix-p "deleted" event)
+                                 (elpher-with-clean-buffer
+                                  (elpher-insert-rendered-html elpher-selector-string)
+                                  (goto-char (point-min))
+                                  (elpher-cache-content
+                                   (elpher-node-address elpher-current-node)
+                                   (buffer-string))))))))))
+
+
+;; Other URL node opening
+
+(defun elpher-get-other-url-node ()
+  "Getter which attempts to open the URL specified by the current node."
+  (let* ((address (elpher-node-address elpher-current-node))
+         (url (elpher-address-to-url address)))
+    (progn
+      (elpher-visit-parent-node) ; Do first in case of non-local exits.
+      (message "Opening URL...")
+      (if elpher-open-urls-with-eww
+          (browse-web url)
+        (browse-url url)))))
 
 ;; Telnet node connection
 
@@ -796,11 +811,11 @@ calls, as is necessary if the match is performed by `string-match'."
            "\n"
            "Start your exploration of gopher space:\n")
    (elpher-insert-index-record "Floodgap Systems Gopher Server"
-                               (elpher-make-address ?1 "" "gopher.floodgap.com" 70))
+                               (elpher-make-gopher-address ?1 "" "gopher.floodgap.com" 70))
    (insert "\n"
            "Alternatively, select the following item and enter some search terms:\n")
    (elpher-insert-index-record "Veronica-2 Gopher Search Engine"
-                               (elpher-make-address ?7 "/v2/vs" "gopher.floodgap.com" 70))
+                               (elpher-make-gopher-address ?7 "/v2/vs" "gopher.floodgap.com" 70))
    (insert "\n"
            "** Refer to the ")
    (let ((help-string "RET,mouse-1: Open Elpher info manual (if available)"))
@@ -828,7 +843,7 @@ calls, as is necessary if the match is performed by `string-match'."
      (if bookmarks
          (dolist (bookmark bookmarks)
            (let ((display-string (elpher-bookmark-display-string bookmark))
-                 (address (elpher-bookmark-address bookmark)))
+                 (address (elpher-address-from-url (elpher-bookmark-url bookmark))))
              (elpher-insert-index-record display-string address)))
        (insert "No bookmarks found.\n")))
    (insert "\n-----------------------\n\n"
@@ -843,11 +858,11 @@ calls, as is necessary if the match is performed by `string-match'."
 ;;; Bookmarks
 ;;
 
-(defun elpher-make-bookmark (display-string address)
+(defun elpher-make-bookmark (display-string url)
   "Make an elpher bookmark.
 DISPLAY-STRING determines how the bookmark will appear in the
 bookmark list, while ADDRESS is the address of the entry."
-  (list display-string address))
+  (list display-string (elpher-address-to-url address)))
   
 (defun elpher-bookmark-display-string (bookmark)
   "Get the display string of BOOKMARK."
@@ -857,46 +872,48 @@ bookmark list, while ADDRESS is the address of the entry."
   "Set the display string of BOOKMARK to DISPLAY-STRING."
   (setcar bookmark display-string))
 
-(defun elpher-bookmark-address (bookmark)
+(defun elpher-bookmark-url (bookmark)
   "Get the address for BOOKMARK."
   (elt bookmark 1))
 
+
 (defun elpher-save-bookmarks (bookmarks)
   "Record the bookmark list BOOKMARKS to the user's bookmark file.
 Beware that this completely replaces the existing contents of the file."
-  (with-temp-file (locate-user-emacs-file "elpher-bookmarks")
+  (with-temp-file (locate-user-emacs-file "elpher2-bookmarks")
     (erase-buffer)
-    (insert "; Elpher gopher bookmarks file\n\n"
-            "; Bookmarks are stored as a list of (label (type selector host port))\n"
-            "; s-expressions, where type is stored as a character (i.e. 49 = ?1).\n"
-            "; Feel free to edit by hand, but ensure this structure remains intact.\n\n")
+    (insert "; Elpher bookmarks file\n\n"
+            "; Bookmarks are stored as a list of (label URL) items.\n"
+            "; Feel free to edit by hand, but take care to ensure\n"
+            "; the list structure remains intact.\n\n")
     (pp bookmarks (current-buffer))))
 
 (defun elpher-load-bookmarks ()
   "Get the list of bookmarks from the users's bookmark file."
   (with-temp-buffer
     (ignore-errors
-      (insert-file-contents (locate-user-emacs-file "elpher-bookmarks"))
+      (insert-file-contents (locate-user-emacs-file "elpher2-bookmarks"))
       (goto-char (point-min))
       (read (current-buffer)))))
 
 (defun elpher-add-address-bookmark (address display-string)
-  "Save a bookmark for ADDRESS with label DISPLAY-STRING.
+  "Save a bookmark for ADDRESS with label DISPLAY-STRING.)))
 If ADDRESS is already bookmarked, update the label only."
-  (let ((bookmarks (elpher-load-bookmarks)))
-    (let ((existing-bookmark (rassoc (list address) bookmarks)))
+  (let ((bookmarks (elpher-load-bookmarks))
+        (url (elpher-address-to-url address)))
+    (let ((existing-bookmark (rassoc (list url) bookmarks)))
       (if existing-bookmark
           (elpher-set-bookmark-display-string existing-bookmark display-string)
-        (add-to-list 'bookmarks (elpher-make-bookmark display-string address))))
+        (add-to-list 'bookmarks (elpher-make-bookmark display-string url))))
     (elpher-save-bookmarks bookmarks)))
 
 (defun elpher-remove-address-bookmark (address)
   "Remove any bookmark to ADDRESS."
+  (let ((url (elpher-address-to-url address)))
     (elpher-save-bookmarks
      (seq-filter (lambda (bookmark)
-                   (not (equal (elpher-bookmark-address bookmark) address)))
-                 (elpher-load-bookmarks))))
-
+                   (not (equal (elpher-bookmark-url bookmark) url)))
+                 (elpher-load-bookmarks)))))
 
 ;;; Interactive procedures
 ;;
@@ -923,15 +940,8 @@ host, selector and port."
   (interactive)
   (let ((node
          (let ((host-or-url (read-string "Gopher host or URL: ")))
-           (if (string-match elpher-url-regex host-or-url)
-               (elpher-make-node-from-matched-url host-or-url)
-             (let ((selector (read-string "Selector (default none): " nil nil ""))
-                   (port-string (read-string "Port (default 70): " nil nil "70")))
-               (elpher-make-node (concat "gopher://" host-or-url
-                                         ":" port-string
-                                         "/1" selector)
-                                 (elpher-make-address ?1 selector host-or-url
-                                                      (string-to-number port-string))))))))
+           (elpher-make-node host-or-url
+                             (elpher-address-from-url host-or-url)))))
     (switch-to-buffer "*elpher*")
     (elpher-visit-node node)))
 
@@ -941,12 +951,8 @@ host, selector and port."
   (let ((address (elpher-node-address elpher-current-node)))
     (if (elpher-address-special-p address)
         (error "Command not valid for this page")
-      (let ((url (read-string "URL: " (elpher-get-address-url address))))
-        (if (string-match elpher-url-regex url)
-            (let ((new-node (elpher-make-node-from-matched-url url)))
-              (unless (equal (elpher-node-address new-node) address)
-                (elpher-visit-node new-node)))
-          (error "Could not parse URL %s" url))))))
+      (let ((url (read-string "URL: " (elpher-address-to-url address))))
+        (elpher-visit-node (elpher-make-node url (elpher-address-from-url url)))))))
 
 (defun elpher-redraw ()
   "Redraw current page."
@@ -1043,21 +1049,19 @@ host, selector and port."
          (host (elpher-address-host address)))
     (if host
         (let ((host (elpher-address-host address))
-              (selector (elpher-address-selector address))
+              (selector (elpher-gopher-address-selector address))
               (port (elpher-address-port address)))
           (if (> (length selector) 0)
-              (let ((root-address (elpher-make-address ?1 "" host port)))
+              (let ((root-address (elpher-make-gopher-address ?1 "" host port)))
                 (elpher-visit-node
-                 (elpher-make-node (concat "gopher://" host
-                                           ":" (number-to-string port)
-                                           "/1/")
-                                   root-address)))
+                 (elpher-make-node (elpher-address-to-url root-address))))
             (error "Already at root directory of current server")))
       (error "Command invalid for this page"))))
 
 (defun elpher-bookmarks-current-p ()
   "Return non-nil if current node is a bookmarks page."
-  (eq (elpher-address-type (elpher-node-address elpher-current-node)) 'bookmarks))
+  (equal (elpher-address-type (elpher-node-address elpher-current-node))
+         '(special bookmarks)))
 
 (defun elpher-reload-bookmarks ()
   "Reload bookmarks if current node is a bookmarks page."
@@ -1117,18 +1121,15 @@ host, selector and port."
   (interactive)
   (switch-to-buffer "*elpher*")
   (elpher-visit-node
-   (elpher-make-node "Bookmarks Page" (elpher-make-address 'bookmarks))))
+   (elpher-make-node "Bookmarks Page" (elpher-make-special-address 'bookmarks))))
 
 (defun elpher-info-node (node)
   "Display information on NODE."
   (let ((display-string (elpher-node-display-string node))
         (address (elpher-node-address node)))
-    (if (not (elpher-address-special-p address))
-        (message "`%s' on %s port %s"
-                (elpher-address-selector address)
-                (elpher-address-host address)
-                (elpher-address-port address))
-      (message "%s" display-string))))
+    (if (elpher-address-special-p address)
+        (message "Special page: %s" display-string)
+      (message (elpher-address-to-url address)))))
 
 (defun elpher-info-link ()
   "Display information on node corresponding to link at point."
@@ -1148,7 +1149,7 @@ host, selector and port."
   (let ((address (elpher-node-address node)))
     (if (elpher-address-special-p address)
         (error (format "Cannot represent %s as URL" (elpher-node-display-string node)))
-      (let ((url (elpher-get-address-url address)))
+      (let ((url (elpher-address-to-url address)))
         (message "Copied \"%s\" to kill-ring/clipboard." url)
         (kill-new url)))))
 
@@ -1206,7 +1207,7 @@ host, selector and port."
     (when (fboundp 'evil-define-key)
       (evil-define-key 'motion map
         (kbd "TAB") 'elpher-next-link
-        (kbd "C-]") 'elpher-follow-current-link
+        (kbd "C-") 'elpher-follow-current-link
         (kbd "C-t") 'elpher-back
         (kbd "u") 'elpher-back
         (kbd "O") 'elpher-root-dir
@@ -1255,7 +1256,7 @@ functions which initialize the gopher client, namely
     (switch-to-buffer "*elpher*")
     (setq elpher-current-node nil)
     (let ((start-node (elpher-make-node "Elpher Start Page"
-                                        (elpher-make-address 'start))))
+                                        (elpher-make-special-address 'start))))
       (elpher-visit-node start-node)))
   "Started Elpher.") ; Otherwise (elpher) evaluates to start page string.