<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/git.git/diff-delta.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-10-15T06:38:01Z</updated>
<entry>
<title>Revert "move pack creation to version 3"</title>
<updated>2006-10-15T06:38:01Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2006-10-15T06:37:41Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=29f049a0c277be72637f74f1f90a89dccd3475bc'/>
<id>urn:sha1:29f049a0c277be72637f74f1f90a89dccd3475bc</id>
<content type='text'>
This reverts commit 16854571aae6302f457c5fbee41ac64669b09595.
Git as recent as v1.1.6 do not understand version 3 delta.

v1.2.0 is Ok and I personally would say it is old enough, but
the improvement between version 2 and version 3 delta is not
bit enough to justify breaking older clients.

We should resurrect this later, but when we do so we shold
make it conditional.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>move pack creation to version 3</title>
<updated>2006-09-23T02:24:52Z</updated>
<author>
<name>Nicolas Pitre</name>
<email>nico@cam.org</email>
</author>
<published>2006-09-21T04:11:59Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=16854571aae6302f457c5fbee41ac64669b09595'/>
<id>urn:sha1:16854571aae6302f457c5fbee41ac64669b09595</id>
<content type='text'>
It's been quite a while now that GIT is able to read version 3 packs.
Let's create them at last.

Signed-off-by: Nicolas Pitre &lt;nico@cam.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Use xrealloc instead of realloc</title>
<updated>2006-08-27T00:54:06Z</updated>
<author>
<name>Jonas Fonseca</name>
<email>fonseca@diku.dk</email>
</author>
<published>2006-08-26T14:16:18Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=83572c1a914d3f7a8dd66d954c11bbc665b7b923'/>
<id>urn:sha1:83572c1a914d3f7a8dd66d954c11bbc665b7b923</id>
<content type='text'>
Change places that use realloc, without a proper error path, to instead use
xrealloc. Drop an erroneous error path in the daemon code that used errno
in the die message in favour of the simpler xrealloc.

Signed-off-by: Jonas Fonseca &lt;fonseca@diku.dk&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Fix a comparison bug in diff-delta.c</title>
<updated>2006-08-23T10:05:27Z</updated>
<author>
<name>Pierre Habouzit</name>
<email>madcoder@debian.org</email>
</author>
<published>2006-08-23T09:17:55Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=b05faa2da9ec24d737bbba47c71e815255f3eaa7'/>
<id>urn:sha1:b05faa2da9ec24d737bbba47c71e815255f3eaa7</id>
<content type='text'>
(1 &lt;&lt; i) &lt; hspace is compared in the `int` space rather that in the
unsigned one.  the result will be wrong if hspace is between 0x40000000
and 0x80000000.

Signed-off-by: Pierre Habouzit &lt;madcoder@debian.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Fix more typos, primarily in the code</title>
<updated>2006-07-10T07:36:44Z</updated>
<author>
<name>Pavel Roskin</name>
<email>proski@gnu.org</email>
</author>
<published>2006-07-10T05:50:18Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=82e5a82fd73edb80a841f5fab1660e14b9b8f3ad'/>
<id>urn:sha1:82e5a82fd73edb80a841f5fab1660e14b9b8f3ad</id>
<content type='text'>
The only visible change is that git-blame doesn't understand
"--compability" anymore, but it does accept "--compatibility" instead,
which is already documented.

Signed-off-by: Pavel Roskin &lt;proski@gnu.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Remove all void-pointer arithmetic.</title>
<updated>2006-06-20T08:59:46Z</updated>
<author>
<name>Florian Forster</name>
<email>octo@verplant.org</email>
</author>
<published>2006-06-18T15:18:09Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=1d7f171c3a456b980d821ee0f68e01535a5f7e36'/>
<id>urn:sha1:1d7f171c3a456b980d821ee0f68e01535a5f7e36</id>
<content type='text'>
ANSI C99 doesn't allow void-pointer arithmetic. This patch fixes this in
various ways. Usually the strategy that required the least changes was used.

Signed-off-by: Florian Forster &lt;octo@verplant.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Initialize FAMs using `FLEX_ARRAY'.</title>
<updated>2006-06-19T04:19:09Z</updated>
<author>
<name>Florian Forster</name>
<email>octo@verplant.org</email>
</author>
<published>2006-06-18T15:18:04Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=63f175693e46c8e1562e68a40086ec864f43886b'/>
<id>urn:sha1:63f175693e46c8e1562e68a40086ec864f43886b</id>
<content type='text'>
When initializing a `flexible array member' the macro `FLEX_ARRAY' should be
used. This was forgotten in `diff-delta.c'.

Signed-off-by: Florian Forster &lt;octo@verplant.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>fix diff-delta bad memory access</title>
<updated>2006-05-10T17:53:59Z</updated>
<author>
<name>Nicolas Pitre</name>
<email>nico@cam.org</email>
</author>
<published>2006-05-10T16:26:08Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=639ca5497279607665847f2e3a11064441a8f2a6'/>
<id>urn:sha1:639ca5497279607665847f2e3a11064441a8f2a6</id>
<content type='text'>
It cannot be assumed that the given buffer will never be moved when
shrinking the allocated memory size with realloc().  So let's ignore
that optimization for now.

This patch makes Electric Fence happy on Linux.

Signed-off-by: Nicolas Pitre &lt;nico@cam.org&gt;
Acked-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>improve diff-delta with sparse and/or repetitive data</title>
<updated>2006-05-03T04:32:39Z</updated>
<author>
<name>Nicolas Pitre</name>
<email>nico@cam.org</email>
</author>
<published>2006-05-03T03:31:00Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=06a9f9203570d21f9ef5fe219cdde527dcdf0990'/>
<id>urn:sha1:06a9f9203570d21f9ef5fe219cdde527dcdf0990</id>
<content type='text'>
It is useless to preserve multiple hash entries for consecutive blocks
with the same hash.  Keeping only the first one will allow for matching
the longest string of identical bytes while subsequent blocks will only
allow for shorter matches.  The backward matching code will match the
end of it as necessary.

This improves both performances (no repeated string compare with long
successions of identical bytes, or even small group of bytes), as well
as compression (less likely to need random hash bucket entry culling),
especially with sparse files.

With well behaved data sets this patch doesn't change much.

Signed-off-by: Nicolas Pitre &lt;nico@cam.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>tiny optimization to diff-delta</title>
<updated>2006-05-03T04:32:32Z</updated>
<author>
<name>Nicolas Pitre</name>
<email>nico@cam.org</email>
</author>
<published>2006-05-03T03:46:51Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=2d08e5dd730680f7f8645a6326ec653435e032df'/>
<id>urn:sha1:2d08e5dd730680f7f8645a6326ec653435e032df</id>
<content type='text'>
This is my assembly freak side looking at generated code again.  And
since create_delta() is certainly pretty high on the radar every bits
count.  In this case shorter code is generated if hash_mask is not
copied to a local variable.

Signed-off-by: Nicolas Pitre &lt;nico@cam.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
</feed>
