Making bars list into linked list

This commit is contained in:
bakkeby 2020-07-18 18:58:39 +02:00
parent aff7c0d8b8
commit 346d7f6b1d

158
dwm.c
View File

@ -203,7 +203,9 @@ typedef struct Bar Bar;
struct Bar { struct Bar {
Window win; Window win;
Monitor *mon; Monitor *mon;
Bar *next;
int idx; int idx;
int topbar;
int bx, by, bw, bh; /* bar geometry */ int bx, by, bw, bh; /* bar geometry */
int w[BARRULES]; // width, array length == barrules, then use r index for lookup purposes int w[BARRULES]; // width, array length == barrules, then use r index for lookup purposes
int x[BARRULES]; // x position, array length == ^ int x[BARRULES]; // x position, array length == ^
@ -355,16 +357,11 @@ struct Monitor {
unsigned int sellt; unsigned int sellt;
unsigned int tagset[2]; unsigned int tagset[2];
int showbar; int showbar;
int topbar;
Client *clients; Client *clients;
Client *sel; Client *sel;
Client *stack; Client *stack;
Monitor *next; Monitor *next;
#if BAR_EXTRABAR_PATCH Bar *bar;
Bar *bars[2];
#else
Bar *bars[1];
#endif // BAR_EXTRABAR_PATCH
const Layout *lt[2]; const Layout *lt[2];
#if BAR_ALTERNATIVE_TAGS_PATCH #if BAR_ALTERNATIVE_TAGS_PATCH
unsigned int alttag; unsigned int alttag;
@ -878,10 +875,11 @@ attachstack(Client *c)
void void
buttonpress(XEvent *e) buttonpress(XEvent *e)
{ {
int click, i, b, r, mi; int click, i, r, mi;
Arg arg = {0}; Arg arg = {0};
Client *c; Client *c;
Monitor *m; Monitor *m;
Bar *bar;
XButtonPressedEvent *ev = &e->xbutton; XButtonPressedEvent *ev = &e->xbutton;
const BarRule *br; const BarRule *br;
BarClickArg carg = { 0, 0, 0, 0 }; BarClickArg carg = { 0, 0, 0, 0 };
@ -899,20 +897,20 @@ buttonpress(XEvent *e)
} }
for (mi = 0, m = mons; m && m != selmon; m = m->next, mi++); // get the monitor index for (mi = 0, m = mons; m && m != selmon; m = m->next, mi++); // get the monitor index
for (b = 0; b < LENGTH(selmon->bars); b++) { for (bar = selmon->bar; bar; bar = bar->next) {
if (ev->window == selmon->bars[b]->win) { if (ev->window == bar->win) {
for (r = 0; r < LENGTH(barrules); r++) { for (r = 0; r < LENGTH(barrules); r++) {
br = &barrules[r]; br = &barrules[r];
if (br->bar != b || (br->monitor == 'A' && m != selmon) || br->clickfunc == NULL) if (br->bar != bar->idx || (br->monitor == 'A' && m != selmon) || br->clickfunc == NULL)
continue; continue;
if (br->monitor != 'A' && br->monitor != -1 && br->monitor != mi) if (br->monitor != 'A' && br->monitor != -1 && br->monitor != mi)
continue; continue;
if (selmon->bars[b]->x[r] <= ev->x && ev->x <= selmon->bars[b]->x[r] + selmon->bars[b]->w[r]) { if (bar->x[r] <= ev->x && ev->x <= bar->x[r] + bar->w[r]) {
carg.rel_x = ev->x - selmon->bars[b]->x[r]; carg.rel_x = ev->x - bar->x[r];
carg.rel_y = ev->y; carg.rel_y = ev->y;
carg.rel_w = selmon->bars[b]->w[r]; carg.rel_w = bar->w[r];
carg.rel_h = selmon->bars[b]->bh; carg.rel_h = bar->bh;
click = br->clickfunc(selmon->bars[b], &arg, &carg); click = br->clickfunc(bar, &arg, &carg);
if (click < 0) if (click < 0)
return; return;
break; break;
@ -1005,7 +1003,7 @@ void
cleanupmon(Monitor *mon) cleanupmon(Monitor *mon)
{ {
Monitor *m; Monitor *m;
int b; Bar *bar;
if (mon == mons) if (mon == mons)
mons = mons->next; mons = mons->next;
@ -1013,12 +1011,11 @@ cleanupmon(Monitor *mon)
for (m = mons; m && m->next != mon; m = m->next); for (m = mons; m && m->next != mon; m = m->next);
m->next = mon->next; m->next = mon->next;
} }
for (b = 0; b < LENGTH(mon->bars); b++) { for (bar = mon->bar; bar; bar = mon->bar) {
if (mon->bars[b]) { XUnmapWindow(dpy, bar->win);
XUnmapWindow(dpy, mon->bars[b]->win); XDestroyWindow(dpy, bar->win);
XDestroyWindow(dpy, mon->bars[b]->win); mon->bar = bar->next;
} free(bar);
free(mon->bars[b]);
} }
free(mon); free(mon);
} }
@ -1047,11 +1044,7 @@ clientmessage(XEvent *e)
return; return;
} }
#if BAR_SYSTRAY_WIN_PATCH
c->mon = selmon; c->mon = selmon;
#else
c->mon = systray->mon;
#endif // BAR_SYSTRAY_WIN_PATCH
c->next = systray->icons; c->next = systray->icons;
systray->icons = c; systray->icons = c;
XGetWindowAttributes(dpy, c->win, &wa); XGetWindowAttributes(dpy, c->win, &wa);
@ -1148,12 +1141,12 @@ void
configurenotify(XEvent *e) configurenotify(XEvent *e)
{ {
Monitor *m; Monitor *m;
Bar *bar;
#if !FAKEFULLSCREEN_PATCH #if !FAKEFULLSCREEN_PATCH
Client *c; Client *c;
#endif // !FAKEFULLSCREEN_PATCH #endif // !FAKEFULLSCREEN_PATCH
XConfigureEvent *ev = &e->xconfigure; XConfigureEvent *ev = &e->xconfigure;
int dirty; int dirty;
int b;
/* TODO: updategeom handling sucks, needs to be simplified */ /* TODO: updategeom handling sucks, needs to be simplified */
if (ev->window == root) { if (ev->window == root) {
@ -1173,9 +1166,8 @@ configurenotify(XEvent *e)
#endif // FAKEFULLSCREEN_CLIENT_PATCH #endif // FAKEFULLSCREEN_CLIENT_PATCH
resizeclient(c, m->mx, m->my, m->mw, m->mh); resizeclient(c, m->mx, m->my, m->mw, m->mh);
#endif // !FAKEFULLSCREEN_PATCH #endif // !FAKEFULLSCREEN_PATCH
for (b = 0; b < LENGTH(m->bars); b++) for (bar = m->bar; bar; bar = bar->next)
if (m->bars[b]) XMoveResizeWindow(dpy, bar->win, bar->bx, bar->by, bar->bw, bar->bh);
XMoveResizeWindow(dpy, m->bars[b]->win, m->bars[b]->bx, m->bars[b]->by, m->bars[b]->bw, m->bars[b]->bh);
} }
focus(NULL); focus(NULL);
arrange(NULL); arrange(NULL);
@ -1243,9 +1235,12 @@ Monitor *
createmon(void) createmon(void)
{ {
Monitor *m; Monitor *m;
int i; int i, n, mi, max_bars = 2, istopbar = topbar;
const BarRule *br;
Bar *bar;
#if MONITOR_RULES_PATCH #if MONITOR_RULES_PATCH
int mi, j; int j;
Monitor *mon; Monitor *mon;
const MonitorRule *mr; const MonitorRule *mr;
#endif // MONITOR_RULES_PATCH #endif // MONITOR_RULES_PATCH
@ -1262,7 +1257,6 @@ createmon(void)
m->nstack = nstack; m->nstack = nstack;
#endif // FLEXTILE_DELUXE_LAYOUT #endif // FLEXTILE_DELUXE_LAYOUT
m->showbar = showbar; m->showbar = showbar;
m->topbar = topbar;
#if SETBORDERPX_PATCH #if SETBORDERPX_PATCH
m->borderpx = borderpx; m->borderpx = borderpx;
#endif // SETBORDERPX_PATCH #endif // SETBORDERPX_PATCH
@ -1273,19 +1267,8 @@ createmon(void)
m->gappov = gappov; m->gappov = gappov;
#endif // VANITYGAPS_PATCH #endif // VANITYGAPS_PATCH
for (i = 0; i < LENGTH(m->bars); i++) {
fprintf(stderr, "creating bar, idx = %d\n", i);
m->bars[i] = ecalloc(1, sizeof(Bar));
m->bars[i]->mon = m;
m->bars[i]->idx = i;
fprintf(stderr, "%s\n", "finished creating bar");
}
// for (r = 0; r < LENGTH(barrules); r++) {
// for (b = 0; b < LENGTH(selmon->bars); b++) {
#if MONITOR_RULES_PATCH
for (mi = 0, mon = mons; mon; mon = mon->next, mi++); // monitor index for (mi = 0, mon = mons; mon; mon = mon->next, mi++); // monitor index
#if MONITOR_RULES_PATCH
for (j = 0; j < LENGTH(monrules); j++) { for (j = 0; j < LENGTH(monrules); j++) {
mr = &monrules[j]; mr = &monrules[j];
if ((mr->monitor == -1 || mr->monitor == mi) if ((mr->monitor == -1 || mr->monitor == mi)
@ -1304,7 +1287,7 @@ createmon(void)
if (mr->showbar > -1) if (mr->showbar > -1)
m->showbar = mr->showbar; m->showbar = mr->showbar;
if (mr->topbar > -1) if (mr->topbar > -1)
m->topbar = mr->topbar; istopbar = mr->topbar;
break; break;
} }
} }
@ -1314,6 +1297,23 @@ createmon(void)
strncpy(m->ltsymbol, layouts[0].symbol, sizeof m->ltsymbol); strncpy(m->ltsymbol, layouts[0].symbol, sizeof m->ltsymbol);
#endif // MONITOR_RULES_PATCH #endif // MONITOR_RULES_PATCH
/* Derive the number of bars for this monitor based on bar rules */
for (n = -1, i = 0; i < LENGTH(barrules); i++) {
br = &barrules[i];
if (br->monitor == 'A' || br->monitor == -1 || br->monitor == mi)
n = MAX(br->bar, n);
}
for (i = 0; i <= n && i < max_bars; i++) {
bar = ecalloc(1, sizeof(Bar));
bar->mon = m;
bar->idx = i;
bar->next = m->bar;
bar->topbar = istopbar;
m->bar = bar;
istopbar = !istopbar;
}
#if FLEXTILE_DELUXE_LAYOUT #if FLEXTILE_DELUXE_LAYOUT
m->ltaxis[LAYOUT] = m->lt[0]->preset.layout; m->ltaxis[LAYOUT] = m->lt[0]->preset.layout;
m->ltaxis[MASTER] = m->lt[0]->preset.masteraxis; m->ltaxis[MASTER] = m->lt[0]->preset.masteraxis;
@ -1450,8 +1450,9 @@ dirtomon(int dir)
void void
drawbar(Monitor *m) drawbar(Monitor *m)
{ {
for (int b = 0; b < LENGTH(m->bars); b++) Bar *bar;
drawbarwin(m->bars[b]); for (bar = m->bar; bar; bar = bar->next)
drawbarwin(bar);
} }
void void
@ -1499,7 +1500,7 @@ drawbarwin(Bar *bar)
w = br->widthfunc(bar, &warg); w = br->widthfunc(bar, &warg);
w = MIN(warg.max_width, w); w = MIN(warg.max_width, w);
if (lw <= 0) { // if left is exhausted, switch to right side if (lw <= 0) { // if left is exhausted then switch to right side, and vice versa
lw = rw; lw = rw;
lx = rx; lx = rx;
} else if (rw <= 0) { } else if (rw <= 0) {
@ -1957,7 +1958,7 @@ manage(Window w, XWindowAttributes *wa)
c->y = c->mon->my + c->mon->mh - HEIGHT(c); c->y = c->mon->my + c->mon->mh - HEIGHT(c);
c->x = MAX(c->x, c->mon->mx); c->x = MAX(c->x, c->mon->mx);
/* only fix client y-offset, if the client center might cover the bar */ /* only fix client y-offset, if the client center might cover the bar */
c->y = MAX(c->y, ((c->mon->bars[0]->by == c->mon->my) && (c->x + (c->w / 2) >= c->mon->wx) c->y = MAX(c->y, ((c->mon->bar->by == c->mon->my) && (c->x + (c->w / 2) >= c->mon->wx)
&& (c->x + (c->w / 2) < c->mon->wx + c->mon->ww)) ? bh : c->mon->my); && (c->x + (c->w / 2) < c->mon->wx + c->mon->ww)) ? bh : c->mon->my);
#if !FLOATPOS_PATCH #if !FLOATPOS_PATCH
#if SETBORDERPX_PATCH #if SETBORDERPX_PATCH
@ -2481,7 +2482,7 @@ restack(Monitor *m)
XRaiseWindow(dpy, m->sel->win); XRaiseWindow(dpy, m->sel->win);
if (m->lt[m->sellt]->arrange) { if (m->lt[m->sellt]->arrange) {
wc.stack_mode = Below; wc.stack_mode = Below;
wc.sibling = m->bars[0]->win; wc.sibling = m->bar->win;
for (c = m->stack; c; c = c->snext) for (c = m->stack; c; c = c->snext)
if (!c->isfloating && ISVISIBLE(c)) { if (!c->isfloating && ISVISIBLE(c)) {
XConfigureWindow(dpy, c->win, CWSibling|CWStackMode, &wc); XConfigureWindow(dpy, c->win, CWSibling|CWStackMode, &wc);
@ -3149,7 +3150,7 @@ tagmon(const Arg *arg)
void void
togglebar(const Arg *arg) togglebar(const Arg *arg)
{ {
int b; Bar *bar;
#if BAR_HOLDBAR_PATCH && PERTAG_PATCH && PERTAGBAR_PATCH #if BAR_HOLDBAR_PATCH && PERTAG_PATCH && PERTAGBAR_PATCH
selmon->showbar = selmon->pertag->showbars[selmon->pertag->curtag] = (selmon->showbar == 2 ? 1 : !selmon->showbar); selmon->showbar = selmon->pertag->showbars[selmon->pertag->curtag] = (selmon->showbar == 2 ? 1 : !selmon->showbar);
#elif BAR_HOLDBAR_PATCH #elif BAR_HOLDBAR_PATCH
@ -3160,9 +3161,8 @@ togglebar(const Arg *arg)
selmon->showbar = !selmon->showbar; selmon->showbar = !selmon->showbar;
#endif // BAR_HOLDBAR_PATCH | PERTAG_PATCH #endif // BAR_HOLDBAR_PATCH | PERTAG_PATCH
updatebarpos(selmon); updatebarpos(selmon);
for (b = 0; b < LENGTH(selmon->bars); b++) for (bar = selmon->bar; bar; bar = bar->next)
if (selmon->bars[b]->win) XMoveResizeWindow(dpy, bar->win, bar->bx, bar->by, bar->bw, bar->bh);
XMoveResizeWindow(dpy, selmon->bars[b]->win, selmon->bars[b]->bx, selmon->bars[b]->by, selmon->bars[b]->bw, selmon->bars[b]->bh);
arrange(selmon); arrange(selmon);
} }
@ -3425,7 +3425,7 @@ unmapnotify(XEvent *e)
void void
updatebars(void) updatebars(void)
{ {
int b; Bar *bar;
Monitor *m; Monitor *m;
XSetWindowAttributes wa = { XSetWindowAttributes wa = {
.override_redirect = True, .override_redirect = True,
@ -3440,20 +3440,20 @@ updatebars(void)
}; };
XClassHint ch = {"dwm", "dwm"}; XClassHint ch = {"dwm", "dwm"};
for (m = mons; m; m = m->next) { for (m = mons; m; m = m->next) {
for (b = 0; b < LENGTH(m->bars); b++) { for (bar = m->bar; bar; bar = bar->next) {
if (!m->bars[b]->win) { // TODO add static status controls to not create / show extra bar? if (!bar->win) { // TODO add static status controls to not create / show extra bar?
#if BAR_ALPHA_PATCH #if BAR_ALPHA_PATCH
m->bars[b]->win = XCreateWindow(dpy, root, m->bars[b]->bx, m->bars[b]->by, m->bars[b]->bw, m->bars[b]->bh, 0, depth, bar->win = XCreateWindow(dpy, root, bar->bx, bar->by, bar->bw, bar->bh, 0, depth,
InputOutput, visual, InputOutput, visual,
CWOverrideRedirect|CWBackPixel|CWBorderPixel|CWColormap|CWEventMask, &wa); CWOverrideRedirect|CWBackPixel|CWBorderPixel|CWColormap|CWEventMask, &wa);
#else #else
m->bars[b]->win = XCreateWindow(dpy, root, m->bars[b]->bx, m->bars[b]->by, m->bars[b]->bw, m->bars[b]->bh, 0, DefaultDepth(dpy, screen), bar->win = XCreateWindow(dpy, root, bar->bx, bar->by, bar->bw, bar->bh, 0, DefaultDepth(dpy, screen),
CopyFromParent, DefaultVisual(dpy, screen), CopyFromParent, DefaultVisual(dpy, screen),
CWOverrideRedirect|CWBackPixmap|CWEventMask, &wa); CWOverrideRedirect|CWBackPixmap|CWEventMask, &wa);
#endif // BAR_ALPHA_PATCH #endif // BAR_ALPHA_PATCH
XDefineCursor(dpy, m->bars[b]->win, cursor[CurNormal]->cursor); XDefineCursor(dpy, bar->win, cursor[CurNormal]->cursor);
XMapRaised(dpy, m->bars[b]->win); XMapRaised(dpy, bar->win);
XSetClassHint(dpy, m->bars[b]->win, &ch); XSetClassHint(dpy, bar->win, &ch);
} }
} }
} }
@ -3464,7 +3464,8 @@ updatebarpos(Monitor *m)
{ {
m->wy = m->my; m->wy = m->my;
m->wh = m->mh; m->wh = m->mh;
int num_bars, b; int num_bars;
Bar *bar;
#if BAR_PADDING_PATCH #if BAR_PADDING_PATCH
int y_pad = vertpad; int y_pad = vertpad;
int x_pad = sidepad; int x_pad = sidepad;
@ -3473,20 +3474,22 @@ updatebarpos(Monitor *m)
int x_pad = 0; int x_pad = 0;
#endif // BAR_PADDING_PATCH #endif // BAR_PADDING_PATCH
num_bars = LENGTH(m->bars); for (num_bars = 0, bar = m->bar; bar; bar = bar->next, num_bars++);
if (m->showbar) { if (m->showbar)
m->wh = m->wh - y_pad * num_bars - bh * num_bars; m->wh = m->wh - y_pad * num_bars - bh * num_bars;
m->wy = m->wy + bh + y_pad;
}
for (b = 0; b < num_bars; b++) { for (bar = m->bar; bar; bar = bar->next) {
m->bars[b]->bx = m->mx + x_pad; bar->bx = m->mx + x_pad;
m->bars[b]->bw = m->ww - 2 * x_pad; bar->bw = m->ww - 2 * x_pad;
m->bars[b]->bh = bh; bar->bh = bh;
if (m->showbar) { if (m->showbar) {
m->bars[b]->by = m->topbar == b ? m->wy + m->wh : m->wy - bh; if (bar->topbar) {
m->wy = m->wy + bh + y_pad;
bar->by = m->wy - bh;
} else
bar->by = m->wy + m->wh;
} else { } else {
m->bars[b]->by = -bh - y_pad; bar->by = -bh - y_pad;
} }
} }
} }
@ -3783,15 +3786,16 @@ wintoclient(Window w)
Monitor * Monitor *
wintomon(Window w) wintomon(Window w)
{ {
int x, y, b; int x, y;
Client *c; Client *c;
Monitor *m; Monitor *m;
Bar *bar;
if (w == root && getrootptr(&x, &y)) if (w == root && getrootptr(&x, &y))
return recttomon(x, y, 1, 1); return recttomon(x, y, 1, 1);
for (m = mons; m; m = m->next) for (m = mons; m; m = m->next)
for (b = 0; b < LENGTH(m->bars); b++) for (bar = m->bar; bar; bar = bar->next)
if (w == m->bars[b]->win) if (w == bar->win)
return m; return m;
if ((c = wintoclient(w))) if ((c = wintoclient(w)))
return c->mon; return c->mon;