On Thu, May 24, 2018 at 9:25 AM, Derrick Stolee <dstolee@xxxxxxxxxxxxx> wrote:
> Signed-off-by: Derrick Stolee <dstolee@xxxxxxxxxxxxx>
> ---
> builtin/commit-graph.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/builtin/commit-graph.c b/builtin/commit-graph.c
> index 37420ae0fd..f0875b8bf3 100644
> --- a/builtin/commit-graph.c
> +++ b/builtin/commit-graph.c
> @@ -51,8 +51,11 @@ static int graph_read(int argc, const char **argv)
> graph_name = get_commit_graph_filename(opts.obj_dir);
> graph = load_commit_graph_one(graph_name);
>
> - if (!graph)
> + if (!graph) {
> + UNLEAK(graph_name);
> die("graph file %s does not exist", graph_name);
Unrelated to this patch: Is the command that ends up die()ing here
a plumbing or porcelain, or: Do we want to translate the message here?
In a lot of commands that show paths we single quote them '%s',
(speaking from experience with a lot of submodule path code)
