*** empty log message ***
[gnus] / lisp / gnus-topic.el
index be170e2..fd27326 100644 (file)
 
 ;;; Code:
 
 
 ;;; Code:
 
+(require 'gnus-load)
+(require 'gnus-group)
+(require 'gnus-start)
 (require 'gnus)
 (require 'gnus)
-(eval-when-compile (require 'cl))
 
 (defvar gnus-topic-mode nil
   "Minor mode for Gnus group buffers.")
 
 (defvar gnus-topic-mode nil
   "Minor mode for Gnus group buffers.")
@@ -48,14 +50,20 @@ with some simple extensions.
 %A  Number of unread articles in the groups in the topic and its subtopics.
 ")
 
 %A  Number of unread articles in the groups in the topic and its subtopics.
 ")
 
-(defvar gnus-topic-unique t
-  "*If non-nil, each group will only belong to one topic.")
-
 (defvar gnus-topic-indent-level 2
   "*How much each subtopic should be indented.")
 
 (defvar gnus-topic-indent-level 2
   "*How much each subtopic should be indented.")
 
+(defvar gnus-topic-display-empty-topics t
+  "*If non-nil, display the topic lines even of topics that have no unread articles.")
+
 ;; Internal variables.
 
 ;; Internal variables.
 
+(defvar gnus-topic-active-topology nil)
+(defvar gnus-topic-active-alist nil)
+
+(defvar gnus-topology-checked-p nil
+  "Whether the topology has been checked in this session.")
+
 (defvar gnus-topic-killed-topics nil)
 (defvar gnus-topic-inhibit-change-level nil)
 (defvar gnus-topic-tallied-groups nil)
 (defvar gnus-topic-killed-topics nil)
 (defvar gnus-topic-inhibit-change-level nil)
 (defvar gnus-topic-tallied-groups nil)
@@ -71,29 +79,252 @@ with some simple extensions.
 
 (defvar gnus-topic-line-format-spec nil)
 
 
 (defvar gnus-topic-line-format-spec nil)
 
-;; Functions.
+;;; Utility functions
 
 (defun gnus-group-topic-name ()
   "The name of the topic on the current line."
 
 (defun gnus-group-topic-name ()
   "The name of the topic on the current line."
-  (get-text-property (gnus-point-at-bol) 'gnus-topic))
+  (let ((topic (get-text-property (gnus-point-at-bol) 'gnus-topic)))
+    (and topic (symbol-name topic))))
 
 (defun gnus-group-topic-level ()
   "The level of the topic on the current line."
   (get-text-property (gnus-point-at-bol) 'gnus-topic-level))
 
 
 (defun gnus-group-topic-level ()
   "The level of the topic on the current line."
   (get-text-property (gnus-point-at-bol) 'gnus-topic-level))
 
-(defun gnus-topic-init-alist ()
-  "Initialize the topic structures."
-  (setq gnus-topic-topology
-       (cons (list "Gnus" 'visible)
-             (mapcar (lambda (topic)
-                       (list (list (car topic) 'visible)))
-                     '(("misc")))))
-  (setq gnus-topic-alist
-       (list (cons "misc"
-                   (mapcar (lambda (info) (gnus-info-group info))
-                           (cdr gnus-newsrc-alist)))
-             (list "Gnus")))
-  (gnus-topic-enter-dribble))
+(defun gnus-group-topic-unread ()
+  "The number of unread articles in topic on the current line."
+  (get-text-property (gnus-point-at-bol) 'gnus-topic-unread))
+
+(defun gnus-topic-unread (topic)
+  "Return the number of unread articles in TOPIC."
+  (or (save-excursion
+       (and (gnus-topic-goto-topic topic)
+            (gnus-group-topic-unread)))
+      0))
+
+(defun gnus-group-topic-p ()
+  "Return non-nil if the current line is a topic."
+  (gnus-group-topic-name))
+
+(defun gnus-topic-visible-p ()
+  "Return non-nil if the current topic is visible."
+  (get-text-property (gnus-point-at-bol) 'gnus-topic-visible))
+
+(defun gnus-topic-articles-in-topic (entries)
+  (let ((total 0)
+       number)
+    (while entries
+      (when (numberp (setq number (car (pop entries))))
+       (incf total number)))
+    total))
+
+(defun gnus-group-topic (group)
+  "Return the topic GROUP is a member of."
+  (let ((alist gnus-topic-alist)
+       out)
+    (while alist
+      (when (member group (cdar alist))
+       (setq out (caar alist)
+             alist nil))
+      (setq alist (cdr alist)))
+    out))
+
+(defun gnus-group-parent-topic (group)
+  "Return the topic GROUP is member of by looking at the group buffer."
+  (save-excursion
+    (set-buffer gnus-group-buffer)
+    (if (gnus-group-goto-group group)
+       (gnus-current-topic)
+      (gnus-group-topic group))))
+
+(defun gnus-topic-goto-topic (topic)
+  "Go to TOPIC."
+  (when topic
+    (gnus-goto-char (text-property-any (point-min) (point-max)
+                                      'gnus-topic (intern topic)))))
+
+(defun gnus-current-topic ()
+  "Return the name of the current topic."
+  (let ((result
+        (or (get-text-property (point) 'gnus-topic)
+            (save-excursion
+              (and (gnus-goto-char (previous-single-property-change
+                                    (point) 'gnus-topic))
+                   (get-text-property (max (1- (point)) (point-min))
+                                      'gnus-topic))))))
+    (when result
+      (symbol-name result))))
+
+(defun gnus-current-topics ()
+  "Return a list of all current topics, lowest in hierarchy first."
+  (let ((topic (gnus-current-topic))
+       topics)
+    (while topic
+      (push topic topics)
+      (setq topic (gnus-topic-parent-topic topic)))
+    (nreverse topics)))
+
+(defun gnus-group-active-topic-p ()
+  "Say whether the current topic comes from the active topics."
+  (save-excursion
+    (beginning-of-line)
+    (get-text-property (point) 'gnus-active)))
+
+(defun gnus-topic-find-groups (topic &optional level all)
+  "Return entries for all visible groups in TOPIC."
+  (let ((groups (cdr (assoc topic gnus-topic-alist)))
+        info clevel unread group lowest params visible-groups entry active)
+    (setq lowest (or lowest 1))
+    (setq level (or level 7))
+    ;; We go through the newsrc to look for matches.
+    (while groups
+      (setq entry (gnus-gethash (setq group (pop groups)) gnus-newsrc-hashtb)
+           info (nth 2 entry)
+           params (gnus-info-params info)
+           active (gnus-active group)
+            unread (or (car entry)
+                      (and (not (equal group "dummy.group"))
+                           active
+                           (- (1+ (cdr active)) (car active))))
+           clevel (or (gnus-info-level info)
+                      (if (member group gnus-zombie-list) 8 9)))
+      (and 
+       unread                          ; nil means that the group is dead.
+       (<= clevel level) 
+       (>= clevel lowest)              ; Is inside the level we want.
+       (or all
+          (if (eq unread t)
+              gnus-group-list-inactive-groups
+            (> unread 0))
+          (and gnus-list-groups-with-ticked-articles
+               (cdr (assq 'tick (gnus-info-marks info))))
+                                       ; Has right readedness.
+          ;; Check for permanent visibility.
+          (and gnus-permanently-visible-groups
+               (string-match gnus-permanently-visible-groups group))
+          (memq 'visible params)
+          (cdr (assq 'visible params)))
+       ;; Add this group to the list of visible groups.
+       (push (or entry group) visible-groups)))
+    (nreverse visible-groups)))
+
+(defun gnus-topic-previous-topic (topic)
+  "Return the previous topic on the same level as TOPIC."
+  (let ((top (cddr (gnus-topic-find-topology
+                   (gnus-topic-parent-topic topic)))))
+    (unless (equal topic (caaar top))
+      (while (and top (not (equal (caaadr top) topic)))
+       (setq top (cdr top)))
+      (caaar top))))
+
+(defun gnus-topic-parent-topic (topic &optional topology)
+  "Return the parent of TOPIC."
+  (unless topology
+    (setq topology gnus-topic-topology))
+  (let ((parent (car (pop topology)))
+       result found)
+    (while (and topology
+               (not (setq found (equal (caaar topology) topic)))
+               (not (setq result (gnus-topic-parent-topic topic 
+                                                          (car topology)))))
+      (setq topology (cdr topology)))
+    (or result (and found parent))))
+
+(defun gnus-topic-next-topic (topic &optional previous)
+  "Return the next sibling of TOPIC."
+  (let ((parentt (cddr (gnus-topic-find-topology 
+                       (gnus-topic-parent-topic topic))))
+       prev)
+    (while (and parentt
+               (not (equal (caaar parentt) topic)))
+      (setq prev (caaar parentt)
+           parentt (cdr parentt)))
+    (if previous
+       prev
+      (caaadr parentt))))
+
+(defun gnus-topic-find-topology (topic &optional topology level remove)
+  "Return the topology of TOPIC."
+  (unless topology
+    (setq topology gnus-topic-topology)
+    (setq level 0))
+  (let ((top topology)
+       result)
+    (if (equal (caar topology) topic)
+       (progn
+         (when remove
+           (delq topology remove))
+         (cons level topology))
+      (setq topology (cdr topology))
+      (while (and topology
+                 (not (setq result (gnus-topic-find-topology
+                                    topic (car topology) (1+ level)
+                                    (and remove top)))))
+       (setq topology (cdr topology)))
+      result)))
+
+(defvar gnus-tmp-topics nil)
+(defun gnus-topic-list (&optional topology)
+  "Return a list of all topics in the topology."
+  (unless topology
+    (setq topology gnus-topic-topology 
+         gnus-tmp-topics nil))
+  (push (caar topology) gnus-tmp-topics)
+  (mapcar 'gnus-topic-list (cdr topology))
+  gnus-tmp-topics)
+
+;;; Topic parameter jazz
+
+(defun gnus-topic-parameters (topic)
+  "Return the parameters for TOPIC."
+  (let ((top (gnus-topic-find-topology topic)))
+    (unless top
+      (error "No such topic: %s" topic))
+    (nth 3 (cadr top))))
+
+(defun gnus-topic-set-parameters (topic parameters)
+  "Set the topic parameters of TOPIC to PARAMETERS."
+  (let ((top (gnus-topic-find-topology topic)))
+    (unless top
+      (error "No such topic: %s" topic))
+    ;; We may have to extend if there is no parameters here
+    ;; to begin with.
+    (unless (nthcdr 2 (car top))
+      (nconc (car top) (list nil)))
+    (unless (nthcdr 3 (car top))
+      (nconc (car top) (list nil)))
+    (setcar (nthcdr 3 (car top)) parameters)))
+
+(defun gnus-group-topic-parameters (group)
+  "Compute the group parameters for GROUP taking into account inheretance from topics."
+  (let ((params-list (list (gnus-group-get-parameter group)))
+       topics params param out)
+    (save-excursion
+      (gnus-group-goto-group group)
+      (setq topics (gnus-current-topics))
+      (while topics
+       (push (gnus-topic-parameters (pop topics)) params-list))
+      ;; We probably have lots of nil elements here, so
+      ;; we remove them.  Probably faster than doing this "properly".
+      (setq params-list (delq nil params-list))
+      ;; Now we have all the parameters, so we go through them
+      ;; and do inheretance in the obvious way.
+      (while (setq params (pop params-list))
+       (while (setq param (pop params))
+         (when (atom param)
+           (setq param (cons param t)))
+         ;; Override any old versions of this param.
+         (setq out (delq (assq (car param) out) out))
+         (push param out)))
+      ;; Return the resulting parameter list.
+      out)))
+
+;;; General utility funtions
+
+(defun gnus-topic-enter-dribble ()
+  (gnus-dribble-enter
+   (format "(setq gnus-topic-topology '%S)" gnus-topic-topology)))
+
+;;; Generating group buffers
 
 (defun gnus-group-prepare-topics (level &optional all lowest regexp list-topic topic-level)
   "List all newsgroups with unread articles of level LEVEL or lower, and
 
 (defun gnus-group-prepare-topics (level &optional all lowest regexp list-topic topic-level)
   "List all newsgroups with unread articles of level LEVEL or lower, and
@@ -102,10 +333,12 @@ If ALL is non-nil, list groups that have no unread articles.
 If LOWEST is non-nil, list all newsgroups of level LOWEST or higher."
   (set-buffer gnus-group-buffer)
   (let ((buffer-read-only nil)
 If LOWEST is non-nil, list all newsgroups of level LOWEST or higher."
   (set-buffer gnus-group-buffer)
   (let ((buffer-read-only nil)
-        (lowest (or lowest 1))
-       tlist info)
+        (lowest (or lowest 1)))
 
 
-    (unless gnus-topic-alist
+    (setq gnus-topic-tallied-groups nil)
+
+    (when (or (not gnus-topic-alist)
+             (not gnus-topology-checked-p))
       (gnus-topic-check-topology))
 
     (unless list-topic 
       (gnus-topic-check-topology))
 
     (unless list-topic 
@@ -124,7 +357,6 @@ If LOWEST is non-nil, list all newsgroups of level LOWEST or higher."
        gnus-level-killed ?K
        regexp))
 
        gnus-level-killed ?K
        regexp))
 
-    (setq gnus-topic-tallied-groups nil)
     ;; Use topics.
     (when (< lowest gnus-level-zombie)
       (if list-topic
     ;; Use topics.
     (when (< lowest gnus-level-zombie)
       (if list-topic
@@ -138,25 +370,26 @@ If LOWEST is non-nil, list all newsgroups of level LOWEST or higher."
   (setq gnus-group-list-mode (cons level all))
   (run-hooks 'gnus-group-prepare-hook))
 
   (setq gnus-group-list-mode (cons level all))
   (run-hooks 'gnus-group-prepare-hook))
 
-(defun gnus-topic-prepare-topic (topic level &optional list-level all silent)
+(defun gnus-topic-prepare-topic (topicl level &optional list-level all silent)
   "Insert TOPIC into the group buffer.
 If SILENT, don't insert anything.  Return the number of unread
 articles in the topic and its subtopics."
   "Insert TOPIC into the group buffer.
 If SILENT, don't insert anything.  Return the number of unread
 articles in the topic and its subtopics."
-  (let* ((type (pop topic))
+  (let* ((type (pop topicl))
         (entries (gnus-topic-find-groups (car type) list-level all))
         (visiblep (and (eq (nth 1 type) 'visible) (not silent)))
         (gnus-group-indentation 
          (make-string (* gnus-topic-indent-level level) ? ))
         (beg (progn (beginning-of-line) (point)))
         (entries (gnus-topic-find-groups (car type) list-level all))
         (visiblep (and (eq (nth 1 type) 'visible) (not silent)))
         (gnus-group-indentation 
          (make-string (* gnus-topic-indent-level level) ? ))
         (beg (progn (beginning-of-line) (point)))
-        (topic (reverse topic))
+        (topicl (reverse topicl))
         (all-entries entries)
         (unread 0)
         (all-entries entries)
         (unread 0)
+        (topic (car type))
         info entry end active)
     ;; Insert any sub-topics.
         info entry end active)
     ;; Insert any sub-topics.
-    (while topic
+    (while topicl
       (incf unread
            (gnus-topic-prepare-topic 
       (incf unread
            (gnus-topic-prepare-topic 
-            (pop topic) (1+ level) list-level all
+            (pop topicl) (1+ level) list-level all
             (not visiblep))))
     (setq end (point))
     (goto-char beg)
             (not visiblep))))
     (setq end (point))
     (goto-char beg)
@@ -177,13 +410,15 @@ articles in the topic and its subtopics."
             (car entry) (gnus-info-method info)))))
       (when (and (listp entry)
                 (numberp (car entry))
             (car entry) (gnus-info-method info)))))
       (when (and (listp entry)
                 (numberp (car entry))
-                (not (member (gnus-info-group info)
+                (not (member (gnus-info-group (setq info (nth 2 entry)))
                              gnus-topic-tallied-groups)))
        (push (gnus-info-group info) gnus-topic-tallied-groups)
        (incf unread (car entry))))
     (goto-char beg)
     ;; Insert the topic line.
                              gnus-topic-tallied-groups)))
        (push (gnus-info-group info) gnus-topic-tallied-groups)
        (incf unread (car entry))))
     (goto-char beg)
     ;; Insert the topic line.
-    (unless silent
+    (when (and (not silent)
+              (or gnus-topic-display-empty-topics
+                  (not (zerop unread))))
       (gnus-extent-start-open (point))
       (gnus-topic-insert-topic-line 
        (car type) visiblep
       (gnus-extent-start-open (point))
       (gnus-topic-insert-topic-line 
        (car type) visiblep
@@ -192,44 +427,6 @@ articles in the topic and its subtopics."
     (goto-char end)
     unread))
 
     (goto-char end)
     unread))
 
-(defun gnus-topic-find-groups (topic &optional level all)
-  "Return entries for all visible groups in TOPIC."
-  (let ((groups (cdr (assoc topic gnus-topic-alist)))
-        info clevel unread group lowest params visible-groups entry active)
-    (setq lowest (or lowest 1))
-    (setq level (or level 7))
-    ;; We go through the newsrc to look for matches.
-    (while groups
-      (setq entry (gnus-gethash (setq group (pop groups)) gnus-newsrc-hashtb)
-           info (nth 2 entry)
-           params (gnus-info-params info)
-           active (gnus-active group)
-            unread (or (car entry)
-                      (and (not (equal group "dummy.group"))
-                           active
-                           (- (1+ (cdr active)) (car active))))
-           clevel (or (gnus-info-level info)
-                      (if (member group gnus-zombie-list) 8 9)))
-      (and 
-       unread                          ; nil means that the group is dead.
-       (<= clevel level) 
-       (>= clevel lowest)              ; Is inside the level we want.
-       (or all
-          (and gnus-group-list-inactive-groups
-               (eq unread t))
-          (> unread 0)
-          (and gnus-list-groups-with-ticked-articles
-               (cdr (assq 'tick (gnus-info-marks info))))
-                                       ; Has right readedness.
-          ;; Check for permanent visibility.
-          (and gnus-permanently-visible-groups
-               (string-match gnus-permanently-visible-groups group))
-          (memq 'visible params)
-          (cdr (assq 'visible params)))
-       ;; Add this group to the list of visible groups.
-       (push (or entry group) visible-groups)))
-    (nreverse visible-groups)))
-
 (defun gnus-topic-remove-topic (&optional insert total-remove hide in-level)
   "Remove the current topic."
   (let ((topic (gnus-group-topic-name))
 (defun gnus-topic-remove-topic (&optional insert total-remove hide in-level)
   "Remove the current topic."
   (let ((topic (gnus-group-topic-name))
@@ -268,14 +465,6 @@ articles in the topic and its subtopics."
            (gnus-topic-remove-topic
             (or insert (not (gnus-topic-visible-p))) nil nil 9)))))))
 
            (gnus-topic-remove-topic
             (or insert (not (gnus-topic-visible-p))) nil nil 9)))))))
 
-(defun gnus-group-topic-p ()
-  "Return non-nil if the current line is a topic."
-  (get-text-property (gnus-point-at-bol) 'gnus-topic))
-
-(defun gnus-topic-visible-p ()
-  "Return non-nil if the current topic is visible."
-  (get-text-property (gnus-point-at-bol) 'gnus-topic-visible))
-
 (defun gnus-topic-insert-topic-line (name visiblep shownp level entries 
                                          &optional unread)
   (let* ((visible (if visiblep "" "..."))
 (defun gnus-topic-insert-topic-line (name visiblep shownp level entries 
                                          &optional unread)
   (let* ((visible (if visiblep "" "..."))
@@ -285,141 +474,26 @@ articles in the topic and its subtopics."
         (active-topic (eq gnus-topic-alist gnus-topic-active-alist)))
     (beginning-of-line)
     ;; Insert the text.
         (active-topic (eq gnus-topic-alist gnus-topic-active-alist)))
     (beginning-of-line)
     ;; Insert the text.
-    (add-text-properties 
+    (gnus-add-text-properties 
      (point)
      (prog1 (1+ (point)) 
        (eval gnus-topic-line-format-spec)
      (point)
      (prog1 (1+ (point)) 
        (eval gnus-topic-line-format-spec)
-       (gnus-topic-remove-excess-properties))
-     (list 'gnus-topic name
+       (gnus-topic-remove-excess-properties)1)
+     (list 'gnus-topic (intern name)
           'gnus-topic-level level
           'gnus-topic-level level
+          'gnus-topic-unread unread
           'gnus-active active-topic
           'gnus-topic-visible visiblep))))
 
           'gnus-active active-topic
           'gnus-topic-visible visiblep))))
 
-(defun gnus-topic-previous-topic (topic)
-  "Return the previous topic on the same level as TOPIC."
-  (let ((top (cddr (gnus-topic-find-topology
-                       (gnus-topic-parent-topic topic)))))
-    (unless (equal topic (caaar top))
-      (while (and top (not (equal (caaadr top) topic)))
-       (setq top (cdr top)))
-      (caaar top))))
-
-(defun gnus-topic-parent-topic (topic &optional topology)
-  "Return the parent of TOPIC."
-  (unless topology
-    (setq topology gnus-topic-topology))
-  (let ((parent (car (pop topology)))
-       result found)
-    (while (and topology
-               (not (setq found (equal (caaar topology) topic)))
-               (not (setq result (gnus-topic-parent-topic topic 
-                                                          (car topology)))))
-      (setq topology (cdr topology)))
-    (or result (and found parent))))
-
-(defun gnus-topic-find-topology (topic &optional topology level remove)
-  "Return the topology of TOPIC."
-  (unless topology
-    (setq topology gnus-topic-topology)
-    (setq level 0))
-  (let ((top topology)
-       result)
-    (if (equal (caar topology) topic)
-       (progn
-         (when remove
-           (delq topology remove))
-         (cons level topology))
-      (setq topology (cdr topology))
-      (while (and topology
-                 (not (setq result (gnus-topic-find-topology
-                                    topic (car topology) (1+ level)
-                                    (and remove top)))))
-       (setq topology (cdr topology)))
-      result)))
-
-(defun gnus-topic-check-topology ()  
-  ;; The first time we set the topology to whatever we have
-  ;; gotten here, which can be rather random.
-  (unless gnus-topic-alist
-    (gnus-topic-init-alist))
-
-  (let ((topics (gnus-topic-list))
-       (alist gnus-topic-alist)
-       changed)
-    (while alist
-      (unless (member (caar alist) topics)
-       (nconc gnus-topic-topology
-              (list (list (list (caar alist) 'visible))))
-       (setq changed t))
-      (setq alist (cdr alist)))
-    (when changed
-      (gnus-topic-enter-dribble)))
-  (let* ((tgroups (apply 'append (mapcar (lambda (entry) (cdr entry))
-                                        gnus-topic-alist)))
-        (entry (assoc (caar gnus-topic-topology) gnus-topic-alist))
-        (newsrc gnus-newsrc-alist)
-        group)
-    (while newsrc
-      (unless (member (setq group (gnus-info-group (pop newsrc))) tgroups)
-       (setcdr entry (cons group (cdr entry)))))))
-
-(defvar gnus-tmp-topics nil)
-(defun gnus-topic-list (&optional topology)
-  (unless topology
-    (setq topology gnus-topic-topology 
-         gnus-tmp-topics nil))
-  (push (caar topology) gnus-tmp-topics)
-  (mapcar 'gnus-topic-list (cdr topology))
-  gnus-tmp-topics)
-
-(defun gnus-topic-enter-dribble ()
-  (gnus-dribble-enter
-   (format "(setq gnus-topic-topology '%S)" gnus-topic-topology)))
-
-(defun gnus-topic-articles-in-topic (entries)
-  (let ((total 0)
-       number)
-    (while entries
-      (when (numberp (setq number (car (pop entries))))
-       (incf total number)))
-    total))
-
-(defun gnus-group-parent-topic ()
-  "Return the topic the current group belongs in."
-  (let ((group (gnus-group-group-name)))
-    (if group
-       (gnus-group-topic group)
-      (gnus-group-topic-name))))
-
-(defun gnus-group-topic (group)
-  "Return the topic GROUP is a member of."
-  (let ((alist gnus-topic-alist)
-       out)
-    (while alist
-      (when (member group (cdar alist))
-       (setq out (caar alist)
-             alist nil))
-      (setq alist (cdr alist)))
-    out))
-
-(defun gnus-topic-goto-topic (topic)
-  (let ((orig (point)))
-    (goto-char (point-min))
-    (while (and (not (equal topic (gnus-group-topic-name)))
-               (zerop (forward-line 1))))
-    (or (gnus-group-topic-name)
-       (progn
-         (goto-char orig)
-         nil))))
-  
 (defun gnus-topic-update-topic ()
   "Update all parent topics to the current group."
   (when (and (eq major-mode 'gnus-group-mode)
             gnus-topic-mode)
 (defun gnus-topic-update-topic ()
   "Update all parent topics to the current group."
   (when (and (eq major-mode 'gnus-group-mode)
             gnus-topic-mode)
-    (let ((group (gnus-group-group-name)))
-      (when (and group (gnus-get-info group))
-       (gnus-topic-goto-topic (gnus-group-parent-topic))
-       (gnus-topic-update-topic-line)
+    (let ((group (gnus-group-group-name))
+         (buffer-read-only nil))
+      (when (and group (gnus-get-info group)
+                (gnus-topic-goto-topic (gnus-current-topic)))
+       (gnus-topic-update-topic-line (gnus-group-topic-name))
        (gnus-group-goto-group group)
        (gnus-group-position-point)))))
 
        (gnus-group-goto-group group)
        (gnus-group-position-point)))))
 
@@ -429,68 +503,239 @@ articles in the topic and its subtopics."
         (groups (cdr (assoc topic gnus-topic-alist)))
         (g (cdr (member group groups)))
         (unfound t))
         (groups (cdr (assoc topic gnus-topic-alist)))
         (g (cdr (member group groups)))
         (unfound t))
-    (while (and g unfound)
-      (when (gnus-group-goto-group (pop g))
-       (beginning-of-line)
-       (setq unfound nil)))
-    (when unfound
+    ;; Try to jump to a visible group.
+    (while (and g (not (gnus-group-goto-group (car g) t)))
+      (pop g))
+    ;; It wasn't visible, so we try to see where to insert it.
+    (when (not g)
       (setq g (cdr (member group (reverse groups))))
       (while (and g unfound)
       (setq g (cdr (member group (reverse groups))))
       (while (and g unfound)
-       (when (gnus-group-goto-group (pop g))
+       (when (gnus-group-goto-group (pop g) t)
          (forward-line 1)
          (setq unfound nil)))
       (when unfound
        (gnus-topic-goto-topic topic)
        (forward-line 1)))))
 
          (forward-line 1)
          (setq unfound nil)))
       (when unfound
        (gnus-topic-goto-topic topic)
        (forward-line 1)))))
 
-(defun gnus-topic-update-topic-line (&optional topic level)
-  (unless topic
-    (setq topic gnus-topic-topology)
-    (setq level 0))
-  (let* ((type (pop topic))
-        (buffer-read-only nil)
+(defun gnus-topic-update-topic-line (topic-name &optional reads)
+  (let* ((top (gnus-topic-find-topology topic-name))
+        (type (cadr top))
+        (children (cddr top))
         (entries (gnus-topic-find-groups 
                   (car type) (car gnus-group-list-mode)
                   (cdr gnus-group-list-mode)))
         (entries (gnus-topic-find-groups 
                   (car type) (car gnus-group-list-mode)
                   (cdr gnus-group-list-mode)))
-        (visiblep (eq (nth 1 type) 'visible))
+        (parent (gnus-topic-parent-topic topic-name))
         (all-entries entries)
         (unread 0)
         (all-entries entries)
         (unread 0)
-        info entry end)
-    ;; Tally any sub-topics.
-    (while topic
-      (incf unread (gnus-topic-update-topic-line (pop topic) (1+ level))))
-    ;; Tally all the groups that belong in this topic.
-    (while (setq info (nth 2 (setq entry (pop entries))))
-      (when (numberp (car entry))
-       (incf unread (car entry))))
-    ;; Insert the topic line.
+        old-unread entry)
     (when (gnus-topic-goto-topic (car type))
     (when (gnus-topic-goto-topic (car type))
+      ;; Tally all the groups that belong in this topic.
+      (if reads
+         (setq unread (- (gnus-group-topic-unread) reads))
+       (while children
+         (incf unread (gnus-topic-unread (caar (pop children)))))
+       (while (setq entry (pop entries))
+         (when (numberp (car entry))
+           (incf unread (car entry)))))
+      (setq old-unread (gnus-group-topic-unread))
+      ;; Insert the topic line.
       (gnus-topic-insert-topic-line 
       (gnus-topic-insert-topic-line 
-       (car type) visiblep
+       (car type) (gnus-topic-visible-p)
        (not (eq (nth 2 type) 'hidden))
        (not (eq (nth 2 type) 'hidden))
-       level all-entries unread)
+       (gnus-group-topic-level) all-entries unread)
       (gnus-delete-line))
       (gnus-delete-line))
+    (when parent
+      (forward-line -1)
+      (gnus-topic-update-topic-line
+       parent (- old-unread (gnus-group-topic-unread))))
     unread))
 
     unread))
 
-(defun gnus-topic-grok-active (&optional force read-active)
+(defun gnus-topic-group-indentation ()
+  (make-string 
+   (* gnus-topic-indent-level
+      (or (save-excursion
+           (gnus-topic-goto-topic (gnus-current-topic))
+           (gnus-group-topic-level)) 0)) ? ))
+
+;;; Initialization
+
+(gnus-add-shutdown 'gnus-topic-close 'gnus)
+
+(defun gnus-topic-close ()
+  (setq gnus-topic-active-topology nil
+       gnus-topic-active-alist nil
+       gnus-topic-killed-topics nil
+       gnus-topic-tallied-groups nil
+       gnus-topology-checked-p nil))
+
+(defun gnus-topic-check-topology ()  
+  ;; The first time we set the topology to whatever we have
+  ;; gotten here, which can be rather random.
+  (unless gnus-topic-alist
+    (gnus-topic-init-alist))
+
+  (setq gnus-topology-checked-p t)
+  ;; Go through the topic alist and make sure that all topics
+  ;; are in the topic topology.
+  (let ((topics (gnus-topic-list))
+       (alist gnus-topic-alist)
+       changed)
+    (while alist
+      (unless (member (caar alist) topics)
+       (nconc gnus-topic-topology
+              (list (list (list (caar alist) 'visible))))
+       (setq changed t))
+      (setq alist (cdr alist)))
+    (when changed
+      (gnus-topic-enter-dribble))
+    ;; Conversely, go through the topology and make sure that all
+    ;; topologies have alists.
+    (while topics
+      (unless (assoc (car topics) gnus-topic-alist)
+       (push (list (car topics)) gnus-topic-alist))
+      (pop topics)))
+  ;; Go through all living groups and make sure that
+  ;; they belong to some topic.
+  (let* ((tgroups (apply 'append (mapcar (lambda (entry) (cdr entry))
+                                        gnus-topic-alist)))
+        (entry (assoc (caar gnus-topic-topology) gnus-topic-alist))
+        (newsrc gnus-newsrc-alist)
+        group)
+    (while newsrc
+      (unless (member (setq group (gnus-info-group (pop newsrc))) tgroups)
+       (setcdr entry (cons group (cdr entry))))))
+  ;; Go through all topics and make sure they contain only living groups.
+  (let ((alist gnus-topic-alist)
+       topic)
+    (while (setq topic (pop alist))
+      (while (cdr topic)
+       (if (gnus-gethash (cadr topic) gnus-newsrc-hashtb)
+           (setq topic (cdr topic))
+         (setcdr topic (cddr topic)))))))
+
+(defun gnus-topic-init-alist ()
+  "Initialize the topic structures."
+  (setq gnus-topic-topology
+       (cons (list "Gnus" 'visible)
+             (mapcar (lambda (topic)
+                       (list (list (car topic) 'visible)))
+                     '(("misc")))))
+  (setq gnus-topic-alist
+       (list (cons "misc"
+                   (mapcar (lambda (info) (gnus-info-group info))
+                           (cdr gnus-newsrc-alist)))
+             (list "Gnus")))
+  (gnus-topic-enter-dribble))
+
+;;; Maintenance
+
+(defun gnus-topic-clean-alist ()
+  "Remove bogus groups from the topic alist."
+  (let ((topic-alist gnus-topic-alist)
+       result topic)
+    (unless gnus-killed-hashtb
+      (gnus-make-hashtable-from-killed))
+    (while (setq topic (pop topic-alist))
+      (let ((topic-name (pop topic))
+           group filtered-topic)
+       (while (setq group (pop topic))
+         (if (and (gnus-gethash group gnus-active-hashtb)
+                  (not (gnus-gethash group gnus-killed-hashtb)))
+             (push group filtered-topic)))
+       (push (cons topic-name (nreverse filtered-topic)) result)))
+    (setq gnus-topic-alist (nreverse result))))
+
+(defun gnus-topic-change-level (group level oldlevel)
+  "Run when changing levels to enter/remove groups from topics."
+  (save-excursion
+    (set-buffer gnus-group-buffer)
+    (when (and gnus-topic-mode 
+              gnus-topic-alist
+              (not gnus-topic-inhibit-change-level))
+      ;; Remove the group from the topics.
+      (when (and (< oldlevel gnus-level-zombie)
+                (>= level gnus-level-zombie))
+       (let (alist)
+         (forward-line -1)
+         (when (setq alist (assoc (gnus-current-topic) gnus-topic-alist))
+           (setcdr alist (gnus-delete-first group (cdr alist))))))
+      ;; If the group is subscribed. then we enter it into the topics.
+      (when (and (< level gnus-level-zombie)
+                (>= oldlevel gnus-level-zombie))
+       (let* ((prev (gnus-group-group-name))
+              (gnus-topic-inhibit-change-level t)
+              (gnus-group-indentation
+               (make-string 
+                (* gnus-topic-indent-level
+                   (or (save-excursion
+                         (gnus-topic-goto-topic (gnus-current-topic))
+                         (gnus-group-topic-level)) 0)) ? ))
+              (yanked (list group))
+              alist talist end)
+         ;; Then we enter the yanked groups into the topics they belong
+         ;; to. 
+         (when (setq alist (assoc (save-excursion
+                                    (forward-line -1)
+                                    (or
+                                     (gnus-current-topic)
+                                     (caar gnus-topic-topology)))
+                                  gnus-topic-alist))
+           (setq talist alist)
+           (when (stringp yanked)
+             (setq yanked (list yanked)))
+           (if (not prev)
+               (nconc alist yanked)
+             (if (not (cdr alist))
+                 (setcdr alist (nconc yanked (cdr alist)))
+               (while (and (not end) (cdr alist))
+                 (when (equal (cadr alist) prev)
+                   (setcdr alist (nconc yanked (cdr alist)))
+                   (setq end t))
+                 (setq alist (cdr alist)))
+               (unless end
+                 (nconc talist yanked))))))
+       (gnus-topic-update-topic)))))
+
+(defun gnus-topic-goto-next-group (group props)
+  "Go to group or the next group after group."
+  (if (null group)
+      (gnus-topic-goto-topic (symbol-name (cadr (memq 'gnus-topic props))))
+    (if (gnus-group-goto-group group)
+       t
+      ;; The group is no longer visible.
+      (let* ((list (assoc (gnus-current-topic) gnus-topic-alist))
+            (after (cdr (member group (cdr list)))))
+       ;; First try to put point on a group after the current one.
+       (while (and after
+                   (not (gnus-group-goto-group (car after))))
+         (setq after (cdr after)))
+       ;; Then try to put point on a group before point.
+       (unless after
+         (setq after (cdr (member group (reverse (cdr list)))))
+         (while (and after 
+                     (not (gnus-group-goto-group (car after))))
+           (setq after (cdr after))))
+       ;; Finally, just put point on the topic.
+       (unless after
+         (gnus-topic-goto-topic (car list))
+         (setq after nil))
+       t))))
+
+;;; Topic-active functions
+
+(defun gnus-topic-grok-active (&optional force)
   "Parse all active groups and create topic structures for them."
   ;; First we make sure that we have really read the active file. 
   (when (or force
            (not gnus-topic-active-alist))
   "Parse all active groups and create topic structures for them."
   ;; First we make sure that we have really read the active file. 
   (when (or force
            (not gnus-topic-active-alist))
-    (when (and read-active
-              (or force
-                  (not (member gnus-select-method 
-                               gnus-have-read-active-file))))
-      (let ((gnus-read-active-file t))
-       (gnus-read-active-file)))
-    (let (topology groups alist)
+    (let (groups)
       ;; Get a list of all groups available.
       (mapatoms (lambda (g) (when (symbol-value g)
                              (push (symbol-name g) groups)))
                gnus-active-hashtb)
       (setq groups (sort groups 'string<))
       ;; Init the variables.
       ;; Get a list of all groups available.
       (mapatoms (lambda (g) (when (symbol-value g)
                              (push (symbol-name g) groups)))
                gnus-active-hashtb)
       (setq groups (sort groups 'string<))
       ;; Init the variables.
-      (setq gnus-topic-active-topology '(("" visible)))
+      (setq gnus-topic-active-topology (list (list "" 'visible)))
       (setq gnus-topic-active-alist nil)
       ;; Descend the top-level hierarchy.
       (gnus-topic-grok-active-1 gnus-topic-active-topology groups)
       (setq gnus-topic-active-alist nil)
       ;; Descend the top-level hierarchy.
       (gnus-topic-grok-active-1 gnus-topic-active-topology groups)
@@ -501,7 +746,7 @@ articles in the topic and its subtopics."
 (defun gnus-topic-grok-active-1 (topology groups)
   (let* ((name (caar topology))
         (prefix (concat "^" (regexp-quote name)))
 (defun gnus-topic-grok-active-1 (topology groups)
   (let* ((name (caar topology))
         (prefix (concat "^" (regexp-quote name)))
-        tgroups nprefix ntopology group)
+        tgroups ntopology group)
     (while (and groups
                (string-match prefix (setq group (car groups))))
       (if (not (string-match "\\." group (match-end 0)))
     (while (and groups
                (string-match prefix (setq group (car groups))))
       (if (not (string-match "\\." group (match-end 0)))
@@ -528,12 +773,6 @@ articles in the topic and its subtopics."
     ;; to this topic.
     groups))
 
     ;; to this topic.
     groups))
 
-(defun gnus-group-active-topic-p ()
-  "Return whether the current active comes from the active topics."
-  (save-excursion
-    (beginning-of-line)
-    (get-text-property (point) 'gnus-active)))
-
 ;;; Topic mode, commands and keymap.
 
 (defvar gnus-topic-mode-map nil)
 ;;; Topic mode, commands and keymap.
 
 (defvar gnus-topic-mode-map nil)
@@ -551,8 +790,8 @@ articles in the topic and its subtopics."
    "\C-k" gnus-topic-kill-group
    "\C-y" gnus-topic-yank-group
    "\M-g" gnus-topic-get-new-news-this-topic
    "\C-k" gnus-topic-kill-group
    "\C-y" gnus-topic-yank-group
    "\M-g" gnus-topic-get-new-news-this-topic
-   "\C-i" gnus-topic-indent
    "AT" gnus-topic-list-active
    "AT" gnus-topic-list-active
+   "Gp" gnus-topic-edit-parameters
    gnus-mouse-2 gnus-mouse-pick-topic)
 
   ;; Define a new submap.
    gnus-mouse-2 gnus-mouse-pick-topic)
 
   ;; Define a new submap.
@@ -568,6 +807,8 @@ articles in the topic and its subtopics."
    "s" gnus-topic-show-topic
    "M" gnus-topic-move-matching
    "C" gnus-topic-copy-matching
    "s" gnus-topic-show-topic
    "M" gnus-topic-move-matching
    "C" gnus-topic-copy-matching
+   "\C-i" gnus-topic-indent
+   [tab] gnus-topic-indent
    "r" gnus-topic-rename
    "\177" gnus-topic-delete))
 
    "r" gnus-topic-rename
    "\177" gnus-topic-delete))
 
@@ -616,21 +857,30 @@ articles in the topic and its subtopics."
              minor-mode-map-alist))
       (add-hook 'gnus-summary-exit-hook 'gnus-topic-update-topic)
       (add-hook 'gnus-group-catchup-group-hook 'gnus-topic-update-topic)
              minor-mode-map-alist))
       (add-hook 'gnus-summary-exit-hook 'gnus-topic-update-topic)
       (add-hook 'gnus-group-catchup-group-hook 'gnus-topic-update-topic)
-      (make-local-variable 'gnus-group-prepare-function)
-      (setq gnus-group-prepare-function 'gnus-group-prepare-topics)
-      (make-local-variable 'gnus-group-goto-next-group-function)
-      (setq gnus-group-goto-next-group-function 
-           'gnus-topic-goto-next-group)
+      (add-hook 'gnus-group-update-group-hook 'gnus-topic-update-topic)
+      (set (make-local-variable 'gnus-group-prepare-function)
+          'gnus-group-prepare-topics)
+      (set (make-local-variable 'gnus-group-get-parameter-function)
+          'gnus-group-topic-parameters)
+      (set (make-local-variable 'gnus-group-goto-next-group-function)
+          'gnus-topic-goto-next-group)
+      (set (make-local-variable 'gnus-group-indentation-function)
+          'gnus-topic-group-indentation)
       (setq gnus-group-change-level-function 'gnus-topic-change-level)
       (setq gnus-goto-missing-group-function 'gnus-topic-goto-missing-group)
       (setq gnus-group-change-level-function 'gnus-topic-change-level)
       (setq gnus-goto-missing-group-function 'gnus-topic-goto-missing-group)
+      (gnus-make-local-hook 'gnus-check-bogus-groups-hook)
+      (add-hook 'gnus-check-bogus-groups-hook 'gnus-topic-clean-alist)
+      (setq gnus-topology-checked-p nil)
       ;; We check the topology.
       ;; We check the topology.
-      (gnus-topic-check-topology)
+      (when gnus-newsrc-alist
+       (gnus-topic-check-topology))
       (run-hooks 'gnus-topic-mode-hook))
     ;; Remove topic infestation.
     (unless gnus-topic-mode
       (remove-hook 'gnus-summary-exit-hook 'gnus-topic-update-topic)
       (remove-hook 'gnus-group-change-level-function 
                   'gnus-topic-change-level)
       (run-hooks 'gnus-topic-mode-hook))
     ;; Remove topic infestation.
     (unless gnus-topic-mode
       (remove-hook 'gnus-summary-exit-hook 'gnus-topic-update-topic)
       (remove-hook 'gnus-group-change-level-function 
                   'gnus-topic-change-level)
+      (remove-hook 'gnus-check-bogus-groups-hook 'gnus-topic-clean-alist)
       (setq gnus-group-prepare-function 'gnus-group-prepare-flat))
     (when redisplay
       (gnus-group-list-groups))))
       (setq gnus-group-prepare-function 'gnus-group-prepare-flat))
     (when redisplay
       (gnus-group-list-groups))))
@@ -639,7 +889,9 @@ articles in the topic and its subtopics."
   "Select this newsgroup.
 No article is selected automatically.
 If ALL is non-nil, already read articles become readable.
   "Select this newsgroup.
 No article is selected automatically.
 If ALL is non-nil, already read articles become readable.
-If ALL is a number, fetch this number of articles."
+If ALL is a number, fetch this number of articles.
+
+If performed over a topic line, toggle folding the topic."
   (interactive "P")
   (if (gnus-group-topic-p)
       (let ((gnus-group-list-mode 
   (interactive "P")
   (if (gnus-group-topic-p)
       (let ((gnus-group-list-mode 
@@ -659,7 +911,9 @@ If the prefix argument ALL is non-nil, already read articles become
 readable.  IF ALL is a number, fetch this number of articles.  If the
 optional argument NO-ARTICLE is non-nil, no article will be
 auto-selected upon group entry.  If GROUP is non-nil, fetch that
 readable.  IF ALL is a number, fetch this number of articles.  If the
 optional argument NO-ARTICLE is non-nil, no article will be
 auto-selected upon group entry.  If GROUP is non-nil, fetch that
-group."
+group.
+
+If performed over a topic line, toggle folding the topic."
   (interactive "P")
   (if (gnus-group-topic-p)
       (let ((gnus-group-list-mode 
   (interactive "P")
   (if (gnus-group-topic-p)
       (let ((gnus-group-list-mode 
@@ -667,18 +921,18 @@ group."
        (gnus-topic-fold all))
     (gnus-group-read-group all no-article group)))
 
        (gnus-topic-fold all))
     (gnus-group-read-group all no-article group)))
 
-(defun gnus-topic-create-topic (topic parent &optional previous)
+(defun gnus-topic-create-topic (topic parent &optional previous full-topic)
   (interactive 
    (list
   (interactive 
    (list
-    (read-string "Create topic: ")
-    (completing-read "Parent topic: " gnus-topic-alist nil t
-                    (cons (gnus-group-parent-topic) 0))))
+    (read-string "New topic: ")
+    (gnus-current-topic)))
   ;; Check whether this topic already exists.
   (when (gnus-topic-find-topology topic)
     (error "Topic aleady exists"))
   (unless parent
     (setq parent (caar gnus-topic-topology)))
   ;; Check whether this topic already exists.
   (when (gnus-topic-find-topology topic)
     (error "Topic aleady exists"))
   (unless parent
     (setq parent (caar gnus-topic-topology)))
-  (let ((top (cdr (gnus-topic-find-topology parent))))
+  (let ((top (cdr (gnus-topic-find-topology parent)))
+       (full-topic (or full-topic `((,topic visible)))))
     (unless top
       (error "No such parent topic: %s" parent))
     (if previous
     (unless top
       (error "No such parent topic: %s" parent))
     (if previous
@@ -686,15 +940,17 @@ group."
          (while (and (cdr top)
                      (not (equal (caaadr top) previous)))
            (setq top (cdr top)))
          (while (and (cdr top)
                      (not (equal (caaadr top) previous)))
            (setq top (cdr top)))
-         (setcdr top (cons (list (list topic 'visible)) (cdr top))))
-      (nconc top (list (list (list topic 'visible)))))
+         (setcdr top (cons full-topic (cdr top))))
+      (nconc top (list full-topic)))
     (unless (assoc topic gnus-topic-alist)
       (push (list topic) gnus-topic-alist)))
   (gnus-topic-enter-dribble)
     (unless (assoc topic gnus-topic-alist)
       (push (list topic) gnus-topic-alist)))
   (gnus-topic-enter-dribble)
-  (gnus-group-list-groups))
+  (gnus-group-list-groups)
+  (gnus-topic-goto-topic topic))
 
 (defun gnus-topic-move-group (n topic &optional copyp)
 
 (defun gnus-topic-move-group (n topic &optional copyp)
-  "Move the current group to a topic."
+  "Move the next N groups to TOPIC.
+If COPYP, copy the groups instead."
   (interactive
    (list current-prefix-arg
         (completing-read "Move to topic: " gnus-topic-alist nil t)))
   (interactive
    (list current-prefix-arg
         (completing-read "Move to topic: " gnus-topic-alist nil t)))
@@ -704,20 +960,26 @@ group."
     (mapcar (lambda (g) 
              (gnus-group-remove-mark g)
              (when (and
     (mapcar (lambda (g) 
              (gnus-group-remove-mark g)
              (when (and
-                    (setq entry (assoc (gnus-group-topic g) gnus-topic-alist))
+                    (setq entry (assoc (gnus-current-topic)
+                                       gnus-topic-alist))
                     (not copyp))
                     (not copyp))
-               (setcdr entry (delete g (cdr entry))))
-             (when topicl
-               (nconc topicl (list g))))
+               (setcdr entry (gnus-delete-first g (cdr entry))))
+             (nconc topicl (list g)))
            groups)
     (gnus-group-position-point))
   (gnus-topic-enter-dribble)
   (gnus-group-list-groups))
 
            groups)
     (gnus-group-position-point))
   (gnus-topic-enter-dribble)
   (gnus-group-list-groups))
 
-(defun gnus-topic-remove-group (n)
-  "Remove the current group the topic."
-  (interactive "P")
-  (gnus-topic-move-group n nil))
+(defun gnus-topic-remove-group ()
+  "Remove the current group from the topic."
+  (interactive)
+  (let ((topicl (assoc (gnus-current-topic) gnus-topic-alist))
+       (group (gnus-group-group-name))
+       (buffer-read-only nil))
+    (when (and topicl group)
+      (gnus-delete-line)
+      (gnus-delete-first group topicl))
+    (gnus-group-position-point)))
 
 (defun gnus-topic-copy-group (n topic)
   "Copy the current group to a topic."
 
 (defun gnus-topic-copy-group (n topic)
   "Copy the current group to a topic."
@@ -726,48 +988,6 @@ group."
         (completing-read "Copy to topic: " gnus-topic-alist nil t)))
   (gnus-topic-move-group n topic t))
 
         (completing-read "Copy to topic: " gnus-topic-alist nil t)))
   (gnus-topic-move-group n topic t))
 
-(defun gnus-topic-change-level (group level oldlevel)
-  "Run when changing levels to enter/remove groups from topics."
-  (when (and gnus-topic-mode 
-            gnus-topic-alist
-            (not gnus-topic-inhibit-change-level))
-    ;; Remove the group from the topics.
-    (when (and (< oldlevel gnus-level-zombie)
-              (>= level gnus-level-zombie))
-      (let (alist)
-       (when (setq alist (assoc (gnus-group-topic group) gnus-topic-alist))
-         (setcdr alist (delete group (cdr alist))))))
-    ;; If the group is subscribed. then we enter it into the topics.
-    (when (and (< level gnus-level-zombie)
-              (>= oldlevel gnus-level-zombie))
-      (let ((entry (assoc (caar gnus-topic-topology) gnus-topic-alist)))
-       (setcdr entry (cons group (cdr entry)))))))
-
-(defun gnus-topic-goto-next-group (group props)
-  "Go to group or the next group after group."
-  (if (null group)
-      (gnus-topic-goto-topic (cadr (memq 'gnus-topic props)))
-    (if (gnus-group-goto-group group)
-       t
-      ;; The group is no longer visible.
-      (let* ((list (assoc (gnus-group-topic group) gnus-topic-alist))
-            (after (cdr (member group (cdr list)))))
-       ;; First try to put point on a group after the current one.
-       (while (and after
-                   (not (gnus-group-goto-group (car after))))
-         (setq after (cdr after)))
-       ;; Then try to put point on a group before point.
-       (unless after
-         (setq after (cdr (member group (reverse (cdr list)))))
-         (while (and after 
-                     (not (gnus-group-goto-group (car after))))
-           (setq after (cdr after))))
-       ;; Finally, just put point on the topic.
-       (unless after
-         (gnus-topic-goto-topic (car list))
-         (setq after nil))
-       t))))
-
 (defun gnus-topic-kill-group (&optional n discard)
   "Kill the next N groups."
   (interactive "P")
 (defun gnus-topic-kill-group (&optional n discard)
   "Kill the next N groups."
   (interactive "P")
@@ -783,26 +1003,30 @@ group."
   "Yank the last topic."
   (interactive "p")
   (if gnus-topic-killed-topics
   "Yank the last topic."
   (interactive "p")
   (if gnus-topic-killed-topics
-      (let ((previous (gnus-group-parent-topic))
-           (item (nth 1 (pop gnus-topic-killed-topics))))
+      (let ((previous 
+            (or (gnus-group-topic-name)
+                (gnus-topic-next-topic (gnus-current-topic))))
+           (item (cdr (pop gnus-topic-killed-topics))))
        (gnus-topic-create-topic
        (gnus-topic-create-topic
-        (car item) (gnus-topic-parent-topic previous) previous))
+        (caar item) (gnus-topic-parent-topic previous) previous
+        item)
+       (gnus-topic-goto-topic (caar item)))
     (let* ((prev (gnus-group-group-name))
           (gnus-topic-inhibit-change-level t)
           (gnus-group-indentation
            (make-string 
             (* gnus-topic-indent-level
                (or (save-excursion
     (let* ((prev (gnus-group-group-name))
           (gnus-topic-inhibit-change-level t)
           (gnus-group-indentation
            (make-string 
             (* gnus-topic-indent-level
                (or (save-excursion
-                     (gnus-topic-goto-topic (gnus-group-parent-topic))
+                     (gnus-topic-goto-topic (gnus-current-topic))
                      (gnus-group-topic-level)) 0)) ? ))
                      (gnus-group-topic-level)) 0)) ? ))
-          yanked group alist)
+          yanked alist)
       ;; We first yank the groups the normal way...
       (setq yanked (gnus-group-yank-group arg))
       ;; Then we enter the yanked groups into the topics they belong
       ;; to. 
       (setq alist (assoc (save-excursion
                           (forward-line -1)
       ;; We first yank the groups the normal way...
       (setq yanked (gnus-group-yank-group arg))
       ;; Then we enter the yanked groups into the topics they belong
       ;; to. 
       (setq alist (assoc (save-excursion
                           (forward-line -1)
-                          (gnus-group-parent-topic))
+                          (gnus-current-topic))
                         gnus-topic-alist))
       (when (stringp yanked)
        (setq yanked (list yanked)))
                         gnus-topic-alist))
       (when (stringp yanked)
        (setq yanked (list yanked)))
@@ -818,10 +1042,10 @@ group."
     (gnus-topic-update-topic)))
 
 (defun gnus-topic-hide-topic ()
     (gnus-topic-update-topic)))
 
 (defun gnus-topic-hide-topic ()
-  "Hide all subtopics under the current topic."
+  "Hide the current topic."
   (interactive)
   (interactive)
-  (when (gnus-group-parent-topic)
-    (gnus-topic-goto-topic (gnus-group-parent-topic))
+  (when (gnus-current-topic)
+    (gnus-topic-goto-topic (gnus-current-topic))
     (gnus-topic-remove-topic nil nil 'hidden)))
 
 (defun gnus-topic-show-topic ()
     (gnus-topic-remove-topic nil nil 'hidden)))
 
 (defun gnus-topic-show-topic ()
@@ -832,16 +1056,16 @@ group."
 
 (defun gnus-topic-mark-topic (topic &optional unmark)
   "Mark all groups in the topic with the process mark."
 
 (defun gnus-topic-mark-topic (topic &optional unmark)
   "Mark all groups in the topic with the process mark."
-  (interactive (list (gnus-group-parent-topic)))
+  (interactive (list (gnus-current-topic)))
   (save-excursion
   (save-excursion
-    (let ((groups (gnus-topic-find-groups topic)))
+    (let ((groups (gnus-topic-find-groups topic 9 t)))
       (while groups
        (funcall (if unmark 'gnus-group-remove-mark 'gnus-group-set-mark)
                 (gnus-info-group (nth 2 (pop groups))))))))
 
 (defun gnus-topic-unmark-topic (topic &optional unmark)
   "Remove the process mark from all groups in the topic."
       (while groups
        (funcall (if unmark 'gnus-group-remove-mark 'gnus-group-set-mark)
                 (gnus-info-group (nth 2 (pop groups))))))))
 
 (defun gnus-topic-unmark-topic (topic &optional unmark)
   "Remove the process mark from all groups in the topic."
-  (interactive (list (gnus-group-parent-topic)))
+  (interactive (list (gnus-current-topic)))
   (gnus-topic-mark-topic topic t))
 
 (defun gnus-topic-get-new-news-this-topic (&optional n)
   (gnus-topic-mark-topic topic t))
 
 (defun gnus-topic-get-new-news-this-topic (&optional n)
@@ -893,17 +1117,16 @@ group."
 (defun gnus-topic-rename (old-name new-name)
   "Rename a topic."
   (interactive
 (defun gnus-topic-rename (old-name new-name)
   "Rename a topic."
   (interactive
-   (let (topic)
-     (list
-      (setq topic (completing-read "Rename topic: " gnus-topic-alist nil t
-                                  (cons (gnus-group-parent-topic) 0)))
-      (read-string (format "Rename %s to: " topic)))))
+   (let ((topic (gnus-current-topic)))
+     (list topic
+          (read-string (format "Rename %s to: " topic)))))
   (let ((top (gnus-topic-find-topology old-name))
        (entry (assoc old-name gnus-topic-alist)))
     (when top
       (setcar (cadr top) new-name))
     (when entry 
       (setcar entry new-name))
   (let ((top (gnus-topic-find-topology old-name))
        (entry (assoc old-name gnus-topic-alist)))
     (when top
       (setcar (cadr top) new-name))
     (when entry 
       (setcar entry new-name))
+    (forward-line -1)
     (gnus-group-list-groups)))
 
 (defun gnus-topic-indent (&optional unindent)
     (gnus-group-list-groups)))
 
 (defun gnus-topic-indent (&optional unindent)
@@ -912,20 +1135,22 @@ If UNINDENT, remove an indentation."
   (interactive "P")
   (if unindent
       (gnus-topic-unindent)
   (interactive "P")
   (if unindent
       (gnus-topic-unindent)
-    (let* ((topic (gnus-group-parent-topic))
+    (let* ((topic (gnus-current-topic))
           (parent (gnus-topic-previous-topic topic)))
       (unless parent
        (error "Nothing to indent %s into" topic))
       (when topic
        (gnus-topic-goto-topic topic)
        (gnus-topic-kill-group)
           (parent (gnus-topic-previous-topic topic)))
       (unless parent
        (error "Nothing to indent %s into" topic))
       (when topic
        (gnus-topic-goto-topic topic)
        (gnus-topic-kill-group)
-       (gnus-topic-create-topic topic parent)
-       (gnus-topic-goto-topic topic)))))
+       (gnus-topic-create-topic
+        topic parent nil (cdr (pop gnus-topic-killed-topics)))
+       (or (gnus-topic-goto-topic topic)
+           (gnus-topic-goto-topic parent))))))
 
 (defun gnus-topic-unindent ()
   "Unindent a topic."
   (interactive)
 
 (defun gnus-topic-unindent ()
   "Unindent a topic."
   (interactive)
-  (let* ((topic (gnus-group-parent-topic))
+  (let* ((topic (gnus-current-topic))
         (parent (gnus-topic-parent-topic topic))
         (grandparent (gnus-topic-parent-topic parent)))
     (unless grandparent
         (parent (gnus-topic-parent-topic topic))
         (grandparent (gnus-topic-parent-topic parent)))
     (unless grandparent
@@ -933,19 +1158,38 @@ If UNINDENT, remove an indentation."
     (when topic
       (gnus-topic-goto-topic topic)
       (gnus-topic-kill-group)
     (when topic
       (gnus-topic-goto-topic topic)
       (gnus-topic-kill-group)
-      (gnus-topic-create-topic topic grandparent)
+      (gnus-topic-create-topic
+       topic grandparent (gnus-topic-next-topic parent)
+       (cdr (pop gnus-topic-killed-topics)))
       (gnus-topic-goto-topic topic))))
 
 (defun gnus-topic-list-active (&optional force)
   "List all groups that Gnus knows about in a topicsified fashion.
 If FORCE, always re-read the active file."
   (interactive "P")
       (gnus-topic-goto-topic topic))))
 
 (defun gnus-topic-list-active (&optional force)
   "List all groups that Gnus knows about in a topicsified fashion.
 If FORCE, always re-read the active file."
   (interactive "P")
-  (gnus-topic-grok-active force force)
+  (when force
+    (gnus-get-killed-groups))
+  (gnus-topic-grok-active force)
   (let ((gnus-topic-topology gnus-topic-active-topology)
        (gnus-topic-alist gnus-topic-active-alist)
        gnus-killed-list gnus-zombie-list)
     (gnus-group-list-groups 9 nil 1)))
 
   (let ((gnus-topic-topology gnus-topic-active-topology)
        (gnus-topic-alist gnus-topic-active-alist)
        gnus-killed-list gnus-zombie-list)
     (gnus-group-list-groups 9 nil 1)))
 
+(defun gnus-topic-edit-parameters (group)
+  "Edit the group parameters of GROUP.
+If performed on a topic, edit the topic parameters instead."
+  (interactive (list (gnus-group-group-name)))
+  (if group
+      (gnus-group-edit-group-parameters group)
+    (if (not (gnus-group-topic-p))
+       (error "Nothing to edit on the current line.")
+      (let ((topic (gnus-group-topic-name)))
+       (gnus-edit-form
+        (gnus-topic-parameters topic)
+        "Editing the topic parameters."
+        `(lambda (form)
+           (gnus-topic-set-parameters ,topic form)))))))
+
 (provide 'gnus-topic)
 
 ;;; gnus-topic.el ends here
 (provide 'gnus-topic)
 
 ;;; gnus-topic.el ends here