From d99082e0e3436f5fcf259aa1a4189473f6bb5ce3 Mon Sep 17 00:00:00 2001 From: Linus Torvalds Date: Fri, 15 Apr 2005 22:53:45 -0700 Subject: [PATCH] Make "read-tree" merge the trees it reads by giving them consecutive states. Normally you'd use state 0 for the "merged" state, and start out with state 1 being "origin", state 2 being "first tree" and state 3 being "second tree". Once all the index entries are back in state 0, we have a successful merge and can write the result tree back. --- read-tree.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 53 insertions(+), 6 deletions(-) diff --git a/read-tree.c b/read-tree.c index 7ee1275b0..59e6950a4 100644 --- a/read-tree.c +++ b/read-tree.c @@ -5,6 +5,8 @@ */ #include "cache.h" +static int stage = 0; + static int read_one_entry(unsigned char *sha1, const char *base, int baselen, const char *pathname, unsigned mode) { int len = strlen(pathname); @@ -14,7 +16,7 @@ static int read_one_entry(unsigned char *sha1, const char *base, int baselen, co memset(ce, 0, size); ce->ce_mode = htonl(mode); - ce->ce_flags = htons(baselen + len); + ce->ce_flags = create_ce_flags(baselen + len, stage); memcpy(ce->name, base, baselen); memcpy(ce->name + baselen, pathname, len+1); memcpy(ce->sha1, sha1, 20); @@ -71,6 +73,49 @@ static void remove_lock_file(void) unlink(".git/index.lock"); } +/* + * This removes all identical entries and collapses them to state 0. + * + * _Any_ other merge (even a trivial one, like both ) is left to user policy. + * That includes "both created the same file", and "both removed the same + * file" - which are trivial, but the user might still want to _note_ it. + */ +static int same_entry(struct cache_entry *a, + struct cache_entry *b, + struct cache_entry *c) +{ + int len = ce_namelen(a); + return a->ce_mode == b->ce_mode && + a->ce_mode == c->ce_mode && + ce_namelen(b) == len && + ce_namelen(c) == len && + !memcmp(a->name, b->name, len) && + !memcmp(a->name, c->name, len) && + !memcmp(a->sha1, b->sha1, 20) && + !memcmp(a->sha1, c->sha1, 20); +} + +static void trivially_merge_cache(struct cache_entry **src, int nr) +{ + struct cache_entry **dst = src; + + while (nr) { + struct cache_entry *ce; + + ce = src[0]; + if (nr > 2 && same_entry(ce, src[1], src[2])) { + ce->ce_flags &= ~htons(CE_STAGEMASK); + src += 2; + nr -= 2; + active_nr -= 2; + } + *dst = ce; + src++; + dst++; + nr--; + } +} + int main(int argc, char **argv) { int i, newfd; @@ -85,19 +130,21 @@ int main(int argc, char **argv) for (i = 1; i < argc; i++) { const char *arg = argv[i]; - /* "-m" stands for "merge" current directory cache */ + /* "-m" stands for "merge", meaning we start in stage 1 */ if (!strcmp(arg, "-m")) { - if (active_cache) - die("read-tree: cannot merge old cache on top of new"); - if (read_cache() < 0) - die("read-tree: corrupt directory cache"); + stage = 1; continue; } if (get_sha1_hex(arg, sha1) < 0) usage("read-tree [-m] "); + if (stage > 3) + usage("can't merge more than two trees"); if (read_tree(sha1, "", 0) < 0) die("failed to unpack tree object %s", arg); + stage++; } + if (stage == 4) + trivially_merge_cache(active_cache, active_nr); if (write_cache(newfd, active_cache, active_nr) || rename(".git/index.lock", ".git/index")) die("unable to write new index file"); -- 2.30.2