Skip to content

Conversation

@davidwelch158
Copy link
Contributor

Summary of changes

Disable pagination on gdb startup. Otherwise the GUI can't accept commands while gdb is waiting for the user to press enter to continue. Also since the gdb window has scrollbars pagination seems pointless.

Test plan

Test suite run successfully.
You can see the difference with a command like gdbgui -g '/usr/bin/gdb -ex "help breakpoints"'

David Welch added 3 commits January 18, 2021 14:52
…e user to press 'enter' to continue.

Use '-iex' rather '-ex' for the gdb startup arguments so they execute before .gdbinit is executed, in case the .gdbinit generates any output requiring paging.
@bjorn3
Copy link

bjorn3 commented Jan 18, 2021

Pagination is still useful for when for example 50000 items would be printed. Without pagination the console would either overflow or it would be hard to find what was immediately before the command that printed 50000 items.

@cs01
Copy link
Owner

cs01 commented May 20, 2021

Pagination is still useful for when for example 50000 items would be printed. Without pagination the console would either overflow or it would be hard to find what was immediately before the command that printed 50000 items.

I tend to agree with this. I can see how it might be annoying at startup if there is pagination from gdb's initial output, but usability might suffer in certain real world scenarios during debugging. If startup is the main annoyance for you, maybe gdbgui -g 'gdb -q' would suffice?

@cs01
Copy link
Owner

cs01 commented May 20, 2021

And thank you for your patience on my reply to this PR, I have been unable to give much attention to work for a few months.

@cs01
Copy link
Owner

cs01 commented Aug 28, 2021

After thinking about this some more, I am going to merge this because I want to optimize for the most common user scenarios. I think it's more common to have pagination be an annoyance than a help, so it can be turned off by default.

@cs01 cs01 merged commit 170daed into cs01:master Aug 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants