git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCHv4 1/4] Preparation to call determine_author_info from prepare_to_commit

From
SBSanti Béjar <sbejar@gmail.com>
Date
May 4, 2008, 16:12 UTC
Message-ID
<8aa486160805040912h38d81750p83353d21e8c2da6d@mail.gmail.com>
In-Reply-To
<1209917092-12146-2-git-send-email-sbejar@gmail.com>
On Sun, May 4, 2008 at 6:04 PM, Santi Béjar <sbejar@gmail.com> wrote:
> From: Santi Béjar <sbjear@gmail.com>
Gggrrr!! I thought I had fixed this. Please remove this line.
Santi
Show 109 quoted lines
>
>  Reorder functions definitions such that determine_author_info is
>  defined before prepare_to_commit. No code changes.
>
>  Signed-off-by: Santi Béjar <sbejar@gmail.com>
>  ---
>   builtin-commit.c |   78 +++++++++++++++++++++++++++---------------------------
>   1 files changed, 39 insertions(+), 39 deletions(-)
>
>  diff --git a/builtin-commit.c b/builtin-commit.c
>  index 256181a..a37d8c3 100644
>  --- a/builtin-commit.c
>  +++ b/builtin-commit.c
>  @@ -395,6 +395,45 @@ static int is_a_merge(const unsigned char *sha1)
>
>   static const char sign_off_header[] = "Signed-off-by: ";
>
>  +static void determine_author_info(struct strbuf *sb)
>  +{
>  +       char *name, *email, *date;
>  +
>  +       name = getenv("GIT_AUTHOR_NAME");
>  +       email = getenv("GIT_AUTHOR_EMAIL");
>  +       date = getenv("GIT_AUTHOR_DATE");
>  +
>  +       if (use_message) {
>  +               const char *a, *lb, *rb, *eol;
>  +
>  +               a = strstr(use_message_buffer, "\nauthor ");
>  +               if (!a)
>  +                       die("invalid commit: %s", use_message);
>  +
>  +               lb = strstr(a + 8, " <");
>  +               rb = strstr(a + 8, "> ");
>  +               eol = strchr(a + 8, '\n');
>  +               if (!lb || !rb || !eol)
>  +                       die("invalid commit: %s", use_message);
>  +
>  +               name = xstrndup(a + 8, lb - (a + 8));
>  +               email = xstrndup(lb + 2, rb - (lb + 2));
>  +               date = xstrndup(rb + 2, eol - (rb + 2));
>  +       }
>  +
>  +       if (force_author) {
>  +               const char *lb = strstr(force_author, " <");
>  +               const char *rb = strchr(force_author, '>');
>  +
>  +               if (!lb || !rb)
>  +                       die("malformed --author parameter");
>  +               name = xstrndup(force_author, lb - force_author);
>  +               email = xstrndup(lb + 2, rb - (lb + 2));
>  +       }
>  +
>  +       strbuf_addf(sb, "author %s\n", fmt_ident(name, email, date, IDENT_ERROR_ON_NO_NAME));
>  +}
>  +
>   static int prepare_to_commit(const char *index_file, const char *prefix)
>   {
>         struct stat statbuf;
>  @@ -622,45 +661,6 @@ static int message_is_empty(struct strbuf *sb, int start)
>         return 1;
>   }
>
>  -static void determine_author_info(struct strbuf *sb)
>  -{
>  -       char *name, *email, *date;
>  -
>  -       name = getenv("GIT_AUTHOR_NAME");
>  -       email = getenv("GIT_AUTHOR_EMAIL");
>  -       date = getenv("GIT_AUTHOR_DATE");
>  -
>  -       if (use_message) {
>  -               const char *a, *lb, *rb, *eol;
>  -
>  -               a = strstr(use_message_buffer, "\nauthor ");
>  -               if (!a)
>  -                       die("invalid commit: %s", use_message);
>  -
>  -               lb = strstr(a + 8, " <");
>  -               rb = strstr(a + 8, "> ");
>  -               eol = strchr(a + 8, '\n');
>  -               if (!lb || !rb || !eol)
>  -                       die("invalid commit: %s", use_message);
>  -
>  -               name = xstrndup(a + 8, lb - (a + 8));
>  -               email = xstrndup(lb + 2, rb - (lb + 2));
>  -               date = xstrndup(rb + 2, eol - (rb + 2));
>  -       }
>  -
>  -       if (force_author) {
>  -               const char *lb = strstr(force_author, " <");
>  -               const char *rb = strchr(force_author, '>');
>  -
>  -               if (!lb || !rb)
>  -                       die("malformed --author parameter");
>  -               name = xstrndup(force_author, lb - force_author);
>  -               email = xstrndup(lb + 2, rb - (lb + 2));
>  -       }
>  -
>  -       strbuf_addf(sb, "author %s\n", fmt_ident(name, email, date, IDENT_ERROR_ON_NO_NAME));
>  -}
>  -
>   static int parse_and_validate_options(int argc, const char *argv[],
>                                       const char * const usage[])
>   {
>  --
>  1.5.5.1.224.gadb29
>
>
Previous: Santi BéjarNext: Santi Béjar
Message 3 of 10 in “[PATCHv4 0/4] Show author and/or committer in some cases”
  1. Santi BéjarMay 4, 2008
  2. 1/4 Preparation to call determine_author_info from prepare_to_commitSanti Béjar, May 4, 2008
  3. Santi BéjarMay 4, 2008
  4. 2/4 commit: Show author if different from committerSanti Béjar, May 4, 2008
  5. Junio C HamanoMay 6, 2008
  6. 3/4 commit: Show committer if automaticSanti Béjar, May 4, 2008
  7. 4/4 user.warnautomatic: add config to control if the committer ident is shownSanti Béjar, May 4, 2008
  8. Junio C HamanoMay 7, 2008
  9. Santi BéjarMay 7, 2008
  10. Santi BéjarMay 4, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.