summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2012-05-21 19:09:43 -0400
committerJunio C Hamano <gitster@pobox.com>2012-05-22 09:07:52 -0700
commitbcb2b0044b3baedf7857613ec069f300c364680c (patch)
tree61dac600dabd49b867ae598f4243a9c5ba12b041
parentcd07cc53125fb2ca77a644831d4e5d9517bb7a05 (diff)
downloadgit-bcb2b0044b3baedf7857613ec069f300c364680c.tar.gz
ident: split setup_ident into separate functions
This function sets up the default name, email, and date, and is not publicly available. Let's split it into three public functions so that callers can get just the parts they need. While we're at it, let's change the interface to simple accessors. The original function was called only by fmt_ident, and contained logic for "if we already have some other value, don't load the default" which properly belongs in fmt_ident. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--cache.h3
-rw-r--r--ident.c35
2 files changed, 22 insertions, 16 deletions
diff --git a/cache.h b/cache.h
index e14ffcd914..0c095d4842 100644
--- a/cache.h
+++ b/cache.h
@@ -894,6 +894,9 @@ extern const char *git_author_info(int);
extern const char *git_committer_info(int);
extern const char *fmt_ident(const char *name, const char *email, const char *date_str, int);
extern const char *fmt_name(const char *name, const char *email);
+extern const char *ident_default_name(void);
+extern const char *ident_default_email(void);
+extern const char *ident_default_date(void);
extern const char *git_editor(void);
extern const char *git_pager(int stdout_is_tty);
diff --git a/ident.c b/ident.c
index 87c697c2b0..0f7dcae8f8 100644
--- a/ident.c
+++ b/ident.c
@@ -117,21 +117,20 @@ static void copy_email(const struct passwd *pw)
sizeof(git_default_email) - len);
}
-static void setup_ident(const char **name, const char **emailp)
+const char *ident_default_name(void)
{
- struct passwd *pw = NULL;
-
- /* Get the name ("gecos") */
- if (!*name && !git_default_name[0]) {
- pw = getpwuid(getuid());
+ if (!git_default_name[0]) {
+ struct passwd *pw = getpwuid(getuid());
if (!pw)
die("You don't exist. Go away!");
copy_gecos(pw, git_default_name, sizeof(git_default_name));
}
- if (!*name)
- *name = git_default_name;
+ return git_default_name;
+}
- if (!*emailp && !git_default_email[0]) {
+const char *ident_default_email(void)
+{
+ if (!git_default_email[0]) {
const char *email = getenv("EMAIL");
if (email && email[0]) {
@@ -139,19 +138,20 @@ static void setup_ident(const char **name, const char **emailp)
sizeof(git_default_email));
user_ident_explicitly_given |= IDENT_MAIL_GIVEN;
} else {
- if (!pw)
- pw = getpwuid(getuid());
+ struct passwd *pw = getpwuid(getuid());
if (!pw)
die("You don't exist. Go away!");
copy_email(pw);
}
}
- if (!*emailp)
- *emailp = git_default_email;
+ return git_default_email;
+}
- /* And set the default date */
+const char *ident_default_date(void)
+{
if (!git_default_date[0])
datestamp(git_default_date, sizeof(git_default_date));
+ return git_default_date;
}
static int add_raw(char *buf, size_t size, int offset, const char *str)
@@ -311,7 +311,10 @@ const char *fmt_ident(const char *name, const char *email,
int warn_on_no_name = (flag & IDENT_WARN_ON_NO_NAME);
int name_addr_only = (flag & IDENT_NO_DATE);
- setup_ident(&name, &email);
+ if (!name)
+ name = ident_default_name();
+ if (!email)
+ email = ident_default_email();
if (!*name) {
struct passwd *pw;
@@ -331,7 +334,7 @@ const char *fmt_ident(const char *name, const char *email,
name = git_default_name;
}
- strcpy(date, git_default_date);
+ strcpy(date, ident_default_date());
if (!name_addr_only && date_str && date_str[0]) {
if (parse_date(date_str, date, sizeof(date)) < 0)
die("invalid date format: %s", date_str);