From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#75305: 31.0.50; gnus-refer-thread-use-search isn't exact enough about how the current group is searched Date: Sun, 05 Jan 2025 08:29:15 +0200 Message-ID: <86wmf9ai4k.fsf@gnu.org> References: <8634i0ia6l.fsf@gnu.org> <86o70md819.fsf@gnu.org> <87bjwmw9f1.fsf@> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="19202"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 75305@debbugs.gnu.org To: =?UTF-8?Q?Bj=C3=B6rn?= Bidar , Andrew G Cohen , Eric Abrahamsen Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Sun Jan 05 07:30:28 2025 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1tUK9b-0004qP-K4 for geb-bug-gnu-emacs@m.gmane-mx.org; Sun, 05 Jan 2025 07:30:27 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tUK9K-0002Da-4a; Sun, 05 Jan 2025 01:30:10 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tUK9G-0002D3-0z for bug-gnu-emacs@gnu.org; Sun, 05 Jan 2025 01:30:06 -0500 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1tUK9C-0002yh-Mi for bug-gnu-emacs@gnu.org; Sun, 05 Jan 2025 01:30:03 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=debbugs.gnu.org; s=debbugs-gnu-org; h=MIME-version:References:In-Reply-To:From:Date:To:Subject; bh=yO5plfZhrn64vvuoOIUxFwVAOkw1mBwyE8qcFOz/IvQ=; b=KvCY4GKSBb3aIhzs9zoSycc6/v6L5z0UPyktINyj3/jalvZPOk+ifH8Y5akY93IjcLzBtV95hDy9B5QpYrO2uxEekrFTfdCqt8Le8P7rt1dRnp3zW95leKuUzkwUitJnHLblwbFYo1L8vrU5b5xy2B8cfisgU+0kuiapE6KKltnL+v+kW+zanIBGw9ey4prLS1DMl8Pbhz3peAzIbtj2kiD3jjxr/RFof7fUomIYfirBGJ5iSETbEPLQTmjengJy5drS4QgXpAkMqgK5MDrswghRlrfOJZ4X7UFFYj4YhkvIvPizqM+WJHrcWJzu+QwcvQuz7+oeTm6ahW7qFZZP9Q==; Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1tUK9C-00066J-IO for bug-gnu-emacs@gnu.org; Sun, 05 Jan 2025 01:30:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 05 Jan 2025 06:30:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 75305 X-GNU-PR-Package: emacs Original-Received: via spool by 75305-submit@debbugs.gnu.org id=B75305.173605856723360 (code B ref 75305); Sun, 05 Jan 2025 06:30:02 +0000 Original-Received: (at 75305) by debbugs.gnu.org; 5 Jan 2025 06:29:27 +0000 Original-Received: from localhost ([127.0.0.1]:59257 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1tUK8c-00064i-UC for submit@debbugs.gnu.org; Sun, 05 Jan 2025 01:29:27 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:47310) by debbugs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1tUK8b-00064R-3z for 75305@debbugs.gnu.org; Sun, 05 Jan 2025 01:29:25 -0500 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tUK8U-0002un-Fm; Sun, 05 Jan 2025 01:29:18 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=MIME-version:References:Subject:In-Reply-To:To:From: Date; bh=yO5plfZhrn64vvuoOIUxFwVAOkw1mBwyE8qcFOz/IvQ=; b=mpCJk3iohBTcN44O0e8F kkQU41lXMYdc1Be97pGCjGaBWtKhVAZorRoqRZ5HiE6AeczH5PCGB8o1GuPs77wPxHVWTcsmJLp49 6sw2EaDdevY7Oakn9qKD0bzb8gLbo2Wk06Rx428O+u1P6CyAtE35LW0GoWuu9GLRfW0+jjaeTiDbG hkRybBcjvTvfos/S0cpvL1JB4l+5DU/o5wn/Za0q3IPcAhGrlCOBqizHZzU8jh1zt9H+D6BeCWrud 4GQca9h5T3TdCHJcPwLTr740oNiI4Dqjfl0nVrfu3QLf9yEkXBO8eLhykpXbncgf14x8+TugFew6L tG5j1Xh655Kx+A==; In-Reply-To: <87bjwmw9f1.fsf@> (message from =?UTF-8?Q?Bj=C3=B6rn?= Bidar on Sat, 04 Jan 2025 23:34:10 +0200) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:298496 Archived-At: > From: Björn Bidar > Cc: 75305@debbugs.gnu.org > Date: Sat, 04 Jan 2025 23:34:10 +0200 > > Eli Zaretskii writes: > > > Thanks, but the patch seems to include unrelated changes in the code? > > I thought we were talking only about fixing the manual? Or what did I > > miss? > > > > I look at the situation further and noticed that adding the current > group does not work without the user writing a custom hook to add it. > I already wrote a hook to change gnus-refer-thread-use-search for > another reason, to add the current gnus-message-archive-group to > gnus-refer-thread-use-search. > To just add the current group writing a hook would be to clumsy. > Using the current symbol similar as in gnus-refer-article-method is much > simpler, makes the setting easier to understand and customize for new > users. > > The customization changes are there to use better custom types which > makes it easier to customize and harder to put false values into the > option when using custom. Then we'll need to wait for our Gnus expert (CC'ed) to review your code changes. I have some comments to the documentation part below: > * lisp/gnus/gnus-sum.el (gnus-refer-thread-use-search): > Make it easier to customize the variable by adding predefine choices > as well as descriptions to each choice. Add option to add the value > `current' to the list of servers and groups to add the current group. > E.g. to add another group and the current group. > * lisp/gnus/gnus-search.el (gnus-refer-thread-maybe-add-current-group, > gnus-search-thread): Add helper function to determine if the option > contains the value current. Use helper function. > * lisp/gnus/nnselect.el (nnselect-request-thread): Use helper function > to check if the value current is in gnus-refer-thread-use-search. > * doc/misc/gnus.texi (gnus-refer-thread-use-search): > Include the current value in the section. Format the section a little > better to have a paragraph for each possible value that the option > may contain. Explain the option to have a list of servers more detailed > , i.e. similarly to the custom description. Highlight that the current > group is not searched unless specified. Explain that unless the articles > of that thread are contained in the searched groups not adding the > current group might not be the desired effect. Some of the lines in the log message are too long. Please be sure to wrap lines at column 64 if possible, and in any case before column 70. > +If @code{gnus-refer-thread-use-search} is @code{current} or @code{nil} > +(the default) then thread-referral only looks for articles in the current group. ^ Comma missing there (yes, it was also missing in the original text). > +If this variable is @code{t} the server to which the current group ^ Same here. > +The variable can also be a list of server each server > +in the list is searched. The servers have to specified in a list ^^^^^^^^^^^^^^^^^ "have to be specified", right? > +where the first element of this has to be the server and any remaining elements > +can be the groups belonging to that server. This is better rephrased in our usual style: The value can also be a list of the form @code{(@var{server} @var{group}@dots{})}, where @var{server} is the server and @var{group}s are the groups belonging to the server. > +To search in the current group in addition to other groups the list of servers ^ Again missing comma. > +If @code{current} is not in the list the current group is not searched, ^ Same here. > +note this may be not the desired effect if any of articles referred in > +the thread are not contained in any of the searched groups. I would divide this sentence into two: If @code{current} is not in the list, ... is not searched. This may not be the desired effect if ... > +(defun gnus-refer-thread-maybe-add-current-group (group) > + "Find if GROUP should be added to `gnus-refer-thread-use-search'." "Find" is to abstract and withholds information. Suggest to rephrase: Return t if GROUP should be added to `gnus-refer-thread-use-search'. Return nil otherwise. > +The list of of server and groups may contain > +the symbol `current' to refer to the current group. > +For example to search in the current group in addition to other groups. ^ Comma missing there. Thanks.