I'm trying out rxvt-unicode, and I discovered that setting the color to bright black (\e[90m) while the bold attribute (\e[1m) is set results in black text (equivalent to \e[0;30m). The other bright colors (91-97) do not exhibit this problem, e.g. \e[1;95m and \e[1;35m produce the same color.

The attached patch fixes this problem by correcting an off-by-one error in the fgcolor_of and bgcolor_of methods.

Cheers,
--
Ben Winslow <[email protected]>
diff -ur rxvt-unicode-9.12.orig/src/rxvt.h rxvt-unicode-9.12/src/rxvt.h
--- rxvt-unicode-9.12.orig/src/rxvt.h	2011-06-22 06:44:37.000000000 -0400
+++ rxvt-unicode-9.12/src/rxvt.h	2011-10-28 00:37:56.016328045 -0400
@@ -1464,7 +1464,7 @@
 # if ENABLE_STYLES
         && option (Opt_intensityStyles)
 # endif
-        && IN_RANGE_INC (base, minCOLOR, minBrightCOLOR))
+        && IN_RANGE_INC (base, minCOLOR, maxCOLOR))
       base += minBrightCOLOR - minCOLOR;
 #endif
     return base;
@@ -1478,7 +1478,7 @@
 # if ENABLE_STYLES
         && option (Opt_intensityStyles)
 # endif
-        && IN_RANGE_INC (base, minCOLOR, minBrightCOLOR))
+        && IN_RANGE_INC (base, minCOLOR, maxCOLOR))
       base += minBrightCOLOR - minCOLOR;
 #endif
     return base;
_______________________________________________
rxvt-unicode mailing list
[email protected]
http://lists.schmorp.de/cgi-bin/mailman/listinfo/rxvt-unicode

Reply via email to