<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/dir.c, branch v2.0.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.0.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.0.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2014-06-02T22:48:48Z</updated>
<entry>
<title>dir.c:trim_trailing_spaces(): fix for " \ " sequence</title>
<updated>2014-06-02T22:48:48Z</updated>
<author>
<name>Pasha Bolokhov</name>
<email>pasha.bolokhov@gmail.com</email>
</author>
<published>2014-06-02T22:36:56Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=e61a6c1d8272f9d7bdc861ed28bfac5fb7e33556'/>
<id>urn:sha1:e61a6c1d8272f9d7bdc861ed28bfac5fb7e33556</id>
<content type='text'>
Discard the unnecessary 'nr_spaces' variable, remove 'strlen()' and
improve the 'if' structure.  Switch to pointers instead of integers
to control the loop.

Slightly more rare occurrences of 'text  \    ' with a backslash
in between spaces are handled correctly.  Namely, the code in
7e2e4b37 (dir: ignore trailing spaces in exclude patterns, 2014-02-09)
does not reset 'last_space' when a backslash is encountered and the above
line stays intact as a result.

Add a test at the end of t/t0008-ignores.sh to exhibit this behavior.

Signed-off-by: Pasha Bolokhov &lt;pasha.bolokhov@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'cb/aix'</title>
<updated>2014-04-03T19:38:38Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-03T19:38:38Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=8ba87adad637b402fb309754e213ec40b74e4f18'/>
<id>urn:sha1:8ba87adad637b402fb309754e213ec40b74e4f18</id>
<content type='text'>
* cb/aix:
  tests: don't rely on strerror text when testing rmdir failure
  dir.c: make git_fnmatch() not inline
</content>
</entry>
<entry>
<title>dir.c: make git_fnmatch() not inline</title>
<updated>2014-03-31T18:50:15Z</updated>
<author>
<name>Charles Bailey</name>
<email>cbailey32@bloomberg.net</email>
</author>
<published>2014-03-29T15:39:00Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=1f26ce615ab4b74cdd018e5d543138c2d188c3d8'/>
<id>urn:sha1:1f26ce615ab4b74cdd018e5d543138c2d188c3d8</id>
<content type='text'>
Now that it calls a static inline function, it cannot be an inline
definition with external linkage. Remove inline and make it an
external definition.

Signed-off-by: Charles Bailey &lt;cbailey32@bloomberg.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'dd/use-alloc-grow'</title>
<updated>2014-03-18T20:50:21Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-03-18T20:50:21Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=fe9122a35213827348c521a16ffd0cf2652c4ac5'/>
<id>urn:sha1:fe9122a35213827348c521a16ffd0cf2652c4ac5</id>
<content type='text'>
Replace open-coded reallocation with ALLOC_GROW() macro.

* dd/use-alloc-grow:
  sha1_file.c: use ALLOC_GROW() in pretend_sha1_file()
  read-cache.c: use ALLOC_GROW() in add_index_entry()
  builtin/mktree.c: use ALLOC_GROW() in append_to_tree()
  attr.c: use ALLOC_GROW() in handle_attr_line()
  dir.c: use ALLOC_GROW() in create_simplify()
  reflog-walk.c: use ALLOC_GROW()
  replace_object.c: use ALLOC_GROW() in register_replace_object()
  patch-ids.c: use ALLOC_GROW() in add_commit()
  diffcore-rename.c: use ALLOC_GROW()
  diff.c: use ALLOC_GROW()
  commit.c: use ALLOC_GROW() in register_commit_graft()
  cache-tree.c: use ALLOC_GROW() in find_subtree()
  bundle.c: use ALLOC_GROW() in add_to_ref_list()
  builtin/pack-objects.c: use ALLOC_GROW() in check_pbase_path()
</content>
</entry>
<entry>
<title>Merge branch 'nd/no-more-fnmatch'</title>
<updated>2014-03-14T21:25:31Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-03-14T21:25:31Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=650c90a18506ce05e2be349d83fe1cef3dc7f8cb'/>
<id>urn:sha1:650c90a18506ce05e2be349d83fe1cef3dc7f8cb</id>
<content type='text'>
We started using wildmatch() in place of fnmatch(3); complete the
process and stop using fnmatch(3).

* nd/no-more-fnmatch:
  actually remove compat fnmatch source code
  stop using fnmatch (either native or compat)
  Revert "test-wildmatch: add "perf" command to compare wildmatch and fnmatch"
  use wildmatch() directly without fnmatch() wrapper
</content>
</entry>
<entry>
<title>Merge branch 'nd/gitignore-trailing-whitespace'</title>
<updated>2014-03-14T21:23:37Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-03-14T21:23:37Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=dfcd354cdf38ff119c05e6e5f0e434bbc163ba41'/>
<id>urn:sha1:dfcd354cdf38ff119c05e6e5f0e434bbc163ba41</id>
<content type='text'>
Trailing whitespaces in .gitignore files, unless they are quoted for
fnmatch(3), e.g. "path\ ", are warned and ignored.

Strictly speaking, this is a backward incompatible change, but very
unlikely to bite any sane user and adjusting should be obvious and
easy.

* nd/gitignore-trailing-whitespace:
  t0008: skip trailing space test on Windows
  dir: ignore trailing spaces in exclude patterns
  dir: warn about trailing spaces in exclude patterns
</content>
</entry>
<entry>
<title>dir.c: use ALLOC_GROW() in create_simplify()</title>
<updated>2014-03-03T22:54:29Z</updated>
<author>
<name>Dmitry S. Dolzhenko</name>
<email>dmitrys.dolzhenko@yandex.ru</email>
</author>
<published>2014-03-03T22:31:58Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=9af49f822bc1e4f00b356e052f666ae7714a0399'/>
<id>urn:sha1:9af49f822bc1e4f00b356e052f666ae7714a0399</id>
<content type='text'>
Signed-off-by: Dmitry S. Dolzhenko &lt;dmitrys.dolzhenko@yandex.ru&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>pathspec: pass directory indicator to match_pathspec_item()</title>
<updated>2014-02-24T22:37:19Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2014-01-24T13:40:33Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ae8d0824217bdf97c69ead49568cd03fc140627b'/>
<id>urn:sha1:ae8d0824217bdf97c69ead49568cd03fc140627b</id>
<content type='text'>
This patch activates the DO_MATCH_DIRECTORY code in m_p_i(), which
makes "git diff HEAD submodule/" and "git diff HEAD submodule" produce
the same output. Previously only the version without trailing slash
returns the difference (if any).

That's the effect of new ce_path_match(). dir_path_match() is not
executed by the new tests. And it should not introduce regressions.

Previously if path "dir/" is passed in with pathspec "dir/", they
obviously match. With new dir_path_match(), the path becomes
_directory_ "dir" vs pathspec "dir/", which is not executed by the old
code path in m_p_i(). The new code path is executed and produces the
same result.

The other case is pathspec "dir" and path "dir/" is now turned to
"dir" (with DO_MATCH_DIRECTORY). Still the same result before or after
the patch.

So why change? Because of the next patch about clean.c.

Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>match_pathspec: match pathspec "foo/" against directory "foo"</title>
<updated>2014-02-24T22:37:19Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2014-01-24T13:40:32Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=68690fdd0b78762eb6387d7a437b588d15b6cf47'/>
<id>urn:sha1:68690fdd0b78762eb6387d7a437b588d15b6cf47</id>
<content type='text'>
Currently we do support matching pathspec "foo/" against directory
"foo". That is because match_pathspec() has no way to tell "foo" is a
directory and matching "foo/" against _file_ "foo" is wrong.

The callers can now tell match_pathspec if "foo" is a directory, we
could make an exception for this case. Code is not executed though
because no callers pass the flag yet.

Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>dir.c: prepare match_pathspec_item for taking more flags</title>
<updated>2014-02-24T22:37:19Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2014-01-24T13:40:31Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=42b0874a7ef66f9bd561c66df6e989f58d0393b9'/>
<id>urn:sha1:42b0874a7ef66f9bd561c66df6e989f58d0393b9</id>
<content type='text'>
Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
