On Thu, Oct 27 2005, Jouni K Seppanen wrote:

> Simon Josefsson <jas@extundo.com> writes:
>
>> Would you like to submit a patch for the manual as well?
>
> Here's a patch. I also documented nnimap-nov-is-evil, or rather my
> understanding of what it seems to do. 

Can someone comment on this?  Is Jouni's documentation correct? [1]

> It occurred to me that people to whom slow expiry is a problem might
> go to the index of the manual and look up "expiring mail". There was
> already one index entry for "expiring IMAP mail", so I added this
> one, and in the process I added entries on some other pages that
> might be relevant to people looking up "expiring mail".

Thanks for your patch.  I'm sorry that we forgot to apply your patch
upto now.  I'd like to install it now in the stable branch.

I'm not sure about changing @cindex to @vindex, though:

,----
| @@ -2842,7 +2844,7 @@
|  See also @code{gnus-total-expirable-newsgroups}.
|  
|  @item expiry-wait
| -@cindex expiry-wait
| +@vindex expiry-wait
|  @vindex nnmail-expiry-wait-function
|  If the group parameter has an element that looks like
|  @code{(expiry-wait . 10)}, this value will override any
| @@ -2852,7 +2854,7 @@
|  symbols @code{never} or @code{immediate}.
|  
|  @item expiry-target
| -@cindex expiry-target
| +@vindex expiry-target
|  Where expired messages end up.  This parameter overrides
|  @code{nnmail-expiry-target}.
`----

I think we use @cindex for such symbols (as these are not variables).
Cf. M-x @cindex RET [^gmn][a-z]+-[a-z]+ RET (*) and M-x occur RET
@vindex [^gmn] RET:

,----
| 23 matches for "@cindex [^gmn][a-z]+-[a-z]+" in buffer: gnus.texi
|    1487:@cindex auto-save
|    2700:@cindex to-address		(*)
|    2723:@cindex to-list		(*)
|    2777:@cindex broken-reply-to	(*)
|    2786:@cindex to-group		(*)
|    2812:@cindex auto-expire		(*)
|    2821:@cindex total-expire		(*)
|    2859:@cindex admin-address		(*)
|    2926:@cindex ignored-charset	(*)
|    2934:@cindex posting-style
|    2953:@cindex post-method		(*)
|[...]
`----

I will install your patch without these @cindex -> @vindex changes.
Thanks for your contribution.

Bye, Reiner.

[1] For convenience, I include the patch once more: