My first patch, I'm hoping this is the way you want them submitted. I
couldn't find any reason for ColBorder to be a #define while ColFg and
ColBg are in an enum together.

---

>From eefea3c310db3c9460c5fdee3e8a8d0cb45c9819 Mon Sep 17 00:00:00 2001
From: ude <ude@ude>
Date: Wed, 14 Mar 2018 10:01:00 -0400
Subject: [PATCH] ColBorder has been moved to the enum with ColFg and ColBg.

---
 drw.h | 2 +-
 dwm.c | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/drw.h b/drw.h
index 4c67419..4bcd5ad 100644
--- a/drw.h
+++ b/drw.h
@@ -12,7 +12,7 @@ typedef struct Fnt {
  struct Fnt *next;
 } Fnt;

-enum { ColFg, ColBg }; /* Clr scheme index */
+enum { ColFg, ColBg, ColBorder }; /* Clr scheme index */
 typedef XftColor Clr;

 typedef struct {
diff --git a/dwm.c b/dwm.c
index ec6a27c..ab16c75 100644
--- a/dwm.c
+++ b/dwm.c
@@ -56,7 +56,6 @@
 #define HEIGHT(X)               ((X)->h + 2 * (X)->bw)
 #define TAGMASK                 ((1 << LENGTH(tags)) - 1)
 #define TEXTW(X)                (drw_fontset_getwidth(drw, (X)) + lrpad)
-#define ColBorder               2

 /* enums */
 enum { CurNormal, CurResize, CurMove, CurLast }; /* cursor */
-- 
2.7.4

Reply via email to