From: smithj4 at bnl.gov (Jason A. Smith)
Subject: [PATCH 2/3] Remove angle brackets from {author,committer}_email
Date: Sat, 25 Feb 2017 11:12:35 -0500 [thread overview]
Message-ID: <20170225161236.30039-3-smithj4@bnl.gov> (raw)
In-Reply-To: <20170225161236.30039-1-smithj4@bnl.gov>
From: John Keeping <john at keeping.me.uk>
This matches the internal representation in libgit.a, so it will make it
much easier to use Git's mailmap code.
The change in ui-atom.c isn't strictly necessary since the code copes
with email addresses both with and without angle brackets, but it's a
nice simplification since we know that the email address will always be
provided in the correct form.
Signed-off-by: John Keeping <john at keeping.me.uk>
Signed-off-by: Jason A. Smith <smithj4 at bnl.gov>
---
filter.c | 10 +++++++++-
parsing.c | 6 +-----
ui-atom.c | 13 +------------
ui-commit.c | 6 ++++--
ui-tag.c | 3 ++-
5 files changed, 17 insertions(+), 21 deletions(-)
diff --git a/filter.c b/filter.c
index 88098ba..ba9000a 100644
--- a/filter.c
+++ b/filter.c
@@ -457,5 +457,13 @@ struct cgit_filter *cgit_new_filter(const char *cmd, filter_type filtertype)
void cgit_open_email_filter(const char *email, const char *origin)
{
- cgit_open_filter(ctx.repo->email_filter, email, origin);
+ struct strbuf sb = STRBUF_INIT;
+
+ /* Don't bother allocating any memory if we don't have a filter. */
+ if (!ctx.repo->email_filter)
+ return;
+
+ strbuf_addf(&sb, "<%s>", email);
+ cgit_open_filter(ctx.repo->email_filter, sb.buf, origin);
+ strbuf_release(&sb);
}
diff --git a/parsing.c b/parsing.c
index 9dacb16..352338d 100644
--- a/parsing.c
+++ b/parsing.c
@@ -72,14 +72,10 @@ static char *substr(const char *head, const char *tail)
static void parse_user(const char *t, char **name, char **email, unsigned long *date, int *tz)
{
struct ident_split ident;
- unsigned email_len;
if (!split_ident_line(&ident, t, strchrnul(t, '\n') - t)) {
*name = substr(ident.name_begin, ident.name_end);
-
- email_len = ident.mail_end - ident.mail_begin;
- *email = xmalloc(strlen("<") + email_len + strlen(">") + 1);
- sprintf(*email, "<%.*s>", email_len, ident.mail_begin);
+ *email = substr(ident.mail_begin, ident.mail_end);
if (ident.date_begin)
*date = strtoul(ident.date_begin, NULL, 10);
diff --git a/ui-atom.c b/ui-atom.c
index 41838d3..7c17d6a 100644
--- a/ui-atom.c
+++ b/ui-atom.c
@@ -15,7 +15,6 @@ static void add_entry(struct commit *commit, const char *host)
{
char delim = '&';
char *hex;
- char *mail, *t, *t2;
struct commitinfo *info;
info = cgit_parse_commit(commit);
@@ -35,19 +34,9 @@ static void add_entry(struct commit *commit, const char *host)
html("</name>\n");
}
if (info->author_email && !ctx.cfg.noplainemail) {
- mail = xstrdup(info->author_email);
- t = strchr(mail, '<');
- if (t)
- t++;
- else
- t = mail;
- t2 = strchr(t, '>');
- if (t2)
- *t2 = '\0';
html("<email>");
- html_txt(t);
+ html_txt(info->author_email);
html("</email>\n");
- free(mail);
}
html("</author>\n");
html("<published>");
diff --git a/ui-commit.c b/ui-commit.c
index 04f0411..dccb5f3 100644
--- a/ui-commit.c
+++ b/ui-commit.c
@@ -50,8 +50,9 @@ void cgit_print_commit(char *hex, const char *prefix)
cgit_open_email_filter(info->author_email, "commit");
html_txt(info->author);
if (!ctx.cfg.noplainemail) {
- html(" ");
+ html(" <");
html_txt(info->author_email);
+ html(">");
}
cgit_close_filter(ctx.repo->email_filter);
html("</td><td class='right'>");
@@ -62,8 +63,9 @@ void cgit_print_commit(char *hex, const char *prefix)
cgit_open_email_filter(info->committer_email, "commit");
html_txt(info->committer);
if (!ctx.cfg.noplainemail) {
- html(" ");
+ html(" <");
html_txt(info->committer_email);
+ html(">");
}
cgit_close_filter(ctx.repo->email_filter);
html("</td><td class='right'>");
diff --git a/ui-tag.c b/ui-tag.c
index d1e5db9..ad7854e 100644
--- a/ui-tag.c
+++ b/ui-tag.c
@@ -86,8 +86,9 @@ void cgit_print_tag(char *revname)
cgit_open_email_filter(info->tagger_email, "tag");
html_txt(info->tagger);
if (info->tagger_email && !ctx.cfg.noplainemail) {
- html(" ");
+ html(" <");
html_txt(info->tagger_email);
+ html(">");
}
cgit_close_filter(ctx.repo->email_filter);
html("</td></tr>\n");
--
2.9.3
next prev parent reply other threads:[~2017-02-25 16:12 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-25 16:12 [PATCH 0/3] Add support for git's mailmap smithj4
2017-02-25 16:12 ` [PATCH 1/3] filter: introduce cgit_open_email_filter() wrapper smithj4
2017-02-25 16:12 ` smithj4 [this message]
2017-02-25 16:12 ` [PATCH 3/3] Add support for git's mailmap smithj4
[not found] <cover.1487952449.git.smithj4@bnl.gov>
2017-02-24 16:19 ` [PATCH 2/3] Remove angle brackets from {author,committer}_email john
[not found] <b710a9bcd7a1408c429ec09f535916074b957d25.1472494937.git.smithj4@bnl.gov>
2016-08-29 18:28 ` john
-- strict thread matches above, loose matches on Subject: below --
2016-08-26 20:12 john
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170225161236.30039-3-smithj4@bnl.gov \
--to=cgit@lists.zx2c4.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).