* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices @ 2023-09-18 12:17 Mauro Aranda 2023-09-18 12:20 ` Mauro Aranda 0 siblings, 1 reply; 8+ messages in thread From: Mauro Aranda @ 2023-09-18 12:17 UTC (permalink / raw) To: 66071 The docstring says term-scroll-to-bottom-on-output may be set to the symbols this and others, but its :type is just a boolean. ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices 2023-09-18 12:17 bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices Mauro Aranda @ 2023-09-18 12:20 ` Mauro Aranda 2023-09-18 14:20 ` Robert Pluim 0 siblings, 1 reply; 8+ messages in thread From: Mauro Aranda @ 2023-09-18 12:20 UTC (permalink / raw) To: 66071 [-- Attachment #1: Type: text/plain, Size: 40 bytes --] tags 66071 patch quit Patch attached. [-- Attachment #2: 0001-Fix-term-scroll-to-bottom-on-output-type-Bug-66071.patch --] [-- Type: text/x-patch, Size: 954 bytes --] From d891a9f705b3b4af2aedeed645f4636535ea6c2e Mon Sep 17 00:00:00 2001 From: Mauro Aranda <maurooaranda@gmail.com> Date: Mon, 18 Sep 2023 09:19:04 -0300 Subject: [PATCH] Fix term-scroll-to-bottom-on-output :type (Bug#66071) * lisp/term.el (term-scroll-to-bottom-on-output): Add missing choices. --- lisp/term.el | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lisp/term.el b/lisp/term.el index b8466b21332..5dd73447cfb 100644 --- a/lisp/term.el +++ b/lisp/term.el @@ -494,7 +494,10 @@ term-scroll-to-bottom-on-output See variable `term-scroll-show-maximum-output'. This variable is buffer-local." - :type 'boolean + :type '(choice (const :tag "Don't scroll" nil) + (const :tag "Scroll selected window only" this) + (const :tag "Scroll unselected windows" others) + (other :tag "Scroll all windows" t)) :group 'term) (defcustom term-scroll-snap-to-bottom t -- 2.34.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices 2023-09-18 12:20 ` Mauro Aranda @ 2023-09-18 14:20 ` Robert Pluim 2023-09-18 15:12 ` Mauro Aranda 0 siblings, 1 reply; 8+ messages in thread From: Robert Pluim @ 2023-09-18 14:20 UTC (permalink / raw) To: Mauro Aranda; +Cc: 66071 >>>>> On Mon, 18 Sep 2023 09:20:07 -0300, Mauro Aranda <maurooaranda@gmail.com> said: Mauro> tags 66071 patch Mauro> quit Mauro> Patch attached. Mauro> From d891a9f705b3b4af2aedeed645f4636535ea6c2e Mon Sep 17 00:00:00 2001 Mauro> From: Mauro Aranda <maurooaranda@gmail.com> Mauro> Date: Mon, 18 Sep 2023 09:19:04 -0300 Mauro> Subject: [PATCH] Fix term-scroll-to-bottom-on-output :type (Bug#66071) Mauro> * lisp/term.el (term-scroll-to-bottom-on-output): Add missing choices. Mauro> --- Mauro> lisp/term.el | 5 ++++- Mauro> 1 file changed, 4 insertions(+), 1 deletion(-) Mauro> diff --git a/lisp/term.el b/lisp/term.el Mauro> index b8466b21332..5dd73447cfb 100644 Mauro> --- a/lisp/term.el Mauro> +++ b/lisp/term.el Mauro> @@ -494,7 +494,10 @@ term-scroll-to-bottom-on-output Mauro> See variable `term-scroll-show-maximum-output'. Mauro> This variable is buffer-local." Mauro> - :type 'boolean Mauro> + :type '(choice (const :tag "Don't scroll" nil) Mauro> + (const :tag "Scroll selected window only" this) Mauro> + (const :tag "Scroll unselected windows" others) Mauro> + (other :tag "Scroll all windows" t)) Mauro> :group 'term) The code (and the docstring) also mention 'all. Robert -- ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices 2023-09-18 14:20 ` Robert Pluim @ 2023-09-18 15:12 ` Mauro Aranda 2023-09-18 15:22 ` Robert Pluim 0 siblings, 1 reply; 8+ messages in thread From: Mauro Aranda @ 2023-09-18 15:12 UTC (permalink / raw) To: Robert Pluim; +Cc: 66071 On 18/9/23 11:20, Robert Pluim wrote: > On Mon, 18 Sep 2023 09:20:07 -0300, Mauro Aranda <maurooaranda@gmail.com> said: > Mauro> Patch attached. > > Mauro> From d891a9f705b3b4af2aedeed645f4636535ea6c2e Mon Sep 17 00:00:00 2001 > Mauro> From: Mauro Aranda <maurooaranda@gmail.com> > Mauro> Date: Mon, 18 Sep 2023 09:19:04 -0300 > Mauro> Subject: [PATCH] Fix term-scroll-to-bottom-on-output :type (Bug#66071) > > Mauro> * lisp/term.el (term-scroll-to-bottom-on-output): Add missing choices. > Mauro> --- > Mauro> lisp/term.el | 5 ++++- > Mauro> 1 file changed, 4 insertions(+), 1 deletion(-) > > Mauro> diff --git a/lisp/term.el b/lisp/term.el > Mauro> index b8466b21332..5dd73447cfb 100644 > Mauro> --- a/lisp/term.el > Mauro> +++ b/lisp/term.el > Mauro> @@ -494,7 +494,10 @@ term-scroll-to-bottom-on-output > > Mauro> See variable `term-scroll-show-maximum-output'. > Mauro> This variable is buffer-local." > Mauro> - :type 'boolean > Mauro> + :type '(choice (const :tag "Don't scroll" nil) > Mauro> + (const :tag "Scroll selected window only" this) > Mauro> + (const :tag "Scroll unselected windows" others) > Mauro> + (other :tag "Scroll all windows" t)) > Mauro> :group 'term) > > The code (and the docstring) also mention 'all. I know, that's why I put other, to catch either t or 'all, since it doesn't make sense to have 2 different items for the same option. And adding something to match only t or 'all didn't seem worth the trouble. Am I missing something? ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices 2023-09-18 15:12 ` Mauro Aranda @ 2023-09-18 15:22 ` Robert Pluim 2023-09-18 15:29 ` Mauro Aranda 2023-09-24 11:38 ` Mauro Aranda 0 siblings, 2 replies; 8+ messages in thread From: Robert Pluim @ 2023-09-18 15:22 UTC (permalink / raw) To: Mauro Aranda; +Cc: 66071 >>>>> On Mon, 18 Sep 2023 12:12:41 -0300, Mauro Aranda <maurooaranda@gmail.com> said: Mauro> On 18/9/23 11:20, Robert Pluim wrote: >> On Mon, 18 Sep 2023 09:20:07 -0300, Mauro Aranda Mauro> <maurooaranda@gmail.com> said: >> Mauro> Patch attached. >> >> Mauro> From d891a9f705b3b4af2aedeed645f4636535ea6c2e Mon Sep 17 Mauro> 00:00:00 2001 >> Mauro> From: Mauro Aranda <maurooaranda@gmail.com> >> Mauro> Date: Mon, 18 Sep 2023 09:19:04 -0300 >> Mauro> Subject: [PATCH] Fix term-scroll-to-bottom-on-output Mauro> :type (Bug#66071) >> >> Mauro> * lisp/term.el (term-scroll-to-bottom-on-output): Add Mauro> missing choices. >> Mauro> --- >> Mauro> lisp/term.el | 5 ++++- >> Mauro> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> Mauro> diff --git a/lisp/term.el b/lisp/term.el >> Mauro> index b8466b21332..5dd73447cfb 100644 >> Mauro> --- a/lisp/term.el >> Mauro> +++ b/lisp/term.el >> Mauro> @@ -494,7 +494,10 @@ term-scroll-to-bottom-on-output >> >> Mauro> See variable `term-scroll-show-maximum-output'. >> Mauro> This variable is buffer-local." >> Mauro> - :type 'boolean >> Mauro> + :type '(choice (const :tag "Don't scroll" nil) >> Mauro> + (const :tag "Scroll selected window Mauro> only" this) >> Mauro> + (const :tag "Scroll unselected windows" Mauro> others) >> Mauro> + (other :tag "Scroll all windows" t)) >> Mauro> :group 'term) >> >> The code (and the docstring) also mention 'all. Mauro> I know, that's why I put other, to catch either t or 'all, since it Mauro> doesn't make sense to have 2 different items for the same option. And Mauro> adding something to match only t or 'all didn't seem worth the trouble. Mauro> Am I missing something? Both before and after your patch, the defcustom and the docstring disagree. The docstring says 'all', but thatʼs not a value you can get via customize. Personally Iʼd be inclined to delete 'all' from the docstring. People already using 'all' wonʼt notice, since the code will still recognize it. Robert -- ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices 2023-09-18 15:22 ` Robert Pluim @ 2023-09-18 15:29 ` Mauro Aranda 2023-09-24 11:38 ` Mauro Aranda 1 sibling, 0 replies; 8+ messages in thread From: Mauro Aranda @ 2023-09-18 15:29 UTC (permalink / raw) To: Robert Pluim; +Cc: 66071 On 18/9/23 12:22, Robert Pluim wrote: >>>>>> On Mon, 18 Sep 2023 12:12:41 -0300, Mauro Aranda <maurooaranda@gmail.com> said: > > Mauro> On 18/9/23 11:20, Robert Pluim wrote: > >> On Mon, 18 Sep 2023 09:20:07 -0300, Mauro Aranda > Mauro> <maurooaranda@gmail.com> said: > >> Mauro> Patch attached. > >> > >> Mauro> From d891a9f705b3b4af2aedeed645f4636535ea6c2e Mon Sep 17 > Mauro> 00:00:00 2001 > >> Mauro> From: Mauro Aranda <maurooaranda@gmail.com> > >> Mauro> Date: Mon, 18 Sep 2023 09:19:04 -0300 > >> Mauro> Subject: [PATCH] Fix term-scroll-to-bottom-on-output > Mauro> :type (Bug#66071) > >> > >> Mauro> * lisp/term.el (term-scroll-to-bottom-on-output): Add > Mauro> missing choices. > >> Mauro> --- > >> Mauro> lisp/term.el | 5 ++++- > >> Mauro> 1 file changed, 4 insertions(+), 1 deletion(-) > >> > >> Mauro> diff --git a/lisp/term.el b/lisp/term.el > >> Mauro> index b8466b21332..5dd73447cfb 100644 > >> Mauro> --- a/lisp/term.el > >> Mauro> +++ b/lisp/term.el > >> Mauro> @@ -494,7 +494,10 @@ term-scroll-to-bottom-on-output > >> > >> Mauro> See variable `term-scroll-show-maximum-output'. > >> Mauro> This variable is buffer-local." > >> Mauro> - :type 'boolean > >> Mauro> + :type '(choice (const :tag "Don't scroll" nil) > >> Mauro> + (const :tag "Scroll selected window > Mauro> only" this) > >> Mauro> + (const :tag "Scroll unselected windows" > Mauro> others) > >> Mauro> + (other :tag "Scroll all windows" t)) > >> Mauro> :group 'term) > >> > >> The code (and the docstring) also mention 'all. > > Mauro> I know, that's why I put other, to catch either t or 'all, since it > Mauro> doesn't make sense to have 2 different items for the same option. And > Mauro> adding something to match only t or 'all didn't seem worth the trouble. > > Mauro> Am I missing something? > > Both before and after your patch, the defcustom and the docstring > disagree. The docstring says 'all', but thatʼs not a value you can get > via customize. > > Personally Iʼd be inclined to delete 'all' from the docstring. People > already using 'all' wonʼt notice, since the code will still recognize > it. That's fine by me, thanks. ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices 2023-09-18 15:22 ` Robert Pluim 2023-09-18 15:29 ` Mauro Aranda @ 2023-09-24 11:38 ` Mauro Aranda 2023-09-30 23:36 ` Stefan Kangas 1 sibling, 1 reply; 8+ messages in thread From: Mauro Aranda @ 2023-09-24 11:38 UTC (permalink / raw) To: 66071 [-- Attachment #1: Type: text/plain, Size: 2601 bytes --] On 18/9/23 12:22, Robert Pluim wrote: >>>>>> On Mon, 18 Sep 2023 12:12:41 -0300, Mauro Aranda <maurooaranda@gmail.com> said: > > Mauro> On 18/9/23 11:20, Robert Pluim wrote: > >> On Mon, 18 Sep 2023 09:20:07 -0300, Mauro Aranda > Mauro> <maurooaranda@gmail.com> said: > >> Mauro> Patch attached. > >> > >> Mauro> From d891a9f705b3b4af2aedeed645f4636535ea6c2e Mon Sep 17 > Mauro> 00:00:00 2001 > >> Mauro> From: Mauro Aranda <maurooaranda@gmail.com> > >> Mauro> Date: Mon, 18 Sep 2023 09:19:04 -0300 > >> Mauro> Subject: [PATCH] Fix term-scroll-to-bottom-on-output > Mauro> :type (Bug#66071) > >> > >> Mauro> * lisp/term.el (term-scroll-to-bottom-on-output): Add > Mauro> missing choices. > >> Mauro> --- > >> Mauro> lisp/term.el | 5 ++++- > >> Mauro> 1 file changed, 4 insertions(+), 1 deletion(-) > >> > >> Mauro> diff --git a/lisp/term.el b/lisp/term.el > >> Mauro> index b8466b21332..5dd73447cfb 100644 > >> Mauro> --- a/lisp/term.el > >> Mauro> +++ b/lisp/term.el > >> Mauro> @@ -494,7 +494,10 @@ term-scroll-to-bottom-on-output > >> > >> Mauro> See variable `term-scroll-show-maximum-output'. > >> Mauro> This variable is buffer-local." > >> Mauro> - :type 'boolean > >> Mauro> + :type '(choice (const :tag "Don't scroll" nil) > >> Mauro> + (const :tag "Scroll selected window > Mauro> only" this) > >> Mauro> + (const :tag "Scroll unselected windows" > Mauro> others) > >> Mauro> + (other :tag "Scroll all windows" t)) > >> Mauro> :group 'term) > >> > >> The code (and the docstring) also mention 'all. > > Mauro> I know, that's why I put other, to catch either t or 'all, since it > Mauro> doesn't make sense to have 2 different items for the same option. And > Mauro> adding something to match only t or 'all didn't seem worth the trouble. > > Mauro> Am I missing something? > > Both before and after your patch, the defcustom and the docstring > disagree. The docstring says 'all', but thatʼs not a value you can get > via customize. > > Personally Iʼd be inclined to delete 'all' from the docstring. People > already using 'all' wonʼt notice, since the code will still recognize > it. > > Robert I've incorporated your suggestion in the attached patch. [-- Attachment #2: 0001-Fix-term-scroll-to-bottom-on-output-type-Bug-66071.patch --] [-- Type: text/x-patch, Size: 1549 bytes --] From d9f7da3ce524a9603c808f51e34af994602234d9 Mon Sep 17 00:00:00 2001 From: Mauro Aranda <maurooaranda@gmail.com> Date: Mon, 18 Sep 2023 09:19:04 -0300 Subject: [PATCH] Fix term-scroll-to-bottom-on-output :type (Bug#66071) * lisp/term.el (term-scroll-to-bottom-on-output): Add missing choices. Don't advertise the value all anymore. --- lisp/term.el | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/lisp/term.el b/lisp/term.el index b8466b21332..b2875e4a17f 100644 --- a/lisp/term.el +++ b/lisp/term.el @@ -486,7 +486,7 @@ term-char-mode-point-at-process-mark (defcustom term-scroll-to-bottom-on-output nil "Controls whether interpreter output causes window to scroll. -If nil, then do not scroll. If t or `all', scroll all windows showing buffer. +If nil, then do not scroll. If t, scroll all windows showing buffer. If `this', scroll only the selected window. If `others', scroll only those that are not the selected window. @@ -494,7 +494,12 @@ term-scroll-to-bottom-on-output See variable `term-scroll-show-maximum-output'. This variable is buffer-local." - :type 'boolean + :type '(choice (const :tag "Don't scroll" nil) + (const :tag "Scroll selected window only" this) + (const :tag "Scroll unselected windows" others) + ;; We also recognize `all', but we don't advertise it + ;; anymore. (Bug#66071) + (other :tag "Scroll all windows" t)) :group 'term) (defcustom term-scroll-snap-to-bottom t -- 2.34.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices 2023-09-24 11:38 ` Mauro Aranda @ 2023-09-30 23:36 ` Stefan Kangas 0 siblings, 0 replies; 8+ messages in thread From: Stefan Kangas @ 2023-09-30 23:36 UTC (permalink / raw) To: Mauro Aranda; +Cc: 66071-done Version: 29.2 Mauro Aranda <maurooaranda@gmail.com> writes: > I've incorporated your suggestion in the attached patch. Thanks, installed on emacs-29. [1: 47770b0eca4]: 2023-10-01 01:35:23 +0200 Fix term-scroll-to-bottom-on-output :type https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=47770b0eca4ba94f6b2a5dd740ff42023c8159c4 ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-09-30 23:36 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-09-18 12:17 bug#66071: 30.0.50; term-scroll-to-bottom-on-output missing choices Mauro Aranda 2023-09-18 12:20 ` Mauro Aranda 2023-09-18 14:20 ` Robert Pluim 2023-09-18 15:12 ` Mauro Aranda 2023-09-18 15:22 ` Robert Pluim 2023-09-18 15:29 ` Mauro Aranda 2023-09-24 11:38 ` Mauro Aranda 2023-09-30 23:36 ` Stefan Kangas
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/emacs.git 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 read-only IMAP folder(s) and NNTP newsgroup(s).