Re: [PATCH 13/24] builtin/pack-objects: convert to struct object_id

From: Stefan Beller <sbeller@google.com>
Date: Oct 2, 2017, 23:20 UTC
Message-ID: <CAGZ79kbULxhSAEx6x6+4Q5nCG4pT98AXVfLBMwQHf-Kx4Wzc=Q@mail.gmail.com>
In-Reply-To: <20171001220831.214705-14-sandals@crustytoothpaste.net>

Show 8 quoted lines

> @@ -2520,17 +2521,17 @@ static void read_object_list_from_stdin(void)
>                         continue;
>                 }
>                 if (line[0] == '-') {
> -                       if (get_sha1_hex(line+1, sha1))
> +                       if (get_oid_hex(line+1, &oid))
>                                 die("expected edge sha1, got garbage:\n %s",
>                                     line);

...

> -               if (get_sha1_hex(line, sha1))
> +               if (parse_oid_hex(line, &oid, &p))
>                         die("expected sha1, got garbage:\n %s", line);

Ideally we do not forget about converting the die() calls, too.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.