refs: handle ALLOW_ONELEVEL normalization with leading slash

A leading slash confuses the name normalization code when the flags
include ALLOW_ONELEVEL. Catch this case in particular to avoid
triggering an assertion in the uppercase check which expects us not to
pass it an empty string.

The existing tests don't catch this as they simply use the NORMAL
flag.

This fixes #1300.

Merge pull request #1297 from arrbee/diff-patch-line-totals

Add helper API for diff line stats from patch

Let people know that the ML is deprecated

Free buffer at end of test

Add helper for diff line stats

This adds a `git_diff_patch_line_stats()` API that gets the total
number of adds, deletes, and context lines in a patch.  This will
make it a little easier to emulate `git diff --stat` and the like.

Right now, this relies on generating the `git_diff_patch` object,
which is a pretty heavyweight way to get stat information.  At
some future point, it would probably be nice to be able to get
this information without allocating the entire `git_diff_patch`,
but that's a much larger project.

Merge pull request #1296 from arrbee/stricter-config-name-checks

Stricter config entry name validation

Merge pull request #1295 from carlosmn/obsd

Fix p_realpath on OpenBSD

Merge pull request #1285 from phkelley/vector

Vector improvements and their fallout

Now with no multiply

Test config name validation

This is @nulltoken's work to test various invalid config section
and key names and make sure we are validating properly.

Implement config key validation rules

This is a new implementation of core git's config key checking
rules that prevents non-alphanumeric characters (and '-') for
the top-level section and key names inside of config files.

This also validates the target section name when renaming
sections.

Test buf join with NULL behavior explicitly

Fix p_realpath on OpenBSD

OpenBSD's realpath(3) doesn't require the last part of the path to
exist. Override p_realpath in this OS to bring it in line with the
library's assumptions.

Merge pull request #1289 from jwiegley/development

Added git_treebuilder_entrycount

Added git_treebuilder_entrycount

Conflicts:
src/tree.c

Fix 2 bugs in online::push tests.

- Fix stack corruption introduced in 9bccf33c due to passing pointer to
local variable _cred_acquire_called.
- Fix strcmp in do_verify_push_status when expected or actual push_status
  is NULL

Merge pull request #1283 from lznuaa/master

Fix local repository clone failure

Vector improvements and their fallout

Fix fail clone local repository because can't found object

avoid use object which is already free

Signed-off-by: Frank Li <lznuaa@gmail.com>

Fix a mutex leak in pack.c

Expand valgrind suppressions even more for libssl, libcrypto

Merge pull request #1278 from sba1/cl-assert-equal-s

Use cl_assert_equal_s() instead of strcmp().

Merge pull request #1281 from phkelley/valgrind

Improve valgrind suppressions

Improve valgrind suppressions

Fix 3 memory leaks

Merge pull request #1279 from carlosmn/config-trailing-backslash

config: support trailing backslashes

Merge pull request #1280 from nulltoken/fix/clone-segfault

clone: Prevent segfault upon faulted remote creation

config: support trailing backslashes

Check whether the backslash at the end of the line is being escaped or
not so as not to consider it a continuation marker when it's e.g. a
Windows-style path.

tests-clar: ifdef GIT_WIN32 win helper functions

clone: Prevent segfault upon faulted remote creation

Use cl_assert_equal_s() instead of strcmp().

Replaced all cl_assert(!strcmp()) or semantically equivalent forms
by cl_assert_equal_s().

Merge pull request #1277 from sba1/branch-name

Add git_branch_name()

Added git_branch_name().

This is a convenience function to get the branch name of a given
ref. The returned branch name is compatible with the name that can
be supplied e.g. to git_branch_lookup(). That is, the prefixes
"refs/heads" or "refs/remotes" are omitted.

Also added a new test for testing the new function.

index: Speed up loading a tree into the index

The index is empty; repeated tree entries cannot collide.

cc github/gitrpc#83

Leak cleanup in push tests

Merge pull request #1250 from jamill/push_update_tips

Update remote tips on push

Merge pull request #1275 from ethomson/examples_windows

update examples to work on windows

update examples to work on windows

opts: Some basic tests

opts: Add getters too

Merge pull request #1273 from sba1/example-diff-fix

Don't clear the opt instance in the diff example.

Don't clear the opt instance in the diff example.

The version field is overwritten otherwise. The opt instance is
already initialized properly.

Merge pull request #1271 from libgit2/global-settings

Global options setter

Global options setter

Merge pull request #1270 from libgit2/packed-peeled-objects-fix

Allow peeled references without trailing newline at end of file

Seperate out a new test that verifies packed-refs with no trailing newline

as per @vmg's request

Allow peeled references without trailing newline at end of file

Also ammends one of the tag tests to make sure it's working.

Merge pull request #1269 from arrbee/tree-iterator-case-sensitivity-bug

Fix case sensitivity bug with tree iterators

Fix case sensitivity bug with tree iterators

With the new code to make tree iterators support ignore_case,
there is a bug in setting the start entry for range bounded
iterators where memcmp was being used instead of strncasecmp.
This fixes that and expands the tree iterator test to cover
the cases that were broken.

Revert "Handle packed peeled objects without trailing newlines"

This reverts commit 28b1cdf3a1bdcd37cf9d550c92b8c19b1782ea6b.

//cc #1262 #1267

Merge pull request #1268 from phkelley/development

A simple perf optimization in pack-objects.c

Update remote tips on push

Fix gen_pktline format specifier for Win32

A simple perf optimization in pack-objects.c

Merge pull request #1267 from libgit2/no-newline

Handle packed peeled objects without trailing newlines

Handle packed peeled objects without trailing newlines

Fixes #1262

Merge pull request #1266 from arrbee/update-clar

Update clar to a80e7f30

Update clar to a80e7f30

Merge pull request #1265 from arrbee/parse-commit-time-as-uint64

Parse commit time as uint64_t to avoid overflow

Parse commit time as uint64_t to avoid overflow

The commit time is already stored as a git_time_t, but we were
parsing is as a uint32_t.  This just switches the parser to use
uint64_t which will handle dates further in the future (and adds
some tests of those future dates).

commit: don't include the LF in the header field value

When the encoding header changed to be treated as an additional
header, the EOL pointer started to point to the byte after the LF,
making the git__strndup call copy the LF into the value.

Increase the EOL pointer value after copying the data to keep the rest
of the semantics but avoid copying LF.

Fix compilation on OpenBSD

Merge pull request #1260 from phkelley/development

Fix really bad error handling in git_smart__negotiate_fetch

Fix really bad error handling in git_smart__negotiate_fetch

Merge pull request #1258 from zcbenz/fix_index_remove_test

Fix linking error caused by ddcb28a41f3774e26fc6ae0a7174a5565e4749ce.

Fix linking error caused by ddcb28a41f3774e26fc6ae0a7174a5565e4749ce.

Merge pull request #1239 from ethomson/index_remove

add an index_remove_bypath that removes conflicts

Merge pull request #1256 from arrbee/asciify-test-data

Move all non-ascii test data to raw hex

Merge pull request #1257 from ethomson/crlf_test_fix

add a git config, don't run crlf tests on non-win32

add a git config, don't run crlf tests on non-win32

Move all non-ascii test data to raw hex

This takes all of the characters in core::env and makes them use
hex sequences instead of keeping tricky character data inline in
the test.

Merge pull request #1244 from carlosmn/pack-evict

A comparison of eviction algorithms for the delta base cache

Merge pull request #1247 from sba1/dont-segfault-if-transport-doesnt-support-push

Don't segfault if transport doesn't support push.

Merge pull request #1255 from arrbee/fix-signed-commit-header-parsing

Add skipping of unknown commit headers

Merge pull request #1254 from ethomson/index_filtered_size

cache should contain on-disk (filtered) file size

Test that pushs properly fail for transports that don't provide a push implementation.

Add skipping of unknown commit headers

This moves the check for the "encoding" header into a loop which
is just scanning for non-required headers at the end of a commit
header.  That loop will skip unrecognized lines (including header
continuation lines) until a terminating completely blank line is
found, and only then does it move to reading the commit message.

cache should contain on-disk (filtered) file size

Don't segfault if transport doesn't support push.

Instead, set an more informative error message.

Merge pull request #1211 from arrbee/fix-icase-status-file

Fix case insensitivity issues in git_status_file

Merge pull request #1248 from sba1/doc-fix

Some doc improvements in transport.h

Merge pull request #1245 from nulltoken/topic/git_branch_tracking_name

Introduce git_branch_tracking_name()

branch: Introduce git_branch_tracking_name()

Fix indentations

Some doc improvements

Shortcut spool and sort for empty iterator

Support case insensitive tree iterators and status

This makes tree iterators directly support case insensitivity by
using a secondary index that can be sorted by icase.  Also, this
fixes the ambiguity check in the git_status_file API to also be
case insensitive.  Lastly, this adds new test cases for case
insensitive range boundary checking for all types of iterators.

With this change, it should be possible to deprecate the spool
and sort iterator, but I haven't done that yet.

Add git_tree_entry_cmp and git_tree_entry_icmp

This adds a new external API git_tree_entry_cmp and a new internal
API git_tree_entry_icmp for sorting tree entries.  The case
insensitive one is internal only because general users should
never be seeing case-insensitively sorted trees.

Add git_path_icmp to case-insensitive path cmp

This adds git_path_icmp to complement git_path_cmp.

Add payload "_r" versions of bsearch and tsort

git__bsearch and git__tsort did not pass a payload through to the
comparison function.  This makes it impossible to implement sorted
lists where the sort order depends on external data (e.g. building
a secondary sort order for the entries in a tree).  This commit
adds git__bsearch_r and git__tsort_r versions that pass a third
parameter to the cmp function of a user payload.

Test for ignore_case ranges on workdir iterator

This adds a test that confirms that the working directory iterator
can actually correctly process ranges of files case insensitively
with proper sorting and proper boundaries.

Update iterator API with flags for ignore_case

This changes the iterator API so that flags can be passed in to
the constructor functions to control the ignore_case behavior.
At this point, the flags are not supported on tree iterators (i.e.
there is no functional change over the old API), but the API
changes are all made to accomodate this.

By the way, I went with a flags parameter because in the future
I have a couple of other ideas for iterator flags that will make
it easier to fix some diff/status/checkout bugs.

Fix err msg for ambiguous path in git_status_file

Returning GIT_EAMBIGUOUS from inside the status callback gets
overridden with GIT_EUSER.  `git_status_file` accounted for this
via the callback payload, but was allowing the error message to
be cleared.  Move the `giterr_set` call outside the callback to
where the EUSER case was being dealt with.

Minor iterator API cleanups

In preparation for further iterator changes, this cleans up a few
small things in the iterator API:

* removed the git_iterator_for_repo_index_range API
* made git_iterator_free not be inlined
* minor param name and test function name tweaks

Simplify git_diff__paired_foreach icase handling

status: Enhance git_status_file() test coverage

Merge pull request #1238 from nulltoken/fix/checkout-index

checkout: Teach checkout to cope with orphaned Head

Merge pull request #1242 from sba1/init-with-template-fix

Some fixes for external template support

pack: evict all of the pages at once

Somewhat surprisingly, this can increase the speed considerably, as we
don't bother trying to decide what to evict, and the most used entries
are quickly back into the cache.

pack: evict objects from the cache in groups of eight

This drops the cache eviction below libcrypto and zlib in the perf
output. The number has been chosen empirically.