# [PATCH] builtin/blame.c: Fix a "Using plain integer as NULL pointer" warning

3 messages from 2012-05-13 to 2012-05-15. Participants: Ramsay Jones, René Scharfe.
Thread: https://gitlist.dev/t/30526

## Ramsay Jones, 2012-05-13 21:41

Subject: [PATCH] builtin/blame.c: Fix a "Using plain integer as NULL pointer" warning
Message-ID: <4FB02A7C.1000605@ramsay1.demon.co.uk>
URL: https://gitlist.dev/e/4FB02A7C.1000605%40ramsay1.demon.co.uk

```


Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
---

Hi Junio,

I try to catch these warnings, while the topics are still in the
pu branch, so that we can squash the fix into them before they
hit next. I don't know how I missed this one (commit 4b4132f,
"blame: factor out helper for calling xdi_diff()", 09-05-2012)
which was part of the 'rs/xdiff-lose-emit-func' branch.

Sorry about that.

ATB,
Ramsay Jones


 builtin/blame.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/blame.c b/builtin/blame.c
index 778d661..24d3dd5 100644
--- a/builtin/blame.c
+++ b/builtin/blame.c
@@ -93,7 +93,7 @@ static int diff_hunks(mmfile_t *file_a, mmfile_t *file_b, long ctxlen,
 {
 	xpparam_t xpp = {0};
 	xdemitconf_t xecfg = {0};
-	xdemitcb_t ecb = {0};
+	xdemitcb_t ecb = {NULL};
 
 	xpp.flags = xdl_opts;
 	xecfg.ctxlen = ctxlen;
-- 
1.7.10

```

## René Scharfe, 2012-05-13 23:39

Subject: Re: [PATCH] builtin/blame.c: Fix a "Using plain integer as NULL pointer" warning
Message-ID: <4FB0463D.4010909@lsrfire.ath.cx>
URL: https://gitlist.dev/e/4FB0463D.4010909%40lsrfire.ath.cx
In-Reply-To: <4FB02A7C.1000605@ramsay1.demon.co.uk>

```
Am 13.05.2012 23:41, schrieb Ramsay Jones:
>
>
> Signed-off-by: Ramsay Jones<ramsay@ramsay1.demon.co.uk>
> ---
>
> Hi Junio,
>
> I try to catch these warnings, while the topics are still in the
> pu branch, so that we can squash the fix into them before they
> hit next. I don't know how I missed this one (commit 4b4132f,
> "blame: factor out helper for calling xdi_diff()", 09-05-2012)
> which was part of the 'rs/xdiff-lose-emit-func' branch.
>
> Sorry about that.
>
> ATB,
> Ramsay Jones
>
>
>   builtin/blame.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/builtin/blame.c b/builtin/blame.c
> index 778d661..24d3dd5 100644
> --- a/builtin/blame.c
> +++ b/builtin/blame.c
> @@ -93,7 +93,7 @@ static int diff_hunks(mmfile_t *file_a, mmfile_t *file_b, long ctxlen,
>   {
>   	xpparam_t xpp = {0};
>   	xdemitconf_t xecfg = {0};
> -	xdemitcb_t ecb = {0};
> +	xdemitcb_t ecb = {NULL};
>
>   	xpp.flags = xdl_opts;
>   	xecfg.ctxlen = ctxlen;

The warning is given by sparse, not a C compiler, correct?  It's 
probably worth shutting it up by making that change; my excuse for using 
{0} is that it is such a nice and portable idiom [1] for initializing 
any structure, however.

René


[1] http://www.ex-parrot.com/~chris/random/initialise.html

```

## Ramsay Jones, 2012-05-15 17:33

Subject: Re: [PATCH] builtin/blame.c: Fix a "Using plain integer as NULL pointer" warning
Message-ID: <4FB29358.90408@ramsay1.demon.co.uk>
URL: https://gitlist.dev/e/4FB29358.90408%40ramsay1.demon.co.uk
In-Reply-To: <4FB0463D.4010909@lsrfire.ath.cx>

```
René Scharfe wrote:
> Am 13.05.2012 23:41, schrieb Ramsay Jones:
>>
>> Signed-off-by: Ramsay Jones<ramsay@ramsay1.demon.co.uk>
>> ---
>>
>> Hi Junio,
>>
>> I try to catch these warnings, while the topics are still in the
>> pu branch, so that we can squash the fix into them before they
>> hit next. I don't know how I missed this one (commit 4b4132f,
>> "blame: factor out helper for calling xdi_diff()", 09-05-2012)
>> which was part of the 'rs/xdiff-lose-emit-func' branch.
>>
>> Sorry about that.
>>
>> ATB,
>> Ramsay Jones
>>
>>
>>   builtin/blame.c |    2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/builtin/blame.c b/builtin/blame.c
>> index 778d661..24d3dd5 100644
>> --- a/builtin/blame.c
>> +++ b/builtin/blame.c
>> @@ -93,7 +93,7 @@ static int diff_hunks(mmfile_t *file_a, mmfile_t *file_b, long ctxlen,
>>   {
>>   	xpparam_t xpp = {0};
>>   	xdemitconf_t xecfg = {0};
>> -	xdemitcb_t ecb = {0};
>> +	xdemitcb_t ecb = {NULL};
>>
>>   	xpp.flags = xdl_opts;
>>   	xecfg.ctxlen = ctxlen;
> 
> The warning is given by sparse, not a C compiler, correct?  It's 
> probably worth shutting it up by making that change; my excuse for using 
> {0} is that it is such a nice and portable idiom [1] for initializing 
> any structure, however.

Yes, sorry, it is indeed a sparse warning. (The original
subject line mentioned sparse, but it was too long ... ;-)

Thanks!

ATB,
Ramsay Jones

```