Re: [hackers] [PATCH 4/8] drw: simplify color schemes

From: Markus Teich <markus.teich_AT_stusta.mhn.de>
Date: Mon, 23 May 2016 21:12:27 +0200

Hiltjo Posthuma wrote:
> I meant using:
>
> typedef XftColor Scm
>
> and *Scm.
>
> instead of
>
> typedef XftColor *Scm
>
> not sure tbh, maybe it's fine as it is now.

Hm, that option is quite good actually. It still hides XftColor being used, but
also exposes that a color scheme is an array which is needed in dwm to access
the border color component. If there are no objections, I'd update the patches
to use `typedef XftColor Scm`.

--Markus
Received on Mon May 23 2016 - 21:12:27 CEST