is_refname_available(): avoid shadowing "dir" variable

The function had a "dir" parameter that was shadowed by a local "dir"
variable within a code block. Use the former in place of the latter.
(This is consistent with "dir"'s use elsewhere in the function.)

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

diff --git a/refs.c b/refs.c
index 776bbcebd6998b73e7a5be4fcdda976e1d6b962c..9d87e84f4227c7640cbf733ebe1c8d26f9d502ee 100644 (file)
--- a/refs.c
+++ b/refs.c
@@ -967,10 +967,10 @@ static int is_refname_available(const char *refname,
                 * "refs/foo/bar/"). It is a problem iff it contains
                 * any ref that is not in "skip".
                 */
-               struct ref_entry *entry = dir->entries[pos];
-               struct ref_dir *dir = get_ref_dir(entry);
                struct nonmatching_ref_data data;
+               struct ref_entry *entry = dir->entries[pos];
 
+               dir = get_ref_dir(entry);
                data.skip = skip;
                sort_ref_dir(dir);
                if (!do_for_each_entry_in_dir(dir, 0, nonmatching_ref_fn, &data))