git.stevedylan.dev

cleanup schemes and colors 77f96d70

Hiltjo Posthuma · 2025-09-27 12:10 3 file(s) · +30 −4

dmenu.c +3 −1

	size_t i;

	XUngrabKeyboard(dpy, CurrentTime);
	for (i = 0; i < SchemeLast; i++)
	for (i = 0; i < SchemeLast; i++) {
		drw_scm_free(drw, scheme[i], 2);
		free(scheme[i]);
	}
	for (i = 0; items && items[i].text; ++i)
		free(items[i].text);
	free(items);

drw.c +25 −3

		die("error, cannot allocate color '%s'", clrname);
}

/* Wrapper to create color schemes. The caller has to call free(3) on the
 * returned color scheme when done using it. */
/* Create color schemes. */
Clr *
drw_scm_create(Drw *drw, const char *clrnames[], size_t clrcount)
{

	Clr *ret;

	/* need at least two colors for a scheme */
	if (!drw || !clrnames || clrcount < 2 || !(ret = ecalloc(clrcount, sizeof(XftColor))))
	if (!drw || !clrnames || clrcount < 2 || !(ret = ecalloc(clrcount, sizeof(Clr))))
		return NULL;

	for (i = 0; i < clrcount; i++)
		drw_clr_create(drw, &ret[i], clrnames[i]);
	return ret;
}

void
drw_clr_free(Drw *drw, Clr *c)
{
	if (!drw || !c)
		return;

	/* c is typedef XftColor Clr */
	XftColorFree(drw->dpy, DefaultVisual(drw->dpy, drw->screen),
	             DefaultColormap(drw->dpy, drw->screen), c);
}

void
drw_scm_free(Drw *drw, Clr *scm, size_t clrcount)
{
	size_t i;

	if (!drw || !scm)
		return;

	for (i = 0; i < clrcount; i++)
		drw_clr_free(drw, &scm[i]);
}

void

drw.h +2 −0


/* Colorscheme abstraction */
void drw_clr_create(Drw *drw, Clr *dest, const char *clrname);
void drw_clr_free(Drw *drw, Clr *c);
Clr *drw_scm_create(Drw *drw, const char *clrnames[], size_t clrcount);
void drw_scm_free(Drw *drw, Clr *scm, size_t clrcount);

/* Cursor abstraction */
Cur *drw_cur_create(Drw *drw, int shape);

101	101		size_t i;
102	102
103	103		XUngrabKeyboard(dpy, CurrentTime);
104		-	for (i = 0; i < SchemeLast; i++)
	104	+	for (i = 0; i < SchemeLast; i++) {
	105	+	drw_scm_free(drw, scheme[i], 2);
105	106		free(scheme[i]);
	107	+	}
106	108		for (i = 0; items && items[i].text; ++i)
107	109		free(items[i].text);
108	110		free(items);

178	178		die("error, cannot allocate color '%s'", clrname);
179	179		}
180	180
181		-	/* Wrapper to create color schemes. The caller has to call free(3) on the
182		-	* returned color scheme when done using it. */
	181	+	/* Create color schemes. */
183	182		Clr *
184	183		drw_scm_create(Drw drw, const char clrnames[], size_t clrcount)
185	184		{

187	186		Clr *ret;
188	187
189	188		/* need at least two colors for a scheme */
190		-	if (!drw \|\| !clrnames \|\| clrcount < 2 \|\| !(ret = ecalloc(clrcount, sizeof(XftColor))))
	189	+	if (!drw \|\| !clrnames \|\| clrcount < 2 \|\| !(ret = ecalloc(clrcount, sizeof(Clr))))
191	190		return NULL;
192	191
193	192		for (i = 0; i < clrcount; i++)
194	193		drw_clr_create(drw, &ret[i], clrnames[i]);
195	194		return ret;
	195	+	}
	196	+
	197	+	void
	198	+	drw_clr_free(Drw drw, Clr c)
	199	+	{
	200	+	if (!drw \|\| !c)
	201	+	return;
	202	+
	203	+	/* c is typedef XftColor Clr */
	204	+	XftColorFree(drw->dpy, DefaultVisual(drw->dpy, drw->screen),
	205	+	DefaultColormap(drw->dpy, drw->screen), c);
	206	+	}
	207	+
	208	+	void
	209	+	drw_scm_free(Drw drw, Clr scm, size_t clrcount)
	210	+	{
	211	+	size_t i;
	212	+
	213	+	if (!drw \|\| !scm)
	214	+	return;
	215	+
	216	+	for (i = 0; i < clrcount; i++)
	217	+	drw_clr_free(drw, &scm[i]);
196	218		}
197	219
198	220		void

40	40
41	41		/* Colorscheme abstraction */
42	42		void drw_clr_create(Drw drw, Clr dest, const char *clrname);
	43	+	void drw_clr_free(Drw drw, Clr c);
43	44		Clr drw_scm_create(Drw drw, const char *clrnames[], size_t clrcount);
	45	+	void drw_scm_free(Drw drw, Clr scm, size_t clrcount);
44	46
45	47		/* Cursor abstraction */
46	48		Cur drw_cur_create(Drw drw, int shape);