Re: [PATCH 30/32] ident.c: fix LGTM warning on the possible abuse of the '=' operator
- From
- Phillip Wood <phillip.wood123@gmail.com>
- Date
- Nov 4, 2019, 15:11 UTC
- Message-ID
- <98f12b0a-d16b-13e2-e413-9230d3d31783@gmail.com>
- In-Reply-To
- <CA+EOSBncQa0h1of+R2BBSw=tSNFmSASi2V5EJPaRNEO_ctvbkw@mail.gmail.com>
Hi Elia
On 04/11/2019 13:55, Elia Pinto wrote:
Show 13 quoted lines
> Il giorno lun 4 nov 2019 alle ore 11:26 SZEDER Gábor > <szeder.dev@gmail.com> ha scritto: >> >> On Mon, Nov 04, 2019 at 09:59:21AM +0000, Elia Pinto wrote: >>> Fix the LGTM warning of the rule that finds uses of the assignment >> >> What is an "LGTM warning"? >> >> I think including the actual compiler warning in the commit message >> would be great. > Yes indeed. I thought I did it, do you think i can do better? Thanks > > https://lgtm.com/rules/2158670641/
It would have been helpful to explain that LGTM was a static analyser for those of us who did not know. As far the patch is concerned I'm not convinced it is an improvement. There are loads of places where git uses this pattern ("git grep 'if (([^=)]*=[^)]*)' | wc" shows 212). So long as the assignment is inside its own set of parentheses it's safe and gcc will warn if the parentheses are missing.
Best Wishes
Phillip
Show 37 quoted lines
>>
>>> operator = in places where the equality operator == would
>>> make more sense.
>>>
>>> Signed-off-by: Elia Pinto <gitter.spiros@gmail.com>
>>> ---
>>> ident.c | 13 ++++++++-----
>>> 1 file changed, 8 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/ident.c b/ident.c
>>> index e666ee4e59..07f2f03b0a 100644
>>> --- a/ident.c
>>> +++ b/ident.c
>>> @@ -172,12 +172,15 @@ const char *ident_default_email(void)
>>> strbuf_addstr(&git_default_email, email);
>>> committer_ident_explicitly_given |= IDENT_MAIL_GIVEN;
>>> author_ident_explicitly_given |= IDENT_MAIL_GIVEN;
>>> - } else if ((email = query_user_email()) && email[0]) {
>>> - strbuf_addstr(&git_default_email, email);
>>> - free((char *)email);
>>> - } else
>>> - copy_email(xgetpwuid_self(&default_email_is_bogus),
>>> + } else {
>>> + email = query_user_email();
>>> + if (email && email[0]) {
>>> + strbuf_addstr(&git_default_email, email);
>>> + free((char *)email);
>>> + } else
>>> + copy_email(xgetpwuid_self(&default_email_is_bogus),
>>> &git_default_email, &default_email_is_bogus);
>>> + }
>>> strbuf_trim(&git_default_email);
>>> }
>>> return git_default_email.buf;
>>> --
>>> 2.24.0.rc0.467.g566ccdd3e4.dirty
>>>