Simplified cases where a strbuf_reset was immediately followed by a
strbuf_add using the new strbuf_set operations.

Signed-off-by: Jeremiah Mahler <jmmah...@gmail.com>
---
 fast-import.c | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/fast-import.c b/fast-import.c
index e8ec34d..c23935c 100644
--- a/fast-import.c
+++ b/fast-import.c
@@ -2741,8 +2741,7 @@ static void parse_new_commit(void)
        hashcpy(b->branch_tree.versions[0].sha1,
                b->branch_tree.versions[1].sha1);
 
-       strbuf_reset(&new_data);
-       strbuf_addf(&new_data, "tree %s\n",
+       strbuf_setf(&new_data, "tree %s\n",
                sha1_to_hex(b->branch_tree.versions[1].sha1));
        if (!is_null_sha1(b->sha1))
                strbuf_addf(&new_data, "parent %s\n", sha1_to_hex(b->sha1));
@@ -2829,9 +2828,7 @@ static void parse_new_tag(void)
        parse_data(&msg, 0, NULL);
 
        /* build the tag object */
-       strbuf_reset(&new_data);
-
-       strbuf_addf(&new_data,
+       strbuf_setf(&new_data,
                    "object %s\n"
                    "type %s\n"
                    "tag %s\n",
@@ -2898,8 +2895,7 @@ static void cat_blob(struct object_entry *oe, unsigned 
char sha1[20])
         * Output based on batch_one_object() from cat-file.c.
         */
        if (type <= 0) {
-               strbuf_reset(&line);
-               strbuf_addf(&line, "%s missing\n", sha1_to_hex(sha1));
+               strbuf_setf(&line, "%s missing\n", sha1_to_hex(sha1));
                cat_blob_write(line.buf, line.len);
                strbuf_release(&line);
                free(buf);
@@ -2910,8 +2906,7 @@ static void cat_blob(struct object_entry *oe, unsigned 
char sha1[20])
        if (type != OBJ_BLOB)
                die("Object %s is a %s but a blob was expected.",
                    sha1_to_hex(sha1), typename(type));
-       strbuf_reset(&line);
-       strbuf_addf(&line, "%s %s %lu\n", sha1_to_hex(sha1),
+       strbuf_setf(&line, "%s %s %lu\n", sha1_to_hex(sha1),
                                                typename(type), size);
        cat_blob_write(line.buf, line.len);
        strbuf_release(&line);
@@ -3034,14 +3029,12 @@ static void print_ls(int mode, const unsigned char 
*sha1, const char *path)
 
        if (!mode) {
                /* missing SP path LF */
-               strbuf_reset(&line);
-               strbuf_addstr(&line, "missing ");
+               strbuf_setstr(&line, "missing ");
                quote_c_style(path, &line, NULL, 0);
                strbuf_addch(&line, '\n');
        } else {
                /* mode SP type SP object_name TAB path LF */
-               strbuf_reset(&line);
-               strbuf_addf(&line, "%06o %s %s\t",
+               strbuf_setf(&line, "%06o %s %s\t",
                                mode & ~NO_DELTA, type, sha1_to_hex(sha1));
                quote_c_style(path, &line, NULL, 0);
                strbuf_addch(&line, '\n');
-- 
2.0.0.573.ged771ce.dirty

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to