osmocom
/
wireshark
11
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

Update a comment about Windows Console color support. 

Change-Id: I39171a3c95ca48a39fbbc90ceadb9b1436f2402c
Reviewed-on: https://code.wireshark.org/review/25196
Reviewed-by: Michael Mann <mmann78@netscape.net>
This commit is contained in:
Gerald Combs 2018-01-08 10:04:18 -08:00 committed by Michael Mann
parent 15746e5922
commit a6a5b552ec
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
epan/print_stream.c
View File

@ -43,9 +43,16 @@ print_color_escape(FILE *fh, const color_t *fg, const color_t *bg)
WORD win_fg_color = FOREGROUND_RED|FOREGROUND_BLUE|FOREGROUND_GREEN;
WORD win_bg_color = 0;
/* Windows seems to offer 1-bit color, so you can't set the red, green, or blue intensities,
* you can only set "{foreground, background} contains {red, green, blue}".
* So include red, green or blue if the numeric intensity is high enough
/* The classic Windows Console offers 1-bit color, so you can't set
* the red, green, or blue intensities, you can only set
* "{foreground, background} contains {red, green, blue}". So
* include red, green or blue if the numeric intensity is high
* enough.
*
* The console in Windows 10 builds 14931 and later supports SGR RGB
* sequences. We might want to print those instead depending on the
* version of Windows or just remove the SetConsoleTextAttribute
* calls and only print SGR sequences.
*/
if (fg) {
if (((fg->red >> 8) & 0xff) >= 0x80)