Junio C Hamano <gitster@xxxxxxxxx> a écrit :
Erik Faye-Lund <kusmabite@xxxxxxxxx> writes:
The awkward thing about doing this, is that the memory allocated by
the strbuf cannot be reclaimed if you go with this. A pointer that has
been adjusted (like cleanup_path can do) cannot be successfully fed to
free.
Yeah, I wouldn't recommend doing that. Either
path = strbuf_detach(&sb, NULL);
retval = xstrdup(cleanup_path(path));
free(path);
return retval;
or
path = xstrdup(cleanup_path(sb.buf));
strbuf_release(&sb);
return path;
would be more sensible.
Erik, what do you prefer ? You can have the final answer.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html