Johannes Sixt venit, vidit, dixit 27.11.2009 08:42: > Junio C Hamano schrieb: >> Johannes Sixt <j.sixt@xxxxxxxxxxxxx> writes: >> >>> Like this? >> >> Yeah, and in addition to "puts", "write(2)" is also not supported, right? > > Correct, good catch! > > --- 8< --- > From: Johannes Sixt <j6t@xxxxxxxx> > Subject: [PATCH] Add a notice that only certain functions can print color escape codes > > We emulate color escape codes on Windows by overriding printf, fprintf, > and fputs. Warn users that these are the only functions that can be used > to print them. > > Signed-off-by: Johannes Sixt <j6t@xxxxxxxx> > --- > color.h | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/color.h b/color.h > index 7d8da6f..3cb4b7f 100644 > --- a/color.h > +++ b/color.h > @@ -4,6 +4,11 @@ > /* "\033[1;38;5;2xx;48;5;2xxm\0" is 23 bytes */ > #define COLOR_MAXLEN 24 > > +/* > + * IMPORTANT: Due to the way these color codes are emulated on Windows, > + * write them only using printf(), fprintf(), and fputs(). In particular, > + * do not use puts() or write(). Nor putchar nor putc... but they're part of the puts family, of course. More worthwhile then listing those negatives could be a hint on another positive: color_fprintf and friends which are declared at the end of color.h. > + */ > #define GIT_COLOR_NORMAL "" > #define GIT_COLOR_RESET "\033[m" > #define GIT_COLOR_BOLD "\033[1m" -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html