Brandon Williams <bmwill@xxxxxxxxxx> wrote:
> +++ b/run-command.c
> @@ -211,14 +211,82 @@ static const char **prepare_shell_cmd(struct argv_array *out, const char **argv)
> #ifndef GIT_WINDOWS_NATIVE
> static int child_notifier = -1;
>
> -static void notify_parent(void)
> +enum child_errcode {
> + CHILD_ERR_CHDIR,
> + CHILD_ERR_ENOENT,
> + CHILD_ERR_SILENT,
> + CHILD_ERR_ERRNO,
> +};
I realize I introduced this in my original, but trailing commas
on the last enum value might not be portable. Checking other
enum usages in our tree suggests we omit the last comma.