Re:
- From
Stefan Beller <sbeller@google.com>
- Date
- Apr 12, 2016, 04:33 UTC
- Message-ID
- <CAGZ79kYD7o3O_-zSGD24_DUy9hDc9=u6pPYXOOWp+-i-PW7vjQ@mail.gmail.com>
- In-Reply-To
- <001a11492f107354a305303a369a@google.com>
On Mon, Apr 11, 2016 at 12:04 PM, <miwilliams@google.com> wrote:
Show 8 quoted lines
> From 7201fe08ede76e502211a781250c9a0b702a78b2 Mon Sep 17 00:00:00 2001 > From: Mike Williams <miwilliams@google.com> > Date: Mon, 11 Apr 2016 14:18:39 -0400 > Subject: [PATCH 1/1] wt-status: Remove '!!' from > wt_status_collect_changed_cb > > The wt_status_collect_changed_cb function uses an extraneous double negation > (!!)
How is an !! errornous?
It serves the purpose to map an integer value(-1,0,1,2,3,4) to a boolean (0,1, or a real bit in a bit field).
Show 30 quoted lines
> when determining whether or not a submodule has new commits. > > Signed-off-by: Mike Williams <miwilliams@google.com> > --- > wt-status.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/wt-status.c b/wt-status.c > index ef74864..b955179 100644 > --- a/wt-status.c > +++ b/wt-status.c > @@ -431,7 +431,7 @@ static void wt_status_collect_changed_cb(struct > diff_queue_struct *q, > d->worktree_status = p->status; > d->dirty_submodule = p->two->dirty_submodule; > if (S_ISGITLINK(p->two->mode)) > - d->new_submodule_commits = !!hashcmp(p->one->sha1, > p->two->sha1); > + d->new_submodule_commits = hashcmp(p->one->sha1, > p->two->sha1); > } > } > > -- > 2.8.0 > > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html