Browse Source

prefixcmp(): fix-up leftover strncmp().

There were instances of strncmp() that were formatted improperly
(e.g. whitespace around parameter before closing parenthesis)
that caused the earlier mechanical conversion step to miss
them.  This step cleans them up.

Signed-off-by: Junio C Hamano <junkio@cox.net>
maint
Junio C Hamano 18 years ago
parent
commit
1968d77dd6
  1. 2
      builtin-ls-tree.c
  2. 2
      builtin-rev-parse.c
  3. 4
      builtin-show-branch.c
  4. 2
      diff.c
  5. 2
      http-fetch.c
  6. 2
      http-push.c
  7. 8
      imap-send.c
  8. 2
      revision.c
  9. 2
      wt-status.c

2
builtin-ls-tree.c

@ -118,7 +118,7 @@ int cmd_ls_tree(int argc, const char **argv, const char *prefix)
chomp_prefix = 0; chomp_prefix = 0;
break; break;
} }
if (!strncmp(argv[1]+2, "abbrev=",7)) { if (!prefixcmp(argv[1]+2, "abbrev=")) {
abbrev = strtoul(argv[1]+9, NULL, 10); abbrev = strtoul(argv[1]+9, NULL, 10);
if (abbrev && abbrev < MINIMUM_ABBREV) if (abbrev && abbrev < MINIMUM_ABBREV)
abbrev = MINIMUM_ABBREV; abbrev = MINIMUM_ABBREV;

2
builtin-rev-parse.c

@ -233,7 +233,7 @@ int cmd_rev_parse(int argc, const char **argv, const char *prefix)
} }
continue; continue;
} }
if (!strncmp(arg,"-n",2)) { if (!prefixcmp(arg, "-n")) {
if ((filter & DO_FLAGS) && (filter & DO_REVS)) if ((filter & DO_FLAGS) && (filter & DO_REVS))
show(arg); show(arg);
continue; continue;

4
builtin-show-branch.c

@ -378,7 +378,7 @@ static int append_head_ref(const char *refname, const unsigned char *sha1, int f
{ {
unsigned char tmp[20]; unsigned char tmp[20];
int ofs = 11; int ofs = 11;
if (strncmp(refname, "refs/heads/", ofs)) if (prefixcmp(refname, "refs/heads/"))
return 0; return 0;
/* If both heads/foo and tags/foo exists, get_sha1 would /* If both heads/foo and tags/foo exists, get_sha1 would
* get confused. * get confused.
@ -392,7 +392,7 @@ static int append_remote_ref(const char *refname, const unsigned char *sha1, int
{ {
unsigned char tmp[20]; unsigned char tmp[20];
int ofs = 13; int ofs = 13;
if (strncmp(refname, "refs/remotes/", ofs)) if (prefixcmp(refname, "refs/remotes/"))
return 0; return 0;
/* If both heads/foo and tags/foo exists, get_sha1 would /* If both heads/foo and tags/foo exists, get_sha1 would
* get confused. * get confused.

2
diff.c

@ -77,7 +77,7 @@ int git_diff_ui_config(const char *var, const char *value)
diff_detect_rename_default = DIFF_DETECT_RENAME; diff_detect_rename_default = DIFF_DETECT_RENAME;
return 0; return 0;
} }
if (!prefixcmp(var, "diff.color.") || !strncmp(var, "color.diff.", 11)) { if (!prefixcmp(var, "diff.color.") || !prefixcmp(var, "color.diff.")) {
int slot = parse_diff_color_slot(var, 11); int slot = parse_diff_color_slot(var, 11);
color_parse(value, var, diff_colors[slot]); color_parse(value, var, diff_colors[slot]);
return 0; return 0;

2
http-fetch.c

@ -718,7 +718,7 @@ static int fetch_indices(struct alt_base *repo)
i++; i++;
if (i + 52 <= buffer.posn && if (i + 52 <= buffer.posn &&
!prefixcmp(data + i, " pack-") && !prefixcmp(data + i, " pack-") &&
!strncmp(data + i + 46, ".pack\n", 6)) { !prefixcmp(data + i + 46, ".pack\n")) {
get_sha1_hex(data + i + 6, sha1); get_sha1_hex(data + i + 6, sha1);
setup_index(repo, sha1); setup_index(repo, sha1);
i += 51; i += 51;

2
http-push.c

@ -1061,7 +1061,7 @@ static int fetch_indices(void)
i++; i++;
if (i + 52 < buffer.posn && if (i + 52 < buffer.posn &&
!prefixcmp(data + i, " pack-") && !prefixcmp(data + i, " pack-") &&
!strncmp(data + i + 46, ".pack\n", 6)) { !prefixcmp(data + i + 46, ".pack\n")) {
get_sha1_hex(data + i + 6, sha1); get_sha1_hex(data + i + 6, sha1);
setup_index(sha1); setup_index(sha1);
i += 51; i += 51;

8
imap-send.c

@ -1192,7 +1192,7 @@ count_messages( msg_data_t *msg )
char *p = msg->data; char *p = msg->data;


while (1) { while (1) {
if (!strncmp( "From ", p, 5 )) { if (!prefixcmp(p, "From ")) {
count++; count++;
p += 5; p += 5;
} }
@ -1216,7 +1216,7 @@ split_msg( msg_data_t *all_msgs, msg_data_t *msg, int *ofs )
data = &all_msgs->data[ *ofs ]; data = &all_msgs->data[ *ofs ];
msg->len = all_msgs->len - *ofs; msg->len = all_msgs->len - *ofs;


if (msg->len < 5 || strncmp( data, "From ", 5 )) if (msg->len < 5 || prefixcmp(data, "From "))
return 0; return 0;


p = strchr( data, '\n' ); p = strchr( data, '\n' );
@ -1267,12 +1267,12 @@ git_imap_config(const char *key, const char *val)
imap_folder = xstrdup( val ); imap_folder = xstrdup( val );
} else if (!strcmp( "host", key )) { } else if (!strcmp( "host", key )) {
{ {
if (!strncmp( "imap:", val, 5 )) if (!prefixcmp(val, "imap:"))
val += 5; val += 5;
if (!server.port) if (!server.port)
server.port = 143; server.port = 143;
} }
if (!strncmp( "//", val, 2 )) if (!prefixcmp(val, "//"))
val += 2; val += 2;
server.host = xstrdup( val ); server.host = xstrdup( val );
} }

2
revision.c

@ -832,7 +832,7 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, const ch
revs->max_count = atoi(argv[++i]); revs->max_count = atoi(argv[++i]);
continue; continue;
} }
if (!strncmp(arg,"-n",2)) { if (!prefixcmp(arg, "-n")) {
revs->max_count = atoi(arg + 2); revs->max_count = atoi(arg + 2);
continue; continue;
} }

2
wt-status.c

@ -344,7 +344,7 @@ int git_status_config(const char *k, const char *v)
wt_status_use_color = git_config_colorbool(k, v); wt_status_use_color = git_config_colorbool(k, v);
return 0; return 0;
} }
if (!prefixcmp(k, "status.color.") || !strncmp(k, "color.status.", 13)) { if (!prefixcmp(k, "status.color.") || !prefixcmp(k, "color.status.")) {
int slot = parse_status_slot(k, 13); int slot = parse_status_slot(k, 13);
color_parse(v, k, wt_status_colors[slot]); color_parse(v, k, wt_status_colors[slot]);
} }

Loading…
Cancel
Save