From: Lars Ingebrigtsen <larsi@gnus.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 21777@debbugs.gnu.org, schwab@linux-m68k.org,
Dima Kogan <dima@secretsauce.net>
Subject: bug#21777: 25.0.50; gud-gdb uses a pager, which is harmful inside emacs
Date: Sun, 20 Sep 2020 23:18:32 +0200 [thread overview]
Message-ID: <878sd4jgev.fsf@gnus.org> (raw)
In-Reply-To: <838u6k1z28.fsf@gnu.org> (Eli Zaretskii's message of "Fri, 30 Oct 2015 11:32:47 +0200")
Eli Zaretskii <eliz@gnu.org> writes:
> IME, users sometimes have some use cases that look really weird to me,
> but are somehow very important to them. So I'm trying to avoid
> stepping on their toes as much as possible, even if I cannot imagine
> those use cases in advance.
I worked up a patch for this, but in testing, there's a number of
problems.
1) When resizing a frame, the height in gdb is overwritten. Where does
that come from? I tried grepping, but couldn't find anything.
2) Sending a command this way to gdb gives us an extra prompt, which
isn't nice.
3) If you start gdb with M-x gdb instead of M-x gud-gdb, the height
variable in gdb is always switched off:
(gdb-input "-gdb-set height 0" 'ignore)
So why shouldn't gud-gdb do the same thing?
diff --git a/doc/emacs/building.texi b/doc/emacs/building.texi
index 7074bd45d7..ac54c570cd 100644
--- a/doc/emacs/building.texi
+++ b/doc/emacs/building.texi
@@ -1040,6 +1040,15 @@ GDB User Interface Layout
need to check that the breakpoints in recently edited source files are
still in the right places.
+@vindex gud-pager-height
+ GDB normally uses a pager when displaying output (like backtraces).
+This can be inconvenient when running inside Emacs, so the default
+value of @code{gud-pager-height} is @code{unlimited}, meaning that the
+pager is disabled. This variable can also be a number (which is used
+as the number of lines by the pager), or @code{nil}, meaning that the
+pager setting shouldn't be changed from the gdb defaults (or
+@file{.gdbinit} settings).
+
@node Source Buffers
@subsubsection Source Buffers
@cindex fringes, for debugging
diff --git a/etc/NEWS b/etc/NEWS
index 1f52341ae4..3014eddbdd 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -1034,6 +1034,10 @@ window after starting). This variable defaults to nil.
** Miscellaneous
+*** New user option 'gud-pager-height'
+This variable is used to set up the gdb pager, and defaults to
+`unlimited', meaning that a pager won't be used.
+
+++
*** Interactive regular expression search now uses faces for sub-groups.
E.g., 'C-M-s foo-\([0-9]+\)' will now use the 'isearch-group-1' face
diff --git a/lisp/progmodes/gud.el b/lisp/progmodes/gud.el
index 84c473ddb7..7acc540b01 100644
--- a/lisp/progmodes/gud.el
+++ b/lisp/progmodes/gud.el
@@ -67,6 +67,19 @@ gud-key-prefix
:type 'key-sequence
:group 'gud)
+(defcustom gud-pager-height 'unlimited
+ "The size of a page when gdb displays output.
+gdb normally uses a pager when displaying things like
+backtraces (or other things that take a lot of room). This
+variable allows customizing the pager when gdb is run from Emacs.
+
+This can be a number (the number of lines), `unlimited' (don't
+use the pager at all), or nil (don't change the defaults)."
+ :type '(choice (const :tag "No paging" unlimited)
+ integer
+ (const :tag "Don't set" nil))
+ :version "28.1")
+
(global-set-key (vconcat gud-key-prefix "\C-l") 'gud-refresh)
;; (define-key ctl-x-map " " 'gud-break); backward compatibility hack
@@ -796,6 +809,9 @@ gud-gdb
(setq gdb-first-prompt t)
(setq gud-running nil)
(setq gud-filter-pending-text nil)
+
+ (when gud-pager-height
+ (gud-basic-call (format "set height %s" gud-pager-height)))
(run-hooks 'gud-gdb-mode-hook))
;; The completion process filter indicates when it is finished.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
next prev parent reply other threads:[~2020-09-20 21:18 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-28 19:20 bug#21777: 25.0.50; gud-gdb uses a pager, which is harmful inside emacs Dima Kogan
2015-10-28 22:27 ` Andreas Schwab
2015-10-28 22:57 ` Dima Kogan
2015-10-29 16:16 ` Eli Zaretskii
2015-10-29 22:58 ` Dima Kogan
2015-10-30 3:43 ` Dima Kogan
2015-10-30 8:17 ` Eli Zaretskii
2015-10-30 9:13 ` Dima Kogan
2015-10-30 9:32 ` Eli Zaretskii
2015-10-30 14:01 ` Eli Zaretskii
2015-10-30 19:05 ` Dima Kogan
2015-10-30 20:34 ` Eli Zaretskii
2020-09-20 21:18 ` Lars Ingebrigtsen [this message]
2020-09-21 14:03 ` Eli Zaretskii
2020-09-21 14:07 ` Lars Ingebrigtsen
2020-09-22 15:46 ` Eli Zaretskii
2020-09-22 15:51 ` Lars Ingebrigtsen
2020-09-22 15:57 ` Andreas Schwab
2020-09-22 16:09 ` Eli Zaretskii
2020-09-22 16:19 ` Andreas Schwab
2020-09-22 16:51 ` Eli Zaretskii
2020-09-22 17:18 ` Andreas Schwab
2020-09-22 17:39 ` Eli Zaretskii
2020-09-22 17:46 ` Andreas Schwab
2020-09-23 12:53 ` Lars Ingebrigtsen
2020-09-23 14:41 ` Eli Zaretskii
2020-09-24 14:15 ` Lars Ingebrigtsen
2020-09-24 15:06 ` Eli Zaretskii
2020-09-25 9:55 ` Lars Ingebrigtsen
2020-09-24 14:16 ` Eli Zaretskii
2020-09-24 14:29 ` Lars Ingebrigtsen
2020-09-24 15:08 ` Eli Zaretskii
2015-10-30 7:54 ` Eli Zaretskii
2019-09-29 21:49 ` Stefan Kangas
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=878sd4jgev.fsf@gnus.org \
--to=larsi@gnus.org \
--cc=21777@debbugs.gnu.org \
--cc=dima@secretsauce.net \
--cc=eliz@gnu.org \
--cc=schwab@linux-m68k.org \
/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.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.