From: Jonathan Tan <jonathantanmy@google.com>
Date: Sat, 05 Nov 2016 02:34:27 GMT
Subject: Re: [PATCH v2 1/6] submodules: add helper functions to determine presence of submodules
Message-ID: <313ffcf0-b75d-425d-050a-1eaa2f850f1e@google.com>
In-Reply-To: <1477953496-103596-2-git-send-email-bmwill@google.com>

On 10/31/2016 03:38 PM, Brandon Williams wrote:
> +		struct strbuf buf = STRBUF_INIT;
> +		char *submodule_url = NULL;
> +
> +		strbuf_addf(&buf, "submodule.%s.url", module->name);
> +		ret = !git_config_get_string(buf.buf, &submodule_url);
> +
> +		free(submodule_url);
> +		strbuf_release(&buf);
> +	}
> +
> +	return ret;
> +}
> +
> +/*
> + * Determine if a submodule has been checked out at a given 'path'
> + */
> +int is_submodule_checked_out(const char *path)
> +{
> +	int ret = 0;
> +	struct strbuf buf = STRBUF_INIT;
> +
> +	strbuf_addf(&buf, "%s/.git", path);
> +	ret = file_exists(buf.buf);
> +
> +	strbuf_release(&buf);

In this and the previous function, you can use xstrfmt.