Heyho,

Thiébaud Weksteen wrote:
> -                     if(llen == 0 && len != 0) {
> +                     caps_state = ev.xkey.state && LockMask;
> +                     if((llen == 0 && len != 0)) {

Why the extra parentheses in the if statement?

> -                                     XSetWindowBackground(dpy, 
> locks[screen]->win, locks[screen]->colors[1]);
> +                                     if(caps_state)
> +                                             XSetWindowBackground(dpy, 
> locks[screen]->win, locks[screen]->colors[2]);
> +                                     else
> +                                             XSetWindowBackground(dpy, 
> locks[screen]->win, locks[screen]->colors[1]);

How about the ?: operator here?

--Markus

Reply via email to