<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/git.git/diffcore-break.c, branch v1.4.4.4</title>
<subtitle>Git
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.4.4.4</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.4.4.4'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/'/>
<updated>2006-08-17T21:23:53Z</updated>
<entry>
<title>Do not use memcmp(sha1_1, sha1_2, 20) with hardcoded length.</title>
<updated>2006-08-17T21:23:53Z</updated>
<author>
<name>David Rientjes</name>
<email>rientjes@google.com</email>
</author>
<published>2006-08-17T18:54:57Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=a89fccd28197fa179828c8596791ff16e2268d20'/>
<id>urn:sha1:a89fccd28197fa179828c8596791ff16e2268d20</id>
<content type='text'>
Introduces global inline:

	hashcmp(const unsigned char *sha1, const unsigned char *sha2)

Uses memcmp for comparison and returns the result based on the length of
the hash name (a future runtime decision).

Acked-by: Alex Riesen &lt;raa.lkml@gmail.com&gt;
Signed-off-by: David Rientjes &lt;rientjes@google.com&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>diffcore-rename: somewhat optimized.</title>
<updated>2006-03-12T11:22:10Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2006-03-12T11:22:10Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=c06c79667c9514aed00d29bcd80bd0cee7cc5a25'/>
<id>urn:sha1:c06c79667c9514aed00d29bcd80bd0cee7cc5a25</id>
<content type='text'>
This changes diffcore-rename to reuse statistics information
gathered during similarity estimation, and updates the hashtable
implementation used to keep track of the statistics to be
denser.  This seems to give better performance.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>diffcore-break: similarity estimator fix.</title>
<updated>2006-03-04T21:26:36Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2006-03-04T09:03:53Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=4d0f39cecf93222401a81216e5cf2528e8abc6ae'/>
<id>urn:sha1:4d0f39cecf93222401a81216e5cf2528e8abc6ae</id>
<content type='text'>
This is a companion patch to the previous fix to diffcore-rename.
The merging-back process should use a logic similar to what is used
there.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>diffcore-rename: split out the delta counting code.</title>
<updated>2006-03-01T04:20:04Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2006-03-01T00:01:36Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=65416758cd83772f2f3c69f1bd1f501608e64745'/>
<id>urn:sha1:65416758cd83772f2f3c69f1bd1f501608e64745</id>
<content type='text'>
This is to rework diffcore break/rename/copy detection code
so that it does not affected when deltifier code gets improved.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>diffcore-break: micro-optimize by avoiding delta between identical files.</title>
<updated>2006-03-01T04:19:47Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2006-03-01T04:19:47Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=aeecd23ae2785a0462d42191974e9d9a8e439fbe'/>
<id>urn:sha1:aeecd23ae2785a0462d42191974e9d9a8e439fbe</id>
<content type='text'>
We did not check if we have the same file on both sides when
computing break score.  This is usually not a problem, but if
the user said --find-copies-harde with -B, we ended up trying a
delta between the same data even when we know the SHA1 hash of
both sides match.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>diffcore-break: do not break too small filepair.</title>
<updated>2005-12-13T01:15:55Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2005-12-13T01:15:55Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=0532a5e46b88cd70c952a2bf5dc63681be32a2d9'/>
<id>urn:sha1:0532a5e46b88cd70c952a2bf5dc63681be32a2d9</id>
<content type='text'>
Somehow we checked only one side and not the other.  By checking
the filesize upfront, we can bypass generating delta
unnecessarily.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>diffcore-break.c: check diff_delta() return value.</title>
<updated>2005-12-12T20:57:25Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junio@twinsun.com</email>
</author>
<published>2005-12-12T20:50:40Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=d28c8af623b0d15740c2af0106d8e2bf54a3ac52'/>
<id>urn:sha1:d28c8af623b0d15740c2af0106d8e2bf54a3ac52</id>
<content type='text'>
This bug caused Darrin Thompson to notice that our deltifier was
half broken and punting on an empty blob.

Signed-off-by: Junio C Hamano &lt;junio@twinsun.com&gt;
</content>
</entry>
<entry>
<title>Revert "[PATCH] plug memory leak in diff.c::diff_free_filepair()"</title>
<updated>2005-09-14T21:06:50Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2005-09-14T21:06:50Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=19397b4521bcc27eb224413fb71519223b94290f'/>
<id>urn:sha1:19397b4521bcc27eb224413fb71519223b94290f</id>
<content type='text'>
This reverts 068eac91ce04b9aca163acb1927c3878c45d1a07 commit.
</content>
</entry>
<entry>
<title>[PATCH] plug memory leak in diff.c::diff_free_filepair()</title>
<updated>2005-08-14T01:28:55Z</updated>
<author>
<name>Yasushi SHOJI</name>
<email>yashi@atmark-techno.com</email>
</author>
<published>2005-08-13T10:58:56Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=068eac91ce04b9aca163acb1927c3878c45d1a07'/>
<id>urn:sha1:068eac91ce04b9aca163acb1927c3878c45d1a07</id>
<content type='text'>
When I run git-diff-tree on big change, it seems the command eats so
much memory.  so I just put git under valgrind to see what's going on.
diff_free_filespec_data() doesn't free diff_filespec itself.

[jc: I ended up doing things slightly differently from Yasushi's
patch.  The original idea was to use free_filespec_data() only to
free the data portion and keep useing the filespec itself, but
no existing code seems to do things that way, so I just yanked
that part out.]

Signed-off-by: Yasushi SHOJI &lt;yashi@atmark-techno.com&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>[PATCH] Fixlets on top of Nico's clean-up.</title>
<updated>2005-06-29T16:11:38Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2005-06-29T07:32:11Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=3c84974207fb6a98eaa393cd4c3481d8ecdb8ce2'/>
<id>urn:sha1:3c84974207fb6a98eaa393cd4c3481d8ecdb8ce2</id>
<content type='text'>
If we prefer 0 as maxsize for diff_delta() to say "unlimited", let's be
consistent about it.

This patch also fixes type mismatch in a call to get_delta_hdr_size()
from packed_delta_info().

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
</feed>
