Cleanup style nit of 'x == NULL' in remote.c
Git style tends to prefer "!x" over "x == NULL". Make it so in these handful of locations that were not following along. Signed-off-by: Shawn O. Pearce <spearce@spearce.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
f4e95765b0
commit
009c5bcd24
6
remote.c
6
remote.c
@ -416,7 +416,7 @@ int remote_find_tracking(struct remote *remote, struct refspec *refspec)
|
|||||||
int i;
|
int i;
|
||||||
|
|
||||||
if (find_src) {
|
if (find_src) {
|
||||||
if (refspec->dst == NULL)
|
if (!refspec->dst)
|
||||||
return error("find_tracking: need either src or dst");
|
return error("find_tracking: need either src or dst");
|
||||||
needle = refspec->dst;
|
needle = refspec->dst;
|
||||||
result = &refspec->src;
|
result = &refspec->src;
|
||||||
@ -613,7 +613,7 @@ static int match_explicit(struct ref *src, struct ref *dst,
|
|||||||
if (!matched_src)
|
if (!matched_src)
|
||||||
errs = 1;
|
errs = 1;
|
||||||
|
|
||||||
if (dst_value == NULL)
|
if (!dst_value)
|
||||||
dst_value = matched_src->name;
|
dst_value = matched_src->name;
|
||||||
|
|
||||||
switch (count_refspec_match(dst_value, dst, &matched_dst)) {
|
switch (count_refspec_match(dst_value, dst, &matched_dst)) {
|
||||||
@ -633,7 +633,7 @@ static int match_explicit(struct ref *src, struct ref *dst,
|
|||||||
dst_value);
|
dst_value);
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
if (errs || matched_dst == NULL)
|
if (errs || !matched_dst)
|
||||||
return 1;
|
return 1;
|
||||||
if (matched_dst->peer_ref) {
|
if (matched_dst->peer_ref) {
|
||||||
errs = 1;
|
errs = 1;
|
||||||
|
Loading…
Reference in New Issue
Block a user