Gnus development mailing list
 help / color / mirror / Atom feed
From: Kai Grossjohann <kai@emptydomain.de>
Subject: Re: Exact article count for nnml groups
Date: Sat, 13 Mar 2004 13:15:13 +0100	[thread overview]
Message-ID: <87ekrxueni.fsf@emptyhost.emptydomain.de> (raw)
In-Reply-To: <uish9sxu9.fsf@isisSEND.vanderbiltME.eduSPAM>

It's good stuff.  In addition to the other comments, I have one other
issue:

If I understand correctly, then your code never tries to invoke the
new backend function unless the gnus-count-articles-in-groups variable
allows it.  If the variable allows it, then you look whether the new
function is available.  If so, you call it.  Otherwise, you fall back
to the old backend function.

I'd suggest to do away with the gnus-count-articles-in-groups variable
-- just always try to invoke the new function if available.

If you really want to allow people to turn it off, you could do like
nnml-nov-is-evil and similar variables, which also allow to turn off
backend features.  (Following Kevin's unhappiness with matching on
group names.)

Kai




      parent reply	other threads:[~2004-03-13 12:15 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-03-12 18:51 David Hanak
2004-03-12 22:23 ` Jesper Harder
2004-03-13 16:07   ` Michael Schierl
2004-03-15 17:03   ` David Hanak
2004-03-17 16:04     ` Kai Grossjohann
2004-03-12 23:56 ` Kevin Greiner
2004-05-16 14:38   ` Lars Magne Ingebrigtsen
2004-03-13 12:15 ` Kai Grossjohann [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87ekrxueni.fsf@emptyhost.emptydomain.de \
    --to=kai@emptydomain.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).