On Sun, 15 Apr 2012 at 23:11:43 +0200, Rodolfo García Peñas wrote:
> - if (ename && eclass) {
> - int len = strlen(ename) + strlen(eclass) + 4;
> - ret = wmalloc(len);
> - snprintf(ret, len, "%s.%s", ename, eclass);
> - wfree(ename);
> - wfree(eclass);
> - } else if (ename) {
> - ret = wstrdup(ename);
> + if (class)
> + eclass = escape_word(class);
> +
> + if (ename || eclass)
> + ret = StrConcatDot(ename, eclass, False);
Here again, why not use StrConcatDot() with True instead and
simply drop the last argument?
--
To unsubscribe, send mail to [email protected].
