Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

configurable colors for addresses #88

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tecknicaltom
Copy link

This is just a rebase (and moving to a branch like I should have done in the first place) of pull request #52 . I didn't even realize that pointers to rodata were colorized until I started looking at the peda source code due to my green terminal text. This change fixes that by allowing users to configure the colors used for pointers. As I said in the other PR:

I use a terminal with green text. This means that with peda, rodata (colored green) and value (no color) both look green and I miss out on context. I think a setup like this is common among people who use peda.

This change introduces the ability to specify colors for code, data, rodata, and values via settings. Defaults match the existing setup. Now, in my .gdbinit, after sourcing peda.py I have:

peda set option color_addr_value white

This commit only allows for the configuration of address colors. Other color coding is not yet configurable.

I use a terminal with green text. This means that with peda, rodata
(colored green) and value (no color) both look green and I miss out on
context. I think a setup like this is common among people who use peda.

This change introduces the ability to specify colors for code, data,
rodata, and values via settings. Defaults match the existing setup. Now,
in my .gdbinit, after sourcing peda.py I have:

  peda set option color_addr_value white

This commit only allows for the configuration of address colors. Other
color coding is not yet configurable.
@longld longld mentioned this pull request Apr 8, 2017
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.

1 participant