Code

Add a notice that only certain functions can print color escape codes
authorJohannes Sixt <j6t@kdbg.org>
Fri, 27 Nov 2009 07:42:25 +0000 (08:42 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 27 Nov 2009 08:28:40 +0000 (00:28 -0800)
We emulate color escape codes on Windows by overriding printf, fprintf,
and fputs. Warn developers that these are the only functions that can be
used to print them.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
color.h

diff --git a/color.h b/color.h
index 7d8da6fe2207ae4adb7ab888edd57ab571665458..3cb4b7fc890880b0fcf19a11c6bc7de6b10d6e8d 100644 (file)
--- 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().
+ */
 #define GIT_COLOR_NORMAL       ""
 #define GIT_COLOR_RESET                "\033[m"
 #define GIT_COLOR_BOLD         "\033[1m"