Currently, creation and addition (to a list) of trailer items are spread
across multiple functions. Streamline this by only having 2 functions:
one to parse the user-supplied string, and one to add the parsed
information to a list.
---
 trailer.c | 135 +++++++++++++++++++++++++++++---------------------------------
 1 file changed, 62 insertions(+), 73 deletions(-)

diff --git a/trailer.c b/trailer.c
index bf3d7d0..e8b1bfb 100644
--- a/trailer.c
+++ b/trailer.c
@@ -335,7 +335,7 @@ static int set_if_missing(struct conf_info *item, const 
char *value)
        return 0;
 }
 
-static void duplicate_conf(struct conf_info *dst, struct conf_info *src)
+static void duplicate_conf(struct conf_info *dst, const struct conf_info *src)
 {
        *dst = *src;
        if (src->name)
@@ -467,10 +467,30 @@ static int git_trailer_config(const char *conf_key, const 
char *value, void *cb)
        return 0;
 }
 
-static int parse_trailer(struct strbuf *tok, struct strbuf *val, const char 
*trailer)
+static const char *token_from_item(struct trailer_item *item, char *tok)
+{
+       if (item->conf.key)
+               return item->conf.key;
+       if (tok)
+               return tok;
+       return item->conf.name;
+}
+
+static int token_matches_item(const char *tok, struct trailer_item *item, int 
tok_len)
+{
+       if (!strncasecmp(tok, item->conf.name, tok_len))
+               return 1;
+       return item->conf.key ? !strncasecmp(tok, item->conf.key, tok_len) : 0;
+}
+
+static int parse_trailer(struct strbuf *tok, struct strbuf *val,
+                        const struct conf_info **conf, const char *trailer)
 {
        size_t len;
        struct strbuf seps = STRBUF_INIT;
+       struct trailer_item *item;
+       int tok_len;
+
        strbuf_addstr(&seps, separators);
        strbuf_addch(&seps, '=');
        len = strcspn(trailer, seps.buf);
@@ -490,73 +510,31 @@ static int parse_trailer(struct strbuf *tok, struct 
strbuf *val, const char *tra
                strbuf_addstr(tok, trailer);
                strbuf_trim(tok);
        }
-       return 0;
-}
-
-static const char *token_from_item(struct trailer_item *item, char *tok)
-{
-       if (item->conf.key)
-               return item->conf.key;
-       if (tok)
-               return tok;
-       return item->conf.name;
-}
-
-static struct trailer_item *new_trailer_item(struct trailer_item *conf_item,
-                                            char *tok, char *val)
-{
-       struct trailer_item *new = xcalloc(sizeof(*new), 1);
-       new->value = val ? val : xstrdup("");
-
-       if (conf_item) {
-               duplicate_conf(&new->conf, &conf_item->conf);
-               new->token = xstrdup(token_from_item(conf_item, tok));
-               free(tok);
-       } else {
-               duplicate_conf(&new->conf, &default_conf_info);
-               new->token = tok;
-       }
-
-       return new;
-}
-
-static int token_matches_item(const char *tok, struct trailer_item *item, int 
tok_len)
-{
-       if (!strncasecmp(tok, item->conf.name, tok_len))
-               return 1;
-       return item->conf.key ? !strncasecmp(tok, item->conf.key, tok_len) : 0;
-}
-
-static struct trailer_item *create_trailer_item(const char *string)
-{
-       struct strbuf tok = STRBUF_INIT;
-       struct strbuf val = STRBUF_INIT;
-       struct trailer_item *item;
-       int tok_len;
-
-       if (parse_trailer(&tok, &val, string))
-               return NULL;
-
-       tok_len = token_len_without_separator(tok.buf, tok.len);
 
        /* Lookup if the token matches something in the config */
+       tok_len = token_len_without_separator(tok->buf, tok->len);
+       *conf = &default_conf_info;
        for (item = first_conf_item; item; item = item->next) {
-               if (token_matches_item(tok.buf, item, tok_len))
-                       return new_trailer_item(item,
-                                               strbuf_detach(&tok, NULL),
-                                               strbuf_detach(&val, NULL));
+               if (token_matches_item(tok->buf, item, tok_len)) {
+                       char *tok_buf = strbuf_detach(tok, NULL);
+                       *conf = &item->conf;
+                       strbuf_addstr(tok, token_from_item(item, tok_buf));
+                       free(tok_buf);
+                       break;
+               }
        }
 
-       return new_trailer_item(NULL,
-                               strbuf_detach(&tok, NULL),
-                               strbuf_detach(&val, NULL));
+       return 0;
 }
 
-static void add_trailer_item(struct trailer_item ***tail,
-                            struct trailer_item *new)
+static void add_trailer_item(struct trailer_item ***tail, char *tok, char *val,
+                            const struct conf_info *conf)
 {
-       if (!new)
-               return;
+       struct trailer_item *new = xcalloc(sizeof(*new), 1);
+       new->token = tok;
+       new->value = val;
+       duplicate_conf(&new->conf, conf);
+
        **tail = new;
        *tail = &new->next;
 }
@@ -567,19 +545,25 @@ static struct trailer_item 
*process_command_line_args(struct string_list *traile
        struct trailer_item **arg_tok_tail = &arg_tok_head;
        struct string_list_item *tr;
        struct trailer_item *item;
+       struct strbuf tok = STRBUF_INIT;
+       struct strbuf val = STRBUF_INIT;
+       const struct conf_info *conf;
 
        /* Add a trailer item for each configured trailer with a command */
-       for (item = first_conf_item; item; item = item->next) {
-               if (item->conf.command) {
-                       struct trailer_item *new = new_trailer_item(item, NULL, 
NULL);
-                       add_trailer_item(&arg_tok_tail, new);
-               }
-       }
+       for (item = first_conf_item; item; item = item->next)
+               if (item->conf.command)
+                       add_trailer_item(&arg_tok_tail,
+                                        xstrdup(token_from_item(item, NULL)),
+                                        xstrdup(""),
+                                        &item->conf);
 
        /* Add a trailer item for each trailer on the command line */
        for_each_string_list_item(tr, trailers) {
-               struct trailer_item *new = create_trailer_item(tr->string);
-               add_trailer_item(&arg_tok_tail, new);
+               if (!parse_trailer(&tok, &val, &conf, tr->string))
+                       add_trailer_item(&arg_tok_tail,
+                                        strbuf_detach(&tok, NULL),
+                                        strbuf_detach(&val, NULL),
+                                        conf);
        }
 
        return arg_tok_head;
@@ -702,6 +686,9 @@ static int process_input_file(FILE *outfile,
 {
        int count = 0;
        int patch_start, trailer_start, trailer_end, i;
+       struct strbuf tok = STRBUF_INIT;
+       struct strbuf val = STRBUF_INIT;
+       const struct conf_info *conf;
 
        /* Get the line count */
        while (lines[count])
@@ -719,10 +706,12 @@ static int process_input_file(FILE *outfile,
 
        /* Parse trailer lines */
        for (i = trailer_start; i < trailer_end; i++) {
-               if (lines[i]->buf[0] != comment_line_char) {
-                       struct trailer_item *new = 
create_trailer_item(lines[i]->buf);
-                       add_trailer_item(in_tok_tail, new);
-               }
+               if (lines[i]->buf[0] != comment_line_char &&
+                   !parse_trailer(&tok, &val, &conf, lines[i]->buf))
+                       add_trailer_item(in_tok_tail,
+                                        strbuf_detach(&tok, NULL),
+                                        strbuf_detach(&val, NULL),
+                                        conf);
        }
 
        return trailer_end;
-- 
2.8.0.rc3.226.g39d4020

Reply via email to