[hackers] [dwm] use buffer instead of pointer for mon->ltsymbol || Anselm R Garbe

From: <hg_AT_suckless.org>
Date: Tue, 22 Sep 2009 08:53:15 +0000 (UTC)

changeset: 1496:de4a2998e1f5
tag: tip
user: Anselm R Garbe <garbeam_AT_gmail.com>
date: Tue Sep 22 09:53:11 2009 +0100
files: dwm.c
description:
use buffer instead of pointer for mon->ltsymbol

diff -r c4c94cff1c3b -r de4a2998e1f5 dwm.c
--- a/dwm.c Tue Sep 22 09:16:48 2009 +0100
+++ b/dwm.c Tue Sep 22 09:53:11 2009 +0100
@@ -121,7 +121,7 @@
 } Layout;
 
 struct Monitor {
- const char *ltsymbol;
+ char ltsymbol[16];
         float mfact;
         int num;
         int by; /* bar geometry */
@@ -386,7 +386,7 @@
                 showhide(m->stack);
         focus(NULL);
         for(m = mons; m; m = m->next) {
- m->ltsymbol = m->lt[m->sellt]->symbol;
+ strncpy(m->ltsymbol, m->lt[m->sellt]->symbol, sizeof m->ltsymbol);
                 if(m->lt[m->sellt]->arrange)
                         m->lt[m->sellt]->arrange(m);
                 restack(m);
@@ -605,7 +605,7 @@
         m->topbar = topbar;
         m->lt[0] = &layouts[0];
         m->lt[1] = &layouts[1 % LENGTH(layouts)];
- m->ltsymbol = layouts[0].symbol;
+ strncpy(m->ltsymbol, layouts[0].symbol, sizeof m->ltsymbol);
         return m;
 }
 
@@ -1163,17 +1163,14 @@
 
 void
 monocle(Monitor *m) {
- static char ntext[8];
         unsigned int n = 0;
         Client *c;
 
         for(c = m->clients; c; c = c->next)
                 if(ISVISIBLE(c))
                         n++;
- if(n > 0) { /* override layout symbol */
- snprintf(ntext, sizeof ntext, "[%d]", n);
- m->ltsymbol = ntext;
- }
+ if(n > 0) /* override layout symbol */
+ snprintf(m->ltsymbol, sizeof m->ltsymbol, "[%d]", n);
         for(c = nexttiled(m->clients); c; c = nexttiled(c->next))
                 resize(c, m->wx, m->wy, m->ww - 2 * c->bw, m->wh - 2 * c->bw, False);
 }
Received on Tue Sep 22 2009 - 08:53:15 UTC

This archive was generated by hypermail 2.2.0 : Tue Sep 22 2009 - 09:00:08 UTC