From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Arun V Newsgroups: gmane.emacs.bugs Subject: bug#56535: 28.1; `no' and `no-ding' values for isearch-wrap-pause option (since Emacs 28.1) not working as documented Date: Wed, 20 Jul 2022 21:46:31 +0530 Message-ID: References: <83a69dw72m.fsf@gnu.org> <83wnchuexs.fsf@gnu.org> <865yk0g5cf.fsf@mail.linkov.net> <8635ez5yf6.fsf@mail.linkov.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="17719"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 56535@debbugs.gnu.org, Eli Zaretskii , visuweshm@gmail.com To: Juri Linkov Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Wed Jul 20 18:17:34 2022 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 1oECOE-0004Ne-Jo for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 20 Jul 2022 18:17:34 +0200 Original-Received: from localhost ([::1]:38434 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oECOD-0007S5-Fk for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 20 Jul 2022 12:17:33 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:49006) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oECNj-0007Rt-3F for bug-gnu-emacs@gnu.org; Wed, 20 Jul 2022 12:17:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:60956) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1oECNi-0005k0-Qh for bug-gnu-emacs@gnu.org; Wed, 20 Jul 2022 12:17:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1oECNi-0008RE-GS for bug-gnu-emacs@gnu.org; Wed, 20 Jul 2022 12:17:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Arun V Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 20 Jul 2022 16:17:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 56535 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: moreinfo Original-Received: via spool by 56535-submit@debbugs.gnu.org id=B56535.165833381132414 (code B ref 56535); Wed, 20 Jul 2022 16:17:02 +0000 Original-Received: (at 56535) by debbugs.gnu.org; 20 Jul 2022 16:16:51 +0000 Original-Received: from localhost ([127.0.0.1]:58715 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oECNX-0008Qk-9c for submit@debbugs.gnu.org; Wed, 20 Jul 2022 12:16:51 -0400 Original-Received: from mail-ej1-f48.google.com ([209.85.218.48]:46785) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oECNU-0008QT-P3 for 56535@debbugs.gnu.org; Wed, 20 Jul 2022 12:16:50 -0400 Original-Received: by mail-ej1-f48.google.com with SMTP id ez10so33903742ejc.13 for <56535@debbugs.gnu.org>; Wed, 20 Jul 2022 09:16:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Qd9xz5IgZSI36eRvnXTZ3jU8D/dOZ50GOjHd+H5vSjw=; b=agchZ2yuzWJp37HaPrmYlKiplB3xIpCKnlTNTfdAMfyKB1GY/1inXu09Co7mvc6f/Q FhpU/492b9E8Ot+mqBnKE7IA8r92j8K8sdeEAmGq94Js58Ad0lvKIBM2FjaOyuU2hc7+ n3xJt+dTWqxAB57IDcWNRmySLJ3qPiRADzOI2rdOzp554DwNuOzWZkzwR9iyqLB1k92S zJE+IOuLdW7vhdODesy3ftVKRQVCBiIuOrpTrVadHKmDN4f7ufM7apYe+FPlHfGo+3b4 JKKEowZ1wM9fYNDvW9lOz3Grxac3ziTpqnfRtkjSV69p8VLajaLrAng8pVxIryFFr2zq InKg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Qd9xz5IgZSI36eRvnXTZ3jU8D/dOZ50GOjHd+H5vSjw=; b=Xbikhl/NemimVx5oJ+KkvCZfQ6cKi4sg3N1BR90zf2ehkciVMHx9go9tQw+kW4Fk84 MqgJBLpK2veOqrEL+aSd0e+e5fEikngV6tVWK8XAc0+EWRKrR47TSEMSKj+Py61juAW9 ylUE9gf5McveMDx0rAZs7/y/y5+ucU5HCvGla00QSiVIf/wqPVygfv97bYbU6EXxFSkO Yj3ZGSg9yM+XB1Utk6k7Y/j2R9JlCyF8xSr8R8MoDxG0bIAjWbXxJI3YbIIQisktx8Zr uPrsdXKo14xODlNgyZE3LpfLOUvGsCjRGxfHIACr7PtxEnh49avfa2E//2rhDhW5swSf k18g== X-Gm-Message-State: AJIora/odTGO0YYAo7OC+1YXNpT9fDbDVe54+QngxMwz+YQT8afanaCF 3g4qeBG5lqJPPvH06KQx0plGK8v8DaOY0JPrM60= X-Google-Smtp-Source: AGRyM1vq7MauOkVhxOElF97vl/D2vMj3ppJlyfF9nIGOhqv7OB/kcKWUJFQpNpAzoEAfcGnWqB7jUWYNt7obPlrnsrY= X-Received: by 2002:a17:907:7fa5:b0:72b:755a:b77e with SMTP id qk37-20020a1709077fa500b0072b755ab77emr36658807ejc.474.1658333802696; Wed, 20 Jul 2022 09:16:42 -0700 (PDT) In-Reply-To: <8635ez5yf6.fsf@mail.linkov.net> 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" Xref: news.gmane.io gmane.emacs.bugs:237514 Archived-At: Definitely ding. Lot of people do prefer visual feedback. How about this? :type '(choice (const :tag "Pause before wrapping" t) (const :tag "No pause before wrapping for non-incremental search" nis-no) (const :tag "No pause and no flashing for non-incremental search" nis-no-ding) (const :tag "Absolutely no pause before wrapping" no) ;;This will cover non-incremental searches as well (const :tag "Absolutely no pause and no flashing" no-ding) ;;This will cover non-incremental searches as well (const :tag "Disable wrapping" nil)) On Mon, 18 Jul 2022 at 01:05, Juri Linkov wrote: > > >>> Please try the following patch after setting isearch-wrap-pause to the > >>> new value 'auto-repeat': > >>> > >> Regret the delayed response. > >> > >> Yes, I do believe this would be a good idea. > >> > >> Let me try this patch atop emacs-28.1 and get back to you. > > > > Thanks Juri. Your patch works. > > No problem with the delay. Thanks for confirming that it works. > > Now we have to find a better name to describe what it really does. > Probably it should have the prefix "no-" since it makes no pause > like the existing two values 'no' and 'no-ding': > > :type '(choice (const :tag "Pause before wrapping" t) > (const :tag "No pause before wrapping" no) > (const :tag "No pause and no flashing" no-ding) > (const :tag "Disable wrapping" nil)) > > This raises the question: should it ding and flash on wrapping? > Or two new separate values are needed: one to ding when wrapping > on typing a character, and another not to ding? To ding, or not to ding? -- -- Thanks and regards, Arunlal Venugopal