Merge branch 'rs/use-strbuf-complete-line'
authorJunio C Hamano <gitster@pobox.com>
Mon, 22 Dec 2014 20:28:22 +0000 (12:28 -0800)
committerJunio C Hamano <gitster@pobox.com>
Mon, 22 Dec 2014 20:28:22 +0000 (12:28 -0800)
* rs/use-strbuf-complete-line:
use strbuf_complete_line() for adding a newline if needed

1  2 
builtin/fmt-merge-msg.c
trace.c
diff --combined builtin/fmt-merge-msg.c
index 37177c6c2928b926cf55cacfdcfcf033bf012e80,b5c1ac2bcbdc132be1f54f1e7206fba313d8146b..af7919e51e717923260080f8be3305bbe1b56d39
@@@ -216,8 -216,7 +216,7 @@@ static void add_branch_desc(struct strb
                        strbuf_addf(out, "  : %.*s", (int)(ep - bp), bp);
                        bp = ep;
                }
-               if (out->buf[out->len - 1] != '\n')
-                       strbuf_addch(out, '\n');
+               strbuf_complete_line(out);
        }
        strbuf_release(&desc);
  }
@@@ -602,7 -601,7 +601,7 @@@ int fmt_merge_msg(struct strbuf *in, st
  
        /* get current branch */
        current_branch = current_branch_to_free =
 -              resolve_refdup("HEAD", head_sha1, 1, NULL);
 +              resolve_refdup("HEAD", RESOLVE_REF_READING, head_sha1, NULL);
        if (!current_branch)
                die("No current branch");
        if (starts_with(current_branch, "refs/heads/"))
diff --combined trace.c
index 477860894196e32d7db990459cef47175277c441,7149651601fc5975bb1f4414895a8058a8edf662..f6f9f3a367242f5f3f10e1d33381f0a52d4fbe55
+++ b/trace.c
@@@ -122,9 -122,7 +122,7 @@@ static int prepare_trace_line(const cha
  
  static void print_trace_line(struct trace_key *key, struct strbuf *buf)
  {
-       /* append newline if missing */
-       if (buf->len && buf->buf[buf->len - 1] != '\n')
-               strbuf_addch(buf, '\n');
+       strbuf_complete_line(buf);
  
        write_or_whine_pipe(get_trace_fd(key), buf->buf, buf->len, err_msg);
        strbuf_release(buf);
@@@ -216,7 -214,7 +214,7 @@@ void trace_argv_printf(const char **arg
        va_end(ap);
  }
  
 -void trace_strbuf(const char *key, const struct strbuf *data)
 +void trace_strbuf(struct trace_key *key, const struct strbuf *data)
  {
        trace_strbuf_fl(NULL, 0, key, data);
  }
@@@ -298,12 -296,13 +296,12 @@@ void trace_repo_setup(const char *prefi
  {
        static struct trace_key key = TRACE_KEY_INIT(SETUP);
        const char *git_work_tree;
 -      char cwd[PATH_MAX];
 +      char *cwd;
  
        if (!trace_want(&key))
                return;
  
 -      if (!getcwd(cwd, PATH_MAX))
 -              die("Unable to get current working directory");
 +      cwd = xgetcwd();
  
        if (!(git_work_tree = get_git_work_tree()))
                git_work_tree = "(null)";
        trace_printf_key(&key, "setup: worktree: %s\n", quote_crnl(git_work_tree));
        trace_printf_key(&key, "setup: cwd: %s\n", quote_crnl(cwd));
        trace_printf_key(&key, "setup: prefix: %s\n", quote_crnl(prefix));
 +
 +      free(cwd);
  }
  
  int trace_want(struct trace_key *key)
@@@ -385,7 -382,7 +383,7 @@@ static inline uint64_t gettimeofday_nan
   * Returns nanoseconds since the epoch (01/01/1970), for performance tracing
   * (i.e. favoring high precision over wall clock time accuracy).
   */
 -inline uint64_t getnanotime(void)
 +uint64_t getnanotime(void)
  {
        static uint64_t offset;
        if (offset > 1) {