Jiang Xin <worldhello....@gmail.com> writes:
> Since there is an enhanced version of relative_path() in path.c,
> remove duplicate counterpart path_relative() in quote.c.
There is no nice comparison chart before and after like you had in
patch 02/16?
> void write_name_quoted_relative(const char *name, size_t len,
> const char *prefix, size_t prefix_len,
> FILE *fp, int terminator)
> {
> struct strbuf sb = STRBUF_INIT;
>
> - name = path_relative(name, len, &sb, prefix, prefix_len);
> + name = relative_path(name, prefix, &sb);
Are we sure nobody calls prefix_len pointing into the middle of
string, not at the end of "prefix"? This is unsafe for such a
caller, and to make sure we catch them, we should remove the
now-unused prefix_len parameter from this function.
> write_name_quoted(name, fp, terminator);
>
> strbuf_release(&sb);
> }
>
> -/*
> - * Give path as relative to prefix.
> - *
> - * The strbuf may or may not be used, so do not assume it contains the
> - * returned path.
> - */
> -static const char *path_relative(const char *in, int len,
> - struct strbuf *sb, const char *prefix,
> - int prefix_len)
> -{
> - int off, i;
> -
> - if (len < 0)
> - len = strlen(in);
> - if (prefix_len < 0) {
> - if (prefix)
> - prefix_len = strlen(prefix);
> - else
> - prefix_len = 0;
> - }
> -
> - off = 0;
> - i = 0;
> - while (i < prefix_len && i < len && prefix[i] == in[i]) {
> - if (prefix[i] == '/')
> - off = i + 1;
> - i++;
> - }
> - in += off;
> - len -= off;
> -
> - if (i >= prefix_len)
> - return in;
> -
> - strbuf_reset(sb);
> - strbuf_grow(sb, len);
> -
> - while (i < prefix_len) {
> - if (prefix[i] == '/')
> - strbuf_addstr(sb, "../");
> - i++;
> - }
> - strbuf_add(sb, in, len);
> -
> - return sb->buf;
> -}
> -
> /* quote path as relative to the given prefix */
> char *quote_path_relative(const char *in, int len,
> struct strbuf *out, const char *prefix)
> {
> struct strbuf sb = STRBUF_INIT;
> - const char *rel = path_relative(in, len, &sb, prefix, -1);
> + const char *rel = relative_path(in, prefix, &sb);
> strbuf_reset(out);
> quote_c_style_counted(rel, strlen(rel), out, NULL, 0);
> strbuf_release(&sb);
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
