<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/diff.c, branch v2.0.2</title>
<subtitle>Fork of git SCM with my patches.</subtitle>
<id>http://git.kilabit.info/git/atom?h=v2.0.2</id>
<link rel='self' href='http://git.kilabit.info/git/atom?h=v2.0.2'/>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/'/>
<updated>2014-06-25T18:47:23Z</updated>
<entry>
<title>Merge branch 'jk/diff-follow-must-take-one-pathspec' into maint</title>
<updated>2014-06-25T18:47:23Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-06-25T18:47:23Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=cb4575fb181c418a85231ec962c01ef314ec9dfc'/>
<id>urn:sha1:cb4575fb181c418a85231ec962c01ef314ec9dfc</id>
<content type='text'>
"git format-patch" did not enforce the rule that the "--follow"
option from the log/diff family of commands must be used with
exactly one pathspec.

* jk/diff-follow-must-take-one-pathspec:
  move "--follow needs one pathspec" rule to diff_setup_done
</content>
</entry>
<entry>
<title>move "--follow needs one pathspec" rule to diff_setup_done</title>
<updated>2014-05-20T18:09:03Z</updated>
<author>
<name>Jeff King</name>
<email>peff@peff.net</email>
</author>
<published>2014-05-20T06:49:20Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=dd63f169d9381a77db469325688a2bef1a79a9e1'/>
<id>urn:sha1:dd63f169d9381a77db469325688a2bef1a79a9e1</id>
<content type='text'>
Because of the way "--follow" is implemented, we must have
exactly one pathspec. "git log" enforces this restriction,
but other users of the revision traversal code do not. For
example, "git format-patch --follow" will segfault during
try_to_follow_renames, as we have no pathspecs at all.

We can push this check down into diff_setup_done, which is
probably a better place anyway. It is the diff code that
introduces this restriction, so other parts of the code
should not need to care themselves.

Reported-by: "Michael S. Tsirkin" &lt;mst@redhat.com&gt;
Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'jk/external-diff-use-argv-array' (early part)</title>
<updated>2014-04-28T22:47:35Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-28T22:47:35Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=5f11a7aad0704918fde4855500caba6a3f130648'/>
<id>urn:sha1:5f11a7aad0704918fde4855500caba6a3f130648</id>
<content type='text'>
Crash fix for codepath that miscounted the necessary size for an
array when spawning an external diff program.

* 'jk/external-diff-use-argv-array' (early part):
  run_external_diff: use an argv_array for the command line
</content>
</entry>
<entry>
<title>run_external_diff: use an argv_array for the command line</title>
<updated>2014-04-21T17:29:50Z</updated>
<author>
<name>Jeff King</name>
<email>peff@peff.net</email>
</author>
<published>2014-04-19T19:17:06Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=82fbf269b9994d172719b2d456db5ef8453b323d'/>
<id>urn:sha1:82fbf269b9994d172719b2d456db5ef8453b323d</id>
<content type='text'>
We currently generate the command-line for the external
command using a fixed-length array of size 10. But if there
is a rename, we actually need 11 elements (10 items, plus a
NULL), and end up writing a random NULL onto the stack.

Rather than bump the limit, let's just use an argv_array, which
makes this sort of error impossible.

Noticed-by: Max L &lt;infthi.inbox@gmail.com&gt;
Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>i18n: remove obsolete comments for translators in diffstat generation</title>
<updated>2014-04-17T18:09:56Z</updated>
<author>
<name>Jiang Xin</name>
<email>worldhello.net@gmail.com</email>
</author>
<published>2014-04-17T05:37:19Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=d1d96a82bbe302f552364d1629958b7d7247fd8f'/>
<id>urn:sha1:d1d96a82bbe302f552364d1629958b7d7247fd8f</id>
<content type='text'>
Since we do not translate diffstat any more, remove the obsolete comments.

Signed-off-by: Jiang Xin &lt;worldhello.net@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'jl/nor-or-nand-and'</title>
<updated>2014-04-08T19:00:28Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-08T19:00:27Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=d59c12d7ad39f942fc60578ba1e934822f40445b'/>
<id>urn:sha1:d59c12d7ad39f942fc60578ba1e934822f40445b</id>
<content type='text'>
Eradicate mistaken use of "nor" (that is, essentially "nor" used
not in "neither A nor B" ;-)) from in-code comments, command output
strings, and documentations.

* jl/nor-or-nand-and:
  code and test: fix misuses of "nor"
  comments: fix misuses of "nor"
  contrib: fix misuses of "nor"
  Documentation: fix misuses of "nor"
</content>
</entry>
<entry>
<title>comments: fix misuses of "nor"</title>
<updated>2014-03-31T22:29:27Z</updated>
<author>
<name>Justin Lebar</name>
<email>jlebar@google.com</email>
</author>
<published>2014-03-31T22:11:46Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=01689909eb9f8366583c44b325d8d9ba259a2538'/>
<id>urn:sha1:01689909eb9f8366583c44b325d8d9ba259a2538</id>
<content type='text'>
Signed-off-by: Justin Lebar &lt;jlebar@google.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'tr/diff-submodule-no-reuse-worktree' into maint</title>
<updated>2014-03-18T21:03:41Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-03-18T21:03:41Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=a5aca6e883639ce3a3ad07271032905edc0ac608'/>
<id>urn:sha1:a5aca6e883639ce3a3ad07271032905edc0ac608</id>
<content type='text'>
"git diff --external-diff" incorrectly fed the submodule directory
in the working tree to the external diff driver when it knew it is
the same as one of the versions being compared.

* tr/diff-submodule-no-reuse-worktree:
  diff: do not reuse_worktree_file for submodules
</content>
</entry>
<entry>
<title>Merge branch 'nd/diff-quiet-stat-dirty' into maint</title>
<updated>2014-03-18T20:59:56Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-03-18T20:59:55Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=34120a5fb555a24904a85e64554bdcc7bcc17963'/>
<id>urn:sha1:34120a5fb555a24904a85e64554bdcc7bcc17963</id>
<content type='text'>
"git diff --quiet -- pathspec1 pathspec2" sometimes did not return
correct status value.

* nd/diff-quiet-stat-dirty:
  diff: do not quit early on stat-dirty files
  diff.c: move diffcore_skip_stat_unmatch core logic out for reuse later
</content>
</entry>
<entry>
<title>Merge branch 'rm/strchrnul-not-strlen'</title>
<updated>2014-03-18T20:51:18Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-03-18T20:51:18Z</published>
<link rel='alternate' type='text/html' href='http://git.kilabit.info/git/commit/?id=6f75e48323bc3071495025fd14105e9dffa9e8dd'/>
<id>urn:sha1:6f75e48323bc3071495025fd14105e9dffa9e8dd</id>
<content type='text'>
* rm/strchrnul-not-strlen:
  use strchrnul() in place of strchr() and strlen()
</content>
</entry>
</feed>
