Skip to content

Commit

Permalink
write_name{_quoted_relative,}(): remove redundant parameters
Browse files Browse the repository at this point in the history
After substitute path_relative() in quote.c with relative_path()
from path.c, parameters (such as len and prefix_len) are redundant
in function write_name() and write_name_quoted_relative().  The
callers have already been audited that the strings they pass are
properly NUL terminated and the length they give are the length of
the string (or -1 that asks the length to be counted by the callee).

Remove these now-redundant parameters.

Signed-off-by: Jiang Xin <worldhello.net@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Jiang Xin authored and Junio C Hamano committed Jun 26, 2013
1 parent 39598f9 commit e9a820c
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 13 deletions.
17 changes: 8 additions & 9 deletions builtin/ls-files.c
Original file line number Diff line number Diff line change
Expand Up @@ -46,15 +46,14 @@ static const char *tag_modified = "";
static const char *tag_skip_worktree = "";
static const char *tag_resolve_undo = "";

static void write_name(const char* name, size_t len)
static void write_name(const char *name)
{
/*
* With "--full-name", prefix_len=0; write_name_quoted_relative()
* ignores prefix_len, so this caller needs to pass empty string
* in that case (a NULL is good for "").
* With "--full-name", prefix_len=0; this caller needs to pass
* an empty string in that case (a NULL is good for "").
*/
write_name_quoted_relative(name, len, prefix_len ? prefix : NULL,
prefix_len, stdout, line_terminator);
write_name_quoted_relative(name, prefix_len ? prefix : NULL,
stdout, line_terminator);
}

static void show_dir_entry(const char *tag, struct dir_entry *ent)
Expand All @@ -68,7 +67,7 @@ static void show_dir_entry(const char *tag, struct dir_entry *ent)
return;

fputs(tag, stdout);
write_name(ent->name, ent->len);
write_name(ent->name);
}

static void show_other_files(struct dir_struct *dir)
Expand Down Expand Up @@ -168,7 +167,7 @@ static void show_ce_entry(const char *tag, struct cache_entry *ce)
find_unique_abbrev(ce->sha1,abbrev),
ce_stage(ce));
}
write_name(ce->name, ce_namelen(ce));
write_name(ce->name);
if (debug_mode) {
printf(" ctime: %d:%d\n", ce->ce_ctime.sec, ce->ce_ctime.nsec);
printf(" mtime: %d:%d\n", ce->ce_mtime.sec, ce->ce_mtime.nsec);
Expand Down Expand Up @@ -201,7 +200,7 @@ static void show_ru_info(void)
printf("%s%06o %s %d\t", tag_resolve_undo, ui->mode[i],
find_unique_abbrev(ui->sha1[i], abbrev),
i + 1);
write_name(path, len);
write_name(path);
}
}
}
Expand Down
3 changes: 1 addition & 2 deletions quote.c
Original file line number Diff line number Diff line change
Expand Up @@ -312,8 +312,7 @@ void write_name_quotedpfx(const char *pfx, size_t pfxlen,
fputc(terminator, fp);
}

void write_name_quoted_relative(const char *name, size_t len,
const char *prefix, size_t prefix_len,
void write_name_quoted_relative(const char *name, const char *prefix,
FILE *fp, int terminator)
{
struct strbuf sb = STRBUF_INIT;
Expand Down
3 changes: 1 addition & 2 deletions quote.h
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,7 @@ extern void quote_two_c_style(struct strbuf *, const char *, const char *, int);
extern void write_name_quoted(const char *name, FILE *, int terminator);
extern void write_name_quotedpfx(const char *pfx, size_t pfxlen,
const char *name, FILE *, int terminator);
extern void write_name_quoted_relative(const char *name, size_t len,
const char *prefix, size_t prefix_len,
extern void write_name_quoted_relative(const char *name, const char *prefix,
FILE *fp, int terminator);

/* quote path as relative to the given prefix */
Expand Down

0 comments on commit e9a820c

Please sign in to comment.