Merge branch 'tf/commit-list-prefix'
authorJunio C Hamano <gitster@pobox.com>
Wed, 22 Dec 2010 22:40:17 +0000 (14:40 -0800)
committerJunio C Hamano <gitster@pobox.com>
Wed, 22 Dec 2010 22:40:17 +0000 (14:40 -0800)
* tf/commit-list-prefix:
commit: Add commit_list prefix in two function names.

Conflicts:
sha1_name.c

1  2 
builtin/describe.c
commit.c
commit.h
revision.c
sha1_name.c
diff --combined builtin/describe.c
index a0f52c1b72b2ffd95e781abffb78d7b66ec583c7,9304dd0ccf3cf10e7a1756ea491520164f590491..342129fdbdc534bdf9277de710f8e7b7ba11c5c4
@@@ -6,7 -6,6 +6,7 @@@
  #include "exec_cmd.h"
  #include "parse-options.h"
  #include "diff.h"
 +#include "hash.h"
  
  #define SEEN          (1u<<0)
  #define MAX_TAGS      (FLAG_BITS - 1)
@@@ -23,8 -22,7 +23,8 @@@ static int tags;      /* Allow lightweight t
  static int longformat;
  static int abbrev = DEFAULT_ABBREV;
  static int max_candidates = 10;
 -static int found_names;
 +static struct hash_table names;
 +static int have_util;
  static const char *pattern;
  static int always;
  static const char *dirty;
@@@ -36,44 -34,16 +36,44 @@@ static const char *diff_index_args[] = 
  
  
  struct commit_name {
 +      struct commit_name *next;
 +      unsigned char peeled[20];
        struct tag *tag;
        unsigned prio:2; /* annotated tag = 2, tag = 1, head = 0 */
        unsigned name_checked:1;
        unsigned char sha1[20];
 -      char path[FLEX_ARRAY]; /* more */
 +      const char *path;
  };
  static const char *prio_names[] = {
        "head", "lightweight", "annotated",
  };
  
 +static inline unsigned int hash_sha1(const unsigned char *sha1)
 +{
 +      unsigned int hash;
 +      memcpy(&hash, sha1, sizeof(hash));
 +      return hash;
 +}
 +
 +static inline struct commit_name *find_commit_name(const unsigned char *peeled)
 +{
 +      struct commit_name *n = lookup_hash(hash_sha1(peeled), &names);
 +      while (n && !!hashcmp(peeled, n->peeled))
 +              n = n->next;
 +      return n;
 +}
 +
 +static int set_util(void *chain)
 +{
 +      struct commit_name *n;
 +      for (n = chain; n; n = n->next) {
 +              struct commit *c = lookup_commit_reference_gently(n->peeled, 1);
 +              if (c)
 +                      c->util = n;
 +      }
 +      return 0;
 +}
 +
  static int replace_name(struct commit_name *e,
                               int prio,
                               const unsigned char *sha1,
  }
  
  static void add_to_known_names(const char *path,
 -                             struct commit *commit,
 +                             const unsigned char *peeled,
                               int prio,
                               const unsigned char *sha1)
  {
 -      struct commit_name *e = commit->util;
 +      struct commit_name *e = find_commit_name(peeled);
        struct tag *tag = NULL;
        if (replace_name(e, prio, sha1, &tag)) {
 -              size_t len = strlen(path)+1;
 -              free(e);
 -              e = xmalloc(sizeof(struct commit_name) + len);
 +              if (!e) {
 +                      void **pos;
 +                      e = xmalloc(sizeof(struct commit_name));
 +                      hashcpy(e->peeled, peeled);
 +                      pos = insert_hash(hash_sha1(peeled), e, &names);
 +                      if (pos) {
 +                              e->next = *pos;
 +                              *pos = e;
 +                      } else {
 +                              e->next = NULL;
 +                      }
 +              }
                e->tag = tag;
                e->prio = prio;
                e->name_checked = 0;
                hashcpy(e->sha1, sha1);
 -              memcpy(e->path, path, len);
 -              commit->util = e;
 +              e->path = path;
        }
 -      found_names = 1;
  }
  
  static int get_name(const char *path, const unsigned char *sha1, int flag, void *cb_data)
  {
        int might_be_tag = !prefixcmp(path, "refs/tags/");
 -      struct commit *commit;
 -      struct object *object;
        unsigned char peeled[20];
        int is_tag, prio;
  
                return 0;
  
        if (!peel_ref(path, peeled) && !is_null_sha1(peeled)) {
 -              commit = lookup_commit_reference_gently(peeled, 1);
 -              if (!commit)
 -                      return 0;
 -              is_tag = !!hashcmp(sha1, commit->object.sha1);
 +              is_tag = !!hashcmp(sha1, peeled);
        } else {
 -              commit = lookup_commit_reference_gently(sha1, 1);
 -              object = parse_object(sha1);
 -              if (!commit || !object)
 -                      return 0;
 -              is_tag = object->type == OBJ_TAG;
 +              hashcpy(peeled, sha1);
 +              is_tag = 0;
        }
  
        /* If --all, then any refs are used.
                if (!prio)
                        return 0;
        }
 -      add_to_known_names(all ? path + 5 : path + 10, commit, prio, sha1);
 +      add_to_known_names(all ? path + 5 : path + 10, peeled, prio, sha1);
        return 0;
  }
  
@@@ -218,7 -189,7 +218,7 @@@ static unsigned long finish_depth_compu
                        struct commit *p = parents->item;
                        parse_commit(p);
                        if (!(p->object.flags & SEEN))
-                               insert_by_date(p, list);
+                               commit_list_insert_by_date(p, list);
                        p->object.flags |= c->object.flags;
                        parents = parents->next;
                }
@@@ -269,7 -240,7 +269,7 @@@ static void describe(const char *arg, i
        if (!cmit)
                die("%s is not a valid '%s' object", arg, commit_type);
  
 -      n = cmit->util;
 +      n = find_commit_name(cmit->object.sha1);
        if (n && (tags || all || n->prio == 2)) {
                /*
                 * Exact match to an existing ref.
        if (debug)
                fprintf(stderr, "searching to describe %s\n", arg);
  
 +      if (!have_util) {
 +              for_each_hash(&names, set_util);
 +              have_util = 1;
 +      }
 +
        list = NULL;
        cmit->object.flags = SEEN;
        commit_list_insert(cmit, &list);
                        struct commit *p = parents->item;
                        parse_commit(p);
                        if (!(p->object.flags & SEEN))
-                               insert_by_date(p, &list);
+                               commit_list_insert_by_date(p, &list);
                        p->object.flags |= c->object.flags;
                        parents = parents->next;
                }
        qsort(all_matches, match_cnt, sizeof(all_matches[0]), compare_pt);
  
        if (gave_up_on) {
-               insert_by_date(gave_up_on, &list);
+               commit_list_insert_by_date(gave_up_on, &list);
                seen_commits--;
        }
        seen_commits += finish_depth_computation(&list, &all_matches[0]);
@@@ -452,9 -418,8 +452,9 @@@ int cmd_describe(int argc, const char *
                return cmd_name_rev(i + argc, args, prefix);
        }
  
 -      for_each_ref(get_name, NULL);
 -      if (!found_names && !always)
 +      init_hash(&names);
 +      for_each_rawref(get_name, NULL);
 +      if (!names.nr && !always)
                die("No names found, cannot describe anything.");
  
        if (argc == 0) {
diff --combined commit.c
index b21335ee4c775d82901fb6b7dc785d63244f2bdd,554dcc3c7dccaba5fb1a671fdcab20ff16b59b1b..74d66018800d7a2a2b3a3365e87a23049690da6b
+++ b/commit.c
@@@ -49,19 -49,6 +49,19 @@@ struct commit *lookup_commit(const unsi
        return check_commit(obj, sha1, 0);
  }
  
 +struct commit *lookup_commit_reference_by_name(const char *name)
 +{
 +      unsigned char sha1[20];
 +      struct commit *commit;
 +
 +      if (get_sha1(name, sha1))
 +              return NULL;
 +      commit = lookup_commit_reference(sha1);
 +      if (!commit || parse_commit(commit))
 +              return NULL;
 +      return commit;
 +}
 +
  static unsigned long parse_commit_date(const char *buf, const char *tail)
  {
        const char *dateptr;
@@@ -150,8 -137,12 +150,8 @@@ struct commit_graft *read_graft_line(ch
                buf[--len] = '\0';
        if (buf[0] == '#' || buf[0] == '\0')
                return NULL;
 -      if ((len + 1) % 41) {
 -      bad_graft_data:
 -              error("bad graft data: %s", buf);
 -              free(graft);
 -              return NULL;
 -      }
 +      if ((len + 1) % 41)
 +              goto bad_graft_data;
        i = (len + 1) / 41 - 1;
        graft = xmalloc(sizeof(*graft) + 20 * i);
        graft->nr_parent = i;
                        goto bad_graft_data;
        }
        return graft;
 +
 +bad_graft_data:
 +      error("bad graft data: %s", buf);
 +      free(graft);
 +      return NULL;
  }
  
  static int read_graft_file(const char *graft_file)
@@@ -374,7 -360,7 +374,7 @@@ void free_commit_list(struct commit_lis
        }
  }
  
- struct commit_list * insert_by_date(struct commit *item, struct commit_list **list)
+ struct commit_list * commit_list_insert_by_date(struct commit *item, struct commit_list **list)
  {
        struct commit_list **pp = list;
        struct commit_list *p;
  }
  
  
- void sort_by_date(struct commit_list **list)
+ void commit_list_sort_by_date(struct commit_list **list)
  {
        struct commit_list *ret = NULL;
        while (*list) {
-               insert_by_date((*list)->item, &ret);
+               commit_list_insert_by_date((*list)->item, &ret);
                *list = (*list)->next;
        }
        *list = ret;
@@@ -412,7 -398,7 +412,7 @@@ struct commit *pop_most_recent_commit(s
                struct commit *commit = parents->item;
                if (!parse_commit(commit) && !(commit->object.flags & mark)) {
                        commit->object.flags |= mark;
-                       insert_by_date(commit, list);
+                       commit_list_insert_by_date(commit, list);
                }
                parents = parents->next;
        }
@@@ -501,7 -487,7 +501,7 @@@ void sort_in_topological_order(struct c
  
        /* process the list in topological order */
        if (!lifo)
-               sort_by_date(&work);
+               commit_list_sort_by_date(&work);
  
        pptr = list;
        *list = NULL;
                         */
                        if (--parent->indegree == 1) {
                                if (!lifo)
-                                       insert_by_date(parent, &work);
+                                       commit_list_insert_by_date(parent, &work);
                                else
                                        commit_list_insert(parent, &work);
                        }
@@@ -587,10 -573,10 +587,10 @@@ static struct commit_list *merge_bases_
        }
  
        one->object.flags |= PARENT1;
-       insert_by_date(one, &list);
+       commit_list_insert_by_date(one, &list);
        for (i = 0; i < n; i++) {
                twos[i]->object.flags |= PARENT2;
-               insert_by_date(twos[i], &list);
+               commit_list_insert_by_date(twos[i], &list);
        }
  
        while (interesting(list)) {
                if (flags == (PARENT1 | PARENT2)) {
                        if (!(commit->object.flags & RESULT)) {
                                commit->object.flags |= RESULT;
-                               insert_by_date(commit, &result);
+                               commit_list_insert_by_date(commit, &result);
                        }
                        /* Mark parents of a found merge stale */
                        flags |= STALE;
                        if (parse_commit(p))
                                return NULL;
                        p->object.flags |= flags;
-                       insert_by_date(p, &list);
+                       commit_list_insert_by_date(p, &list);
                }
        }
  
        while (list) {
                struct commit_list *next = list->next;
                if (!(list->item->object.flags & STALE))
-                       insert_by_date(list->item, &result);
+                       commit_list_insert_by_date(list->item, &result);
                free(list);
                list = next;
        }
@@@ -725,7 -711,7 +725,7 @@@ struct commit_list *get_merge_bases_man
        result = NULL;
        for (i = 0; i < cnt; i++) {
                if (rslt[i])
-                       insert_by_date(rslt[i], &result);
+                       commit_list_insert_by_date(rslt[i], &result);
        }
        free(rslt);
        return result;
diff --combined commit.h
index 3bfb31b5e0faad6ea04fde9e2f932f54d2e6c235,6452928d556a37f6ad1b67a544400af286278113..eb6c5af1f6b18546b3b3f1683142b15bb0ae9e34
+++ b/commit.h
@@@ -36,23 -36,22 +36,23 @@@ struct commit *lookup_commit(const unsi
  struct commit *lookup_commit_reference(const unsigned char *sha1);
  struct commit *lookup_commit_reference_gently(const unsigned char *sha1,
                                              int quiet);
 +struct commit *lookup_commit_reference_by_name(const char *name);
  
  int parse_commit_buffer(struct commit *item, void *buffer, unsigned long size);
  int parse_commit(struct commit *item);
  
  /* Find beginning and length of commit subject. */
  int find_commit_subject(const char *commit_buffer, const char **subject);
  
- struct commit_list * commit_list_insert(struct commit *item, struct commit_list **list_p);
+ struct commit_list *commit_list_insert(struct commit *item,
+                                       struct commit_list **list);
  unsigned commit_list_count(const struct commit_list *l);
- struct commit_list * insert_by_date(struct commit *item, struct commit_list **list);
+ struct commit_list *commit_list_insert_by_date(struct commit *item,
+                                   struct commit_list **list);
+ void commit_list_sort_by_date(struct commit_list **list);
  
  void free_commit_list(struct commit_list *list);
  
- void sort_by_date(struct commit_list **list);
  /* Commit formats */
  enum cmit_fmt {
        CMIT_FMT_RAW,
@@@ -77,7 -76,6 +77,7 @@@ struct pretty_print_contex
        int need_8bit_cte;
        int show_notes;
        struct reflog_walk_info *reflog_info;
 +      const char *output_encoding;
  };
  
  struct userformat_want {
@@@ -86,8 -84,6 +86,8 @@@
  
  extern int has_non_ascii(const char *text);
  struct rev_info; /* in revision.h, it circularly uses enum cmit_fmt */
 +extern char *logmsg_reencode(const struct commit *commit,
 +                           const char *output_encoding);
  extern char *reencode_commit_message(const struct commit *commit,
                                     const char **encoding_p);
  extern void get_commit_format(const char *arg, struct rev_info *);
diff --combined revision.c
index ded881263ba0ff703d9fd468e6785270ea68df5f,f8d4f87275c9552f2c2c2df7faaee65e99237653..7b9eaefae4ed03e994c2122453144b3c09591b9c
@@@ -444,15 -444,15 +444,15 @@@ static void try_to_simplify_commit(stru
        commit->object.flags |= TREESAME;
  }
  
- static void insert_by_date_cached(struct commit *p, struct commit_list **head,
+ static void commit_list_insert_by_date_cached(struct commit *p, struct commit_list **head,
                    struct commit_list *cached_base, struct commit_list **cache)
  {
        struct commit_list *new_entry;
  
        if (cached_base && p->date < cached_base->item->date)
-               new_entry = insert_by_date(p, &cached_base->next);
+               new_entry = commit_list_insert_by_date(p, &cached_base->next);
        else
-               new_entry = insert_by_date(p, head);
+               new_entry = commit_list_insert_by_date(p, head);
  
        if (cache && (!*cache || p->date < (*cache)->item->date))
                *cache = new_entry;
@@@ -494,7 -494,7 +494,7 @@@ static int add_parents_to_list(struct r
                        if (p->object.flags & SEEN)
                                continue;
                        p->object.flags |= SEEN;
-                       insert_by_date_cached(p, list, cached_base, cache_ptr);
+                       commit_list_insert_by_date_cached(p, list, cached_base, cache_ptr);
                }
                return 0;
        }
                p->object.flags |= left_flag;
                if (!(p->object.flags & SEEN)) {
                        p->object.flags |= SEEN;
-                       insert_by_date_cached(p, list, cached_base, cache_ptr);
+                       commit_list_insert_by_date_cached(p, list, cached_base, cache_ptr);
                }
                if (revs->first_parent_only)
                        break;
@@@ -1891,7 -1891,7 +1891,7 @@@ int prepare_revision_walk(struct rev_in
                if (commit) {
                        if (!(commit->object.flags & SEEN)) {
                                commit->object.flags |= SEEN;
-                               insert_by_date(commit, &revs->commits);
+                               commit_list_insert_by_date(commit, &revs->commits);
                        }
                }
                e++;
@@@ -2030,10 -2030,8 +2030,10 @@@ static struct commit *get_revision_1(st
                revs->commits = entry->next;
                free(entry);
  
 -              if (revs->reflog_info)
 +              if (revs->reflog_info) {
                        fake_reflog_parent(revs->reflog_info, commit);
 +                      commit->object.flags &= ~(ADDED | SEEN | SHOWN);
 +              }
  
                /*
                 * If we haven't done the list limiting, we need to look at
diff --combined sha1_name.c
index ceb9cdd860da78bf3fe0a1819846ded0e87fda8c,a96de0bdae482087dc44b2b193c3b75f65ffda0a..709ff2eee64cf106191ad274bede82a95d00e2a3
@@@ -7,8 -7,6 +7,8 @@@
  #include "refs.h"
  #include "remote.h"
  
 +static int get_sha1_oneline(const char *, unsigned char *, struct commit_list *);
 +
  static int find_short_object_filename(int len, const char *name, unsigned char *sha1)
  {
        struct alternate_object_database *alt;
@@@ -208,9 -206,7 +208,9 @@@ const char *find_unique_abbrev(const un
                if (exists
                    ? !status
                    : status == SHORT_NAME_NOT_FOUND) {
 -                      hex[len] = 0;
 +                      int cut_at = len + unique_abbrev_extra_length;
 +                      cut_at = (cut_at < 40) ? cut_at : 40;
 +                      hex[cut_at] = 0;
                        return hex;
                }
                len++;
@@@ -564,8 -560,6 +564,8 @@@ static int peel_onion(const char *name
                expected_type = OBJ_BLOB;
        else if (sp[0] == '}')
                expected_type = OBJ_NONE;
 +      else if (sp[0] == '/')
 +              expected_type = OBJ_COMMIT;
        else
                return -1;
  
                if (!o || (!o->parsed && !parse_object(o->sha1)))
                        return -1;
                hashcpy(sha1, o->sha1);
 +              return 0;
        }
 -      else {
 +
 +      /*
 +       * At this point, the syntax look correct, so
 +       * if we do not get the needed object, we should
 +       * barf.
 +       */
 +      o = peel_to_type(name, len, o, expected_type);
 +      if (!o)
 +              return -1;
 +
 +      hashcpy(sha1, o->sha1);
 +      if (sp[0] == '/') {
 +              /* "$commit^{/foo}" */
 +              char *prefix;
 +              int ret;
 +              struct commit_list *list = NULL;
 +
                /*
 -               * At this point, the syntax look correct, so
 -               * if we do not get the needed object, we should
 -               * barf.
 +               * $commit^{/}. Some regex implementation may reject.
 +               * We don't need regex anyway. '' pattern always matches.
                 */
 -              o = peel_to_type(name, len, o, expected_type);
 -              if (o) {
 -                      hashcpy(sha1, o->sha1);
 +              if (sp[1] == '}')
                        return 0;
 -              }
 -              return -1;
 +
 +              prefix = xstrndup(sp + 1, name + len - 1 - (sp + 1));
 +              commit_list_insert((struct commit *)o, &list);
 +              ret = get_sha1_oneline(prefix, sha1, list);
 +              free(prefix);
 +              return ret;
        }
        return 0;
  }
@@@ -707,15 -683,16 +707,15 @@@ static int handle_one_ref(const char *p
        }
        if (object->type != OBJ_COMMIT)
                return 0;
-       insert_by_date((struct commit *)object, list);
+       commit_list_insert_by_date((struct commit *)object, list);
 -      object->flags |= ONELINE_SEEN;
        return 0;
  }
  
 -static int get_sha1_oneline(const char *prefix, unsigned char *sha1)
 +static int get_sha1_oneline(const char *prefix, unsigned char *sha1,
 +                          struct commit_list *list)
  {
 -      struct commit_list *list = NULL, *backup = NULL, *l;
 -      int retval = -1;
 -      char *temp_commit_buffer = NULL;
 +      struct commit_list *backup = NULL, *l;
 +      int found = 0;
        regex_t regex;
  
        if (prefix[0] == '!') {
        if (regcomp(&regex, prefix, REG_EXTENDED))
                die("Invalid search pattern: %s", prefix);
  
 -      for_each_ref(handle_one_ref, &list);
 -      for (l = list; l; l = l->next)
 +      for (l = list; l; l = l->next) {
 +              l->item->object.flags |= ONELINE_SEEN;
                commit_list_insert(l->item, &backup);
 +      }
        while (list) {
 -              char *p;
 +              char *p, *to_free = NULL;
                struct commit *commit;
                enum object_type type;
                unsigned long size;
 +              int matches;
  
                commit = pop_most_recent_commit(&list, ONELINE_SEEN);
                if (!parse_object(commit->object.sha1))
                        continue;
 -              free(temp_commit_buffer);
                if (commit->buffer)
                        p = commit->buffer;
                else {
                        p = read_sha1_file(commit->object.sha1, &type, &size);
                        if (!p)
                                continue;
 -                      temp_commit_buffer = p;
 +                      to_free = p;
                }
 -              if (!(p = strstr(p, "\n\n")))
 -                      continue;
 -              if (!regexec(&regex, p + 2, 0, NULL, 0)) {
 +
 +              p = strstr(p, "\n\n");
 +              matches = p && !regexec(&regex, p + 2, 0, NULL, 0);
 +              free(to_free);
 +
 +              if (matches) {
                        hashcpy(sha1, commit->object.sha1);
 -                      retval = 0;
 +                      found = 1;
                        break;
                }
        }
        regfree(&regex);
 -      free(temp_commit_buffer);
        free_commit_list(list);
        for (l = backup; l; l = l->next)
                clear_commit_marks(l->item, ONELINE_SEEN);
 -      return retval;
 +      free_commit_list(backup);
 +      return found ? 0 : -1;
  }
  
  struct grab_nth_branch_switch_cbdata {
@@@ -961,24 -934,6 +961,24 @@@ int interpret_branch_name(const char *n
        return len;
  }
  
 +int strbuf_branchname(struct strbuf *sb, const char *name)
 +{
 +      int len = strlen(name);
 +      if (interpret_branch_name(name, sb) == len)
 +              return 0;
 +      strbuf_add(sb, name, len);
 +      return len;
 +}
 +
 +int strbuf_check_branch_ref(struct strbuf *sb, const char *name)
 +{
 +      strbuf_branchname(sb, name);
 +      if (name[0] == '-')
 +              return CHECK_REF_FORMAT_ERROR;
 +      strbuf_splice(sb, 0, 0, "refs/heads/", 11);
 +      return check_ref_format(sb->buf);
 +}
 +
  /*
   * This is like "get_sha1_basic()", except it allows "sha1 expressions",
   * notably "xyz^" for "parent of xyz"
@@@ -1091,23 -1046,6 +1091,23 @@@ int get_sha1_with_mode_1(const char *na
        return ret;
  }
  
 +static char *resolve_relative_path(const char *rel)
 +{
 +      if (prefixcmp(rel, "./") && prefixcmp(rel, "../"))
 +              return NULL;
 +
 +      if (!startup_info)
 +              die("BUG: startup_info struct is not initialized.");
 +
 +      if (!is_inside_work_tree())
 +              die("relative path syntax can't be used outside working tree.");
 +
 +      /* die() inside prefix_path() if resolved path is outside worktree */
 +      return prefix_path(startup_info->prefix,
 +                         startup_info->prefix ? strlen(startup_info->prefix) : 0,
 +                         rel);
 +}
 +
  int get_sha1_with_context_1(const char *name, unsigned char *sha1,
                            struct object_context *oc,
                            int gently, const char *prefix)
        if (!ret)
                return ret;
        /* sha1:path --> object name of path in ent sha1
 -       * :path -> object name of path in index
 +       * :path -> object name of absolute path in index
 +       * :./path -> object name of path relative to cwd in index
         * :[0-3]:path -> object name of path in index at stage
         * :/foo -> recent commit matching foo
         */
        if (name[0] == ':') {
                int stage = 0;
                struct cache_entry *ce;
 +              char *new_path = NULL;
                int pos;
 -              if (namelen > 2 && name[1] == '/')
 -                      /* don't need mode for commit */
 -                      return get_sha1_oneline(name + 2, sha1);
 +              if (namelen > 2 && name[1] == '/') {
 +                      struct commit_list *list = NULL;
 +                      for_each_ref(handle_one_ref, &list);
 +                      return get_sha1_oneline(name + 2, sha1, list);
 +              }
                if (namelen < 3 ||
                    name[2] != ':' ||
                    name[1] < '0' || '3' < name[1])
                        stage = name[1] - '0';
                        cp = name + 3;
                }
 -              namelen = namelen - (cp - name);
 +              new_path = resolve_relative_path(cp);
 +              if (!new_path) {
 +                      namelen = namelen - (cp - name);
 +              } else {
 +                      cp = new_path;
 +                      namelen = strlen(cp);
 +              }
  
                strncpy(oc->path, cp,
                        sizeof(oc->path));
                        if (ce_stage(ce) == stage) {
                                hashcpy(sha1, ce->sha1);
                                oc->mode = ce->ce_mode;
 +                              free(new_path);
                                return 0;
                        }
                        pos++;
                }
                if (!gently)
                        diagnose_invalid_index_path(stage, prefix, cp);
 +              free(new_path);
                return -1;
        }
        for (cp = name, bracket_depth = 0; *cp; cp++) {
                }
                if (!get_sha1_1(name, cp-name, tree_sha1)) {
                        const char *filename = cp+1;
 +                      char *new_filename = NULL;
 +
 +                      new_filename = resolve_relative_path(filename);
 +                      if (new_filename)
 +                              filename = new_filename;
                        ret = get_tree_entry(tree_sha1, filename, sha1, &oc->mode);
                        if (!gently) {
                                diagnose_invalid_sha1_path(prefix, filename,
                                sizeof(oc->path));
                        oc->path[sizeof(oc->path)-1] = '\0';
  
 +                      free(new_filename);
                        return ret;
                } else {
                        if (!gently)