public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug cli/17151] New: pagination should be per line
@ 2014-07-14  2:50 xdje42 at gmail dot com
  2022-01-09  1:32 ` [Bug cli/17151] " tromey at sourceware dot org
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: xdje42 at gmail dot com @ 2014-07-14  2:50 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17151

            Bug ID: 17151
           Summary: pagination should be per line
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: minor
          Priority: P2
         Component: cli
          Assignee: unassigned at sourceware dot org
          Reporter: xdje42 at gmail dot com

While playing with bug 16809 I discovered this.

set height 1
i thr

-->

(gdb) i thr
---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
Id---Type <return> to continue, or q <return> to quit---
  ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
Target Id---Type <return> to continue, or q <return> to quit---
        ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
Frame---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---

---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
5---Type <return> to continue, or q <return> to quit---
   ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
Thread 0x7ffff67c6700 (LWP 31214) "forever-threads"---Type <return> to
continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
(running)
---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
4---Type <return> to continue, or q <return> to quit---
   ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
Thread 0x7ffff6fc7700 (LWP 31213) "forever-threads"---Type <return> to
continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
(running)
---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
3---Type <return> to continue, or q <return> to quit---
   ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
Thread 0x7ffff77c8700 (LWP 31212) "forever-threads"---Type <return> to
continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
(running)
---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
2---Type <return> to continue, or q <return> to quit---
   ---Type <return> to continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
Thread 0x7ffff7fc9700 (LWP 31211) "forever-threads"---Type <return> to
continue, or q <return> to quit---
 ---Type <return> to continue, or q <return> to quit---
(running)
---Type <return> to continue, or q <return> to quit---
[at this point I gave up]

A height of 2 works ok.
So either 1 needs to be disallowed or something more sensible done.

(gdb) set height 2
(gdb) i thr
  Id   Target Id         Frame 
---Type <return> to continue, or q <return> to quit---
  5    Thread 0x7ffff67c6700 (LWP 31214) "forever-threads" (running)
---Type <return> to continue, or q <return> to quit---
  4    Thread 0x7ffff6fc7700 (LWP 31213) "forever-threads" (running)
---Type <return> to continue, or q <return> to quit---
  3    Thread 0x7ffff77c8700 (LWP 31212) "forever-threads" (running)
---Type <return> to continue, or q <return> to quit---
  2    Thread 0x7ffff7fc9700 (LWP 31211) "forever-threads" (running)
---Type <return> to continue, or q <return> to quit---
* 1    Thread 0x7ffff7fcb740 (LWP 31207) "forever-threads" (running)

Testcase: forever-threads.c from bug 17147.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug cli/17151] pagination should be per line
  2014-07-14  2:50 [Bug cli/17151] New: pagination should be per line xdje42 at gmail dot com
@ 2022-01-09  1:32 ` tromey at sourceware dot org
  2022-01-11 14:46 ` tromey at sourceware dot org
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: tromey at sourceware dot org @ 2022-01-09  1:32 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17151

Tom Tromey <tromey at sourceware dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tromey at sourceware dot org

--- Comment #1 from Tom Tromey <tromey at sourceware dot org> ---
I think ignoring the height=1 case is probably simplest.
Having a 1 line terminal is pathological anyway.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug cli/17151] pagination should be per line
  2014-07-14  2:50 [Bug cli/17151] New: pagination should be per line xdje42 at gmail dot com
  2022-01-09  1:32 ` [Bug cli/17151] " tromey at sourceware dot org
@ 2022-01-11 14:46 ` tromey at sourceware dot org
  2022-04-15 17:54 ` cvs-commit at gcc dot gnu.org
  2022-04-15 17:55 ` tromey at sourceware dot org
  3 siblings, 0 replies; 5+ messages in thread
From: tromey at sourceware dot org @ 2022-01-11 14:46 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17151

Tom Tromey <tromey at sourceware dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at sourceware dot org   |tromey at sourceware dot org

--- Comment #2 from Tom Tromey <tromey at sourceware dot org> ---
I have a fix for this.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug cli/17151] pagination should be per line
  2014-07-14  2:50 [Bug cli/17151] New: pagination should be per line xdje42 at gmail dot com
  2022-01-09  1:32 ` [Bug cli/17151] " tromey at sourceware dot org
  2022-01-11 14:46 ` tromey at sourceware dot org
@ 2022-04-15 17:54 ` cvs-commit at gcc dot gnu.org
  2022-04-15 17:55 ` tromey at sourceware dot org
  3 siblings, 0 replies; 5+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-04-15 17:54 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17151

--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tom Tromey <tromey@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=111d19818a4a7c96c87e330c6c143fa03bb86819

commit 111d19818a4a7c96c87e330c6c143fa03bb86819
Author: Tom Tromey <tom@tromey.com>
Date:   Sat Jan 8 19:37:38 2022 -0700

    Handle "set height 1"

    PR cli/17151 points out that "set height 1" has pathological behavior
    in gdb.  What I see is that gdb will endlessly print the pagination
    prompt.  This patch takes a simple and expedient approach to a fix:
    pretend that the height is really 2.

    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=17151

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug cli/17151] pagination should be per line
  2014-07-14  2:50 [Bug cli/17151] New: pagination should be per line xdje42 at gmail dot com
                   ` (2 preceding siblings ...)
  2022-04-15 17:54 ` cvs-commit at gcc dot gnu.org
@ 2022-04-15 17:55 ` tromey at sourceware dot org
  3 siblings, 0 replies; 5+ messages in thread
From: tromey at sourceware dot org @ 2022-04-15 17:55 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17151

Tom Tromey <tromey at sourceware dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |13.1
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #4 from Tom Tromey <tromey at sourceware dot org> ---
Fixed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2022-04-15 17:55 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-14  2:50 [Bug cli/17151] New: pagination should be per line xdje42 at gmail dot com
2022-01-09  1:32 ` [Bug cli/17151] " tromey at sourceware dot org
2022-01-11 14:46 ` tromey at sourceware dot org
2022-04-15 17:54 ` cvs-commit at gcc dot gnu.org
2022-04-15 17:55 ` tromey at sourceware dot org

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).