<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/git.git/diff.c, branch v1.7.3.5</title>
<subtitle>Git
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.7.3.5</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.7.3.5'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/'/>
<updated>2010-12-20T02:41:32Z</updated>
<entry>
<title>fill_textconv(): Don't get/put cache if sha1 is not valid</title>
<updated>2010-12-20T02:41:32Z</updated>
<author>
<name>Kirill Smelkov</name>
<email>kirr@landau.phys.spbu.ru</email>
</author>
<published>2010-12-18T14:54:12Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=9ec09b0495f5e672223025f5e681371034da2538'/>
<id>urn:sha1:9ec09b0495f5e672223025f5e681371034da2538</id>
<content type='text'>
When blaming files in the working tree, the filespec is marked with
!sha1_valid, as we have not given the contents an object name yet.  The
function to cache textconv results (keyed on the object name), however,
didn't check this condition, and ended up on storing the cached result
under a random object name.

Cc: Axel Bonnet &lt;axel.bonnet@ensimag.imag.fr&gt;
Cc: Clément Poulain &lt;clement.poulain@ensimag.imag.fr&gt;
Cc: Diane Gasselin &lt;diane.gasselin@ensimag.imag.fr&gt;
Cc: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Kirill Smelkov &lt;kirr@landau.phys.spbu.ru&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'np/diff-in-corrupt-repository' into maint</title>
<updated>2010-12-09T18:36:39Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-12-09T18:36:39Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=8577def6fc1dda3f18a284162aba07819fbcbcd4'/>
<id>urn:sha1:8577def6fc1dda3f18a284162aba07819fbcbcd4</id>
<content type='text'>
* np/diff-in-corrupt-repository:
  diff: don't presume empty file when corresponding object is missing
</content>
</entry>
<entry>
<title>Merge branch 'cm/diff-check-at-eol' into maint</title>
<updated>2010-12-09T18:36:10Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-12-09T18:36:10Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=ae0a37cd6bc05b5ab511d0d1a541dc57d52753e6'/>
<id>urn:sha1:ae0a37cd6bc05b5ab511d0d1a541dc57d52753e6</id>
<content type='text'>
* cm/diff-check-at-eol:
  diff --check: correct line numbers of new blank lines at EOF
</content>
</entry>
<entry>
<title>Merge branch 'cb/diff-fname-optim' into maint</title>
<updated>2010-11-24T20:46:26Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-11-24T20:46:26Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=9cffe2018a8c400cbd50a8aec4eddea97c4ede2f'/>
<id>urn:sha1:9cffe2018a8c400cbd50a8aec4eddea97c4ede2f</id>
<content type='text'>
* cb/diff-fname-optim:
  diff: avoid repeated scanning while looking for funcname
  do not search functions for patch ID
  add rebase patch id tests
</content>
</entry>
<entry>
<title>Merge branch 'jk/no-textconv-symlink' into maint</title>
<updated>2010-11-24T20:46:20Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-11-24T20:46:20Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=78bce6c7e93f4c8797da06cf93f59b7ac8e84ec0'/>
<id>urn:sha1:78bce6c7e93f4c8797da06cf93f59b7ac8e84ec0</id>
<content type='text'>
* jk/no-textconv-symlink:
  diff: don't use pathname-based diff drivers for symlinks
</content>
</entry>
<entry>
<title>Fix copy-pasted comments related to tree diff handling.</title>
<updated>2010-10-25T07:21:56Z</updated>
<author>
<name>Yann Dirson</name>
<email>ydirson@altern.org</email>
</author>
<published>2010-10-24T21:03:10Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=c3fced64981e3d2700d83c0a6661ac347df5121e'/>
<id>urn:sha1:c3fced64981e3d2700d83c0a6661ac347df5121e</id>
<content type='text'>
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>diff: don't presume empty file when corresponding object is missing</title>
<updated>2010-10-22T05:23:34Z</updated>
<author>
<name>Nicolas Pitre</name>
<email>nico@fluxnic.net</email>
</author>
<published>2010-10-22T04:02:27Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=c50c4316e1eed362bee387e4cbfbe1138957f75b'/>
<id>urn:sha1:c50c4316e1eed362bee387e4cbfbe1138957f75b</id>
<content type='text'>
The low-level diff code will happily produce totally bogus diff output
with a broken repository via format-patch and friends by treating missing
objects as empty files.  Let's prevent that from happening any longer.

Reported-by: Uwe Kleine-König &lt;u.kleine-koenig@pengutronix.de&gt;
Signed-off-by: Nicolas Pitre &lt;nico@fluxnic.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>diff --check: correct line numbers of new blank lines at EOF</title>
<updated>2010-10-17T01:57:35Z</updated>
<author>
<name>Christoph Mallon</name>
<email>christoph.mallon@gmx.de</email>
</author>
<published>2010-10-10T17:24:06Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=8837d33595b78dc4a52038856bc9a5c3f4b2ff50'/>
<id>urn:sha1:8837d33595b78dc4a52038856bc9a5c3f4b2ff50</id>
<content type='text'>
The whitespace check printed the value of the wrong variable, i.e. the
beginning of the block of blank lines at the EOF (possibly absent) in the
old file.

As "git diff --check" is used by users to check their changes before
making a commit, we should point at the line number in the file after
the change.

Signed-off-by: Christoph Mallon &lt;christoph.mallon@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>diff: trivial fix for --output file error message</title>
<updated>2010-09-29T20:25:17Z</updated>
<author>
<name>Matthieu Moy</name>
<email>Matthieu.Moy@imag.fr</email>
</author>
<published>2010-09-29T07:26:23Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=9ec26eb7cd7767faffac9bac5e22ede0d25e20e3'/>
<id>urn:sha1:9ec26eb7cd7767faffac9bac5e22ede0d25e20e3</id>
<content type='text'>
The option argument is either after the equal sign in --output=... or in
the next command-line argument. optarg is the reliable way to access it.

Signed-off-by: Matthieu Moy &lt;Matthieu.Moy@imag.fr&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>do not search functions for patch ID</title>
<updated>2010-09-24T01:35:07Z</updated>
<author>
<name>Clemens Buchacher</name>
<email>drizzd@aon.at</email>
</author>
<published>2010-09-19T09:59:28Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=ad14b450c0c70bfcf90de0aa19ffa5e44e402b69'/>
<id>urn:sha1:ad14b450c0c70bfcf90de0aa19ffa5e44e402b69</id>
<content type='text'>
Visual aids, such as the function name in the hunk
header, are not necessary for the purposes of
computing a patch ID.

This is a performance optimization.

Signed-off-by: Clemens Buchacher &lt;drizzd@aon.at&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
