Stefan Beller wrote:

[...]
> --- a/submodule.c
> +++ b/submodule.c
[...]
> @@ -1070,11 +1070,12 @@ unsigned is_submodule_modified(const char *path, int 
> ignore_untracked)
[...]
>       while (strbuf_getwholeline(&buf, fp, '\n') != EOF) {
> -             if ((buf.buf[0] == '?') && (buf.buf[1] == '?'))
> +             /* regular untracked files */
> +             if (buf.buf[0] == '?')
>                       dirty_submodule |= DIRTY_SUBMODULE_UNTRACKED;
>               else
>                       dirty_submodule |= DIRTY_SUBMODULE_MODIFIED;

At first I didn't believe it could be so simple. :)

Unlike ordinary files that use 1, 2, or u, ignored files use '!'.
You're not passing --ignored so you don't have to worry about them.

Reviewed-by: Jonathan Nieder <jrnie...@gmail.com>

Very nice.

Reply via email to