patch for "/ *"

patch for "/ *"

[Eric Marsden]

[-- Attachment #1: Type: text/plain, Size: 252 bytes --]

Hi,

Most of the limiting commands work by filtering out articles from the
summary buffer. The "/ *" (limit-to-cache) command is different, since
it is aliased to "Y c" (insert-cached-articles). The attached patch
makes "/ *" as I would expect it to.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: make \"/ *\" operate like other limiting commands --]
[-- Type: text/x-patch, Size: 706 bytes --]

--- gnus-cache.el.orig	Thu Oct 11 11:29:32 2001
+++ gnus-cache.el	Thu Oct 11 11:31:45 2001
@@ -370,8 +370,16 @@
     (while cached
       (gnus-summary-goto-subject (pop cached) t))))
 
-(defalias 'gnus-summary-limit-include-cached
-  'gnus-summary-insert-cached-articles)
+(defun gnus-summary-limit-include-cached ()
+  "Limit the summary buffer to articles that are cached."
+  (interactive)
+  (let ((cached (sort (copy-sequence gnus-newsgroup-cached) '>))
+	(gnus-verbose (max 6 gnus-verbose)))
+    (if cached
+        (progn
+          (gnus-summary-limit cached)
+          (gnus-summary-position-point))
+        (gnus-message 3 "No cached articles for this group"))))
 
 ;;; Internal functions.
 

[-- Attachment #3: Type: text/plain, Size: 79 bytes --]


-- 
Eric Marsden                          <URL:http://www.laas.fr/~emarsden/>

Re: patch for "/ *"

[Simon Josefsson]

Eric Marsden <emarsden@laas.fr> writes:

> Hi,
>
> Most of the limiting commands work by filtering out articles from the
> summary buffer. The "/ *" (limit-to-cache) command is different, since
> it is aliased to "Y c" (insert-cached-articles). The attached patch
> makes "/ *" as I would expect it to.

Committed, thanks.  Have you signed FSF papers?  The patch was longer
than 5 lines, so I guess that would be necessary.

Re: patch for "/ *"

[Eric Marsden]

>>>>> "sj" == Simon Josefsson <jas@extundo.com> writes:

  sj> Committed, thanks. Have you signed FSF papers? The patch was
  sj> longer than 5 lines, so I guess that would be necessary.

I have signed papers for Emacs, yes -- are there separate ones for
Gnus?  
  
-- 
Eric Marsden                          <URL:http://www.laas.fr/~emarsden/>

Re: patch for "/ *"

[Simon Josefsson]

Eric Marsden <emarsden@laas.fr> writes:

>>>>>> "sj" == Simon Josefsson <jas@extundo.com> writes:
>
>   sj> Committed, thanks. Have you signed FSF papers? The patch was
>   sj> longer than 5 lines, so I guess that would be necessary.
>
> I have signed papers for Emacs, yes -- are there separate ones for
> Gnus?  

I don't think so, but I don't grok legalese.  Anyone?

Re: patch for "/ *"

[ShengHuo ZHU]

Simon Josefsson <jas@extundo.com> writes:

> Eric Marsden <emarsden@laas.fr> writes:
>
>>>>>>> "sj" == Simon Josefsson <jas@extundo.com> writes:
>>
>>   sj> Committed, thanks. Have you signed FSF papers? The patch was
>>   sj> longer than 5 lines, so I guess that would be necessary.
>>
>> I have signed papers for Emacs, yes -- are there separate ones for
>> Gnus?  
>
> I don't think so, but I don't grok legalese.  Anyone?

No separate papers are required.

ShengHuo