Skip to content

Improve color output for ZydisInfo and ZydisDisasm#433

Merged
flobernd merged 2 commits intomasterfrom
no-color
Apr 5, 2023
Merged

Improve color output for ZydisInfo and ZydisDisasm#433
flobernd merged 2 commits intomasterfrom
no-color

Conversation

@flobernd
Copy link
Copy Markdown
Member

@flobernd flobernd commented Apr 4, 2023

This adds support for NO_COLOR and FORCE_COLOR (required in rare cases where output is redirected but VT100 sequences should still be emitted - e.g. when the output is displayed in a TTY later).

Additionally adds color support for ZydisDisasm.

@flobernd flobernd requested a review from athre0z April 4, 2023 19:52
@flobernd flobernd changed the title ZydisInfo: Add NO_COLOR and FORCE_COLOR support Improve color output for ZydisInfo and ZydisDisasm Apr 5, 2023
Copy link
Copy Markdown
Member

@athre0z athre0z left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shiny! LGTM.

@flobernd flobernd merged commit 4a661c6 into master Apr 5, 2023
@flobernd flobernd deleted the no-color branch April 5, 2023 22:12
athre0z pushed a commit that referenced this pull request Feb 4, 2024
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.

2 participants