OSDN Git Service

report_refname_conflict(): inline function
authorMichael Haggerty <mhagger@alum.mit.edu>
Mon, 11 May 2015 15:25:08 +0000 (17:25 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 11 May 2015 18:50:18 +0000 (11:50 -0700)
It wasn't pulling its weight. And we are about to need code similar to
this where no ref_entry is available and with more diverse error
messages. Rather than try to generalize the function, just inline it.

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
refs.c

diff --git a/refs.c b/refs.c
index 6bdd34f..7422594 100644 (file)
--- a/refs.c
+++ b/refs.c
@@ -857,12 +857,6 @@ static int nonmatching_ref_fn(struct ref_entry *entry, void *vdata)
        return 1;
 }
 
-static void report_refname_conflict(struct ref_entry *entry,
-                                   const char *refname)
-{
-       error("'%s' exists; cannot create '%s'", entry->name, refname);
-}
-
 /*
  * Return true iff a reference named refname could be created without
  * conflicting with the name of an existing reference in dir.  If
@@ -918,7 +912,7 @@ static int is_refname_available(const char *refname,
                                 */
                                return 1;
                        }
-                       report_refname_conflict(entry, refname);
+                       error("'%s' exists; cannot create '%s'", entry->name, refname);
                        return 0;
                }
 
@@ -969,7 +963,7 @@ static int is_refname_available(const char *refname,
                if (!do_for_each_entry_in_dir(dir, 0, nonmatching_ref_fn, &data))
                        return 1;
 
-               report_refname_conflict(data.found, refname);
+               error("'%s' exists; cannot create '%s'", data.found->name, refname);
                return 0;
        }