David Barr <davidbarr@xxxxxxxxxx> writes:
> Comparisons in svndump.c are always guarded by length.
> As a bonus, elimate dependency on prefixcmp for upstream.
>
> Signed-off-by: David Barr <davidbarr@xxxxxxxxxx>
It feels suboptimal, from cross-project maintenance point of view,
that "do not use prefixcmp() in the source in this directory" has to
be an unwritten rule. Is there something we can do better to avoid
having to apply a patch like this in the future?
> vcs-svn/svndump.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/vcs-svn/svndump.c b/vcs-svn/svndump.c
> index 0899790..8d0ae9c 100644
> --- a/vcs-svn/svndump.c
> +++ b/vcs-svn/svndump.c
> @@ -361,7 +361,7 @@ void svndump_read(const char *url)
> reset_rev_ctx(atoi(val));
> break;
> case sizeof("Node-path"):
> - if (prefixcmp(t, "Node-"))
> + if (constcmp(t, "Node-"))
> continue;
> if (!constcmp(t + strlen("Node-"), "path")) {
> if (active_ctx == NODE_CTX)
--
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