Volume XXII, number 280Wednesday, October 7, 2026Latest message 1 hour ago

The Git List

News and archive of git@vger.kernel.org, since April 2005

patchparse-options: fix sparse 'plain integer as NULL pointer'

4 messages between Apr 23, 2026 and Apr 29, 2026, from Ramsay Jones, Junio C Hamano, Johannes Schindelin, Jiamu Sun.

Plain Markdown or JSON for tools and agents. Diffs are folded; open one to read it.

Ramsay JonesApr 23, 2026, 16:05 UTC on lore
Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
---
Hi Jiamu Sun,

If you need to re-roll your 'js/parseopt-subcommand-autocorrection' branch, could you please squash this into the patch corresponding to commit b9e6a2d30a ("parseopt: autocorrect mistyped subcommands", 2026-04-23).

Thanks.

ATB, Ramsay Jones

 parse-options.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Show changes to parse-options.c +1 −1
diff --git a/parse-options.c b/parse-options.c
index d60e7bd3c9..14f3f385eb 100644
--- a/parse-options.c
+++ b/parse-options.c
@@ -658,7 +658,7 @@ static const char *autocorrect_subcommand(const char *cmd,
 
 	for_each_string_list_item(cand, cmds) {
 		if (starts_with(cand->string, cmd)) {
-			cand->util = 0;
+			cand->util = NULL;
 		} else {
 			int edit = levenshtein(cmd, cand->string,
 					       0, 2, 1, 3) + 1;

-- 
2.54.0
Junio C HamanoApr 24, 2026, 02:47 UTC in reply to Ramsay Jones on lore

Re: [PATCH] parse-options: fix sparse 'plain integer as NULL pointer'

Another thing from GitHub CI
https://github.com/git/git/actions/runs/24825391649/job/72659919418#step:9:144
  Error: parse-options.c:680:30: comparison of integer expressions of
different signedness: 'unsigned int' and 'int' [-Werror=sign-compare]
    680 |        (n < cmds->nr && best == (intptr_t)cmds->items[n].util);
        |                              ^~
2026年4月24日(金) 1:05 Ramsay Jones <ramsay@ramsayjones.plus.com>:
Show 35 quoted lines
>
>
> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
> ---
>
> Hi Jiamu Sun,
>
> If you need to re-roll your 'js/parseopt-subcommand-autocorrection'
> branch, could you please squash this into the patch corresponding
> to commit b9e6a2d30a ("parseopt: autocorrect mistyped subcommands",
> 2026-04-23).
>
> Thanks.
>
> ATB,
> Ramsay Jones
>
>  parse-options.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/parse-options.c b/parse-options.c
> index d60e7bd3c9..14f3f385eb 100644
> --- a/parse-options.c
> +++ b/parse-options.c
> @@ -658,7 +658,7 @@ static const char *autocorrect_subcommand(const char *cmd,
>
>         for_each_string_list_item(cand, cmds) {
>                 if (starts_with(cand->string, cmd)) {
> -                       cand->util = 0;
> +                       cand->util = NULL;
>                 } else {
>                         int edit = levenshtein(cmd, cand->string,
>                                                0, 2, 1, 3) + 1;
> --
> 2.54.0
Johannes SchindelinApr 26, 2026, 12:14 UTC in reply to Junio C Hamano on lore

Re: [PATCH] parse-options: fix sparse 'plain integer as NULL pointer'

Hi Junio,
On Sun, 26 Apr 2026, Junio C Hamano wrote:
Show 8 quoted lines
> Another thing from GitHub CI
> 
> https://github.com/git/git/actions/runs/24825391649/job/72659919418#step:9:144
> 
>   Error: parse-options.c:680:30: comparison of integer expressions of
> different signedness: 'unsigned int' and 'int' [-Werror=sign-compare]
>     680 |        (n < cmds->nr && best == (intptr_t)cmds->items[n].util);
>         |                              ^~

Indeed. With this patch on top of Ramsay's fixup, it passes the build: https://github.com/dscho/git/actions/runs/24955417618

-- snip --
From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Sun, 26 Apr 2026 11:11:35 +0000
Subject: [PATCH] fixup! parseopt: autocorrect mistyped subcommands

Fix a build warning in `linux32`: Change `best` from `unsigned int` to `intptr_t` to match the `(intptr_t)` casts it is compared against, fixing a sign-compare warning on 32-bit platforms where `intptr_t` is `int`.

Assisted-by: Claude Opus 4.6
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
 parse-options.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Show changes to parse-options.c +2 −2
diff --git a/parse-options.c b/parse-options.c
index 14f3f385eb4..0ba6905fed5 100644
--- a/parse-options.c
+++ b/parse-options.c
@@ -648,7 +648,8 @@ static const char *autocorrect_subcommand(const char *cmd,
 					  struct string_list *cmds)
 {
 	struct autocorrect autocorrect = { 0 };
-	unsigned int n = 0, best = 0;
+	unsigned int n = 0;
+	intptr_t best = 0;
 	struct string_list_item *cand;
 
 	autocorrect_resolve(&autocorrect);
-- snap --

Ciao,
Johannes

> 
> 2026年4月24日(金) 1:05 Ramsay Jones <ramsay@ramsayjones.plus.com>:
> >
> >
> > Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
> > ---
> >
> > Hi Jiamu Sun,
> >
> > If you need to re-roll your 'js/parseopt-subcommand-autocorrection'
> > branch, could you please squash this into the patch corresponding
> > to commit b9e6a2d30a ("parseopt: autocorrect mistyped subcommands",
> > 2026-04-23).
> >
> > Thanks.
> >
> > ATB,
> > Ramsay Jones
> >
> >  parse-options.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/parse-options.c b/parse-options.c
> > index d60e7bd3c9..14f3f385eb 100644
> > --- a/parse-options.c
> > +++ b/parse-options.c
> > @@ -658,7 +658,7 @@ static const char *autocorrect_subcommand(const char *cmd,
> >
> >         for_each_string_list_item(cand, cmds) {
> >                 if (starts_with(cand->string, cmd)) {
> > -                       cand->util = 0;
> > +                       cand->util = NULL;
> >                 } else {
> >                         int edit = levenshtein(cmd, cand->string,
> >                                                0, 2, 1, 3) + 1;
> > --
> > 2.54.0
> 
>
Jiamu SunApr 29, 2026, 05:13 UTC in reply to Johannes Schindelin on lore

Re: [PATCH] parse-options: fix sparse 'plain integer as NULL pointer'

On Sun, Apr 26, 2026 at 02:14:03PM +0200, Johannes Schindelin wrote:
Show 44 quoted lines
> Indeed. With this patch on top of Ramsay's fixup, it passes the build:
> 
> Assisted-by: Claude Opus 4.6
> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> ---
>  parse-options.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/parse-options.c b/parse-options.c
> index 14f3f385eb4..0ba6905fed5 100644
> --- a/parse-options.c
> +++ b/parse-options.c
> @@ -648,7 +648,8 @@ static const char *autocorrect_subcommand(const char *cmd,
>  					  struct string_list *cmds)
>  {
>  	struct autocorrect autocorrect = { 0 };
> -	unsigned int n = 0, best = 0;
> +	unsigned int n = 0;
> +	intptr_t best = 0;
>  	struct string_list_item *cand;
>  
>  	autocorrect_resolve(&autocorrect);
> -- snap --
> 
> > > Ramsay Jones
> > >
> > >  parse-options.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/parse-options.c b/parse-options.c
> > > index d60e7bd3c9..14f3f385eb 100644
> > > --- a/parse-options.c
> > > +++ b/parse-options.c
> > > @@ -658,7 +658,7 @@ static const char *autocorrect_subcommand(const char *cmd,
> > >
> > >         for_each_string_list_item(cand, cmds) {
> > >                 if (starts_with(cand->string, cmd)) {
> > > -                       cand->util = 0;
> > > +                       cand->util = NULL;
> > >                 } else {
> > >                         int edit = levenshtein(cmd, cand->string,
> > >                                                0, 2, 1, 3) + 1;
> > > --
> > > 2.54.0
Will squash these in next re-roll.
Thanks everyone for catching these issues and providing the fixups.
-- 
Jiamu Sun <39@barroit.sh>
          <sunjiamu@outlook.com>

Back to recent threads