@miconda commented on this pull request.
if (result == NULL || t == NULL) { PKG_MEM_ERROR; goto clean; }
- snprintf(result, elements->len - toRemove->len, "%*s", elements->len - toRemove->len, ""); - snprintf(t, elements->len+1, "%s", elements->s); + memset(result, 32, elements->len - toRemove->len);
What is the purpose of filling `result` buffer with space (char code `32`)?
}
}
if (elements->len-toRemove->len > 0) { - snprintf(elements->s, elements->len, "%*s", elements->len-toRemove->len, ""); + memset(elements->s, 32, elements->len - toRemove->len); snprintf(elements->s, (strlen(result)%elements->len)+1, "%s", result);
Another `memset` with char code 32 (space), then `snprintf` in the same buffer.
Maybe it is ok, but `memset` with `space` is quite unusual, therefore asking for further details.