author | Linus Torvalds <torvalds@linux-foundation.org> | |
Sun, 16 Mar 2008 18:42:50 +0000 (11:42 -0700) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Sun, 16 Mar 2008 21:25:53 +0000 (14:25 -0700) | ||
commit | 7f8ab8dc07ba4e892d0c2b5e7ac8436f9e845920 | |
tree | 005c400fe384d494b3d87843d5cf0235ea7939a1 | tree | snapshot |
parent | 198724ad4ef0630892e8dbbab92e92ff4f979075 | commit | diff |
Don't update unchanged merge entries
In commit 34110cd4e394e3f92c01a4709689b384c34645d8 ("Make 'unpack_trees()'
have a separate source and destination index") I introduced a really
stupid bug in that it would always add merged entries with the CE_UPDATE
flag set. That caused us to always re-write the file, even when it was
already up-to-date in the source index.
Not only is that really stupid from a performance angle, but more
importantly it's actively wrong: if we have dirty state in the tree when
we merge, overwriting it with the result of the merge will incorrectly
overwrite that dirty state.
This trivially fixes the problem - simply don't set the CE_UPDATE flag
when the merge result matches the old state.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
In commit 34110cd4e394e3f92c01a4709689b384c34645d8 ("Make 'unpack_trees()'
have a separate source and destination index") I introduced a really
stupid bug in that it would always add merged entries with the CE_UPDATE
flag set. That caused us to always re-write the file, even when it was
already up-to-date in the source index.
Not only is that really stupid from a performance angle, but more
importantly it's actively wrong: if we have dirty state in the tree when
we merge, overwriting it with the result of the merge will incorrectly
overwrite that dirty state.
This trivially fixes the problem - simply don't set the CE_UPDATE flag
when the merge result matches the old state.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t1004-read-tree-m-u-wf.sh | diff | blob | history | |
unpack-trees.c | diff | blob | history |