tests-clar: ifdef GIT_WIN32 win helper functions

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.

Altered the description of the template.

Before, it was identical to the default template making it difficult
to check, if the proper template was copied an external template
test.

Now checks in the template test whether the description file has
been properly copied.

This is a minimal effort to test whether the template really has
been used when creating an repo with external templates.

Do not use GIT_CPDIR_CHMOD flag when copying the template.

This is an intermin solution. While this essentially disables the
--shared flag feature, previously external templates did not work
at all. This change fixes the previously corrected, and since
then failing, repo_init__extended_with_template() test.

The problem is now documented in the source code comments.

Fix Travis compilation warnings

checkout: Teach checkout to cope with orphaned Head

Fix #1236

add an index_remove_bypath that removes conflicts, renamed add_from_workdir to match

Clear local error variable after invoking giterr_clear().

pack: fixes to the cache

The offset should be git_off_t, and we should check the return value
of the mutex lock function.

tests: plug leaks

indexer: properly free the packfile resources

The indexer needs to call the packfile's free function so it takes care of
freeing the caches.

We still need to close the mwf descriptor manually so we can rename the
packfile into its final name on Windows.

Added flag GIT_REPOSITORY_INIT_EXTERNAL_TEMPLATE to test_repo_init__extended_with_template().

Otherwise the template functionallity is not tested (as a TODO we
also shall test that the specified template really got copied).

Merge pull request #1222 from scunz/clone_branch

Switch to specified branch during clone

Default git_clone_options' checkout strategy to GIT_CHECKOUT_SAFE_CREATE

buf: Is this the function you were looking for?

Kill vestigal dangling-remote code

Fixes #1232

Merge pull request #1215 from phkelley/binaryunicode

Add a failing test for CRLF filters

Fix up binaryunicode test repo

Tests: Add test for check out of given branch during clone

Test: Cleanup some cleaning code

Check for binary blobs in checkout

This adds a git_buf_text_is_binary check to blobs before applying
filters when the blob data is being written to disk.

Merge pull request #1230 from arrbee/match-core-git-diff-binary-detection

Match binary file check of core git in diff

Merge pull request #1229 from arrbee/fix-diff-patch-line-numbers

Fix diff patch line number calculation

Match binary file check of core git in diff

Core git just looks for NUL bytes in files when deciding about
is-binary inside diff (although it uses a better algorithm in
checkout, when deciding if CRLF conversion should be done).
Libgit2 was using the better algorithm in both places, but that
is causing some confusion.  For now, this makes diff just look
for NUL bytes to decide if a file is binary by content in diff.

Fix diff patch line number calculation

This was just wrong.  Added a test that verifying patch line
numbers even for hunks further into a file and then fixed the
algorithm.  I needed to add a little extra state into the patch
so that I could track old and new file numbers independently,
but it should be okay.

Revert "pack: packfile_free -> git_packfile_free and use it in the indexers"

This reverts commit f289f886cb81bb570bed747053d5ebf8aba6bef7, which
makes the tests fail on Windows. Revert until we can figure out a
solution.

Merge pull request #1227 from nulltoken/topic/emergeconflict

Introduce EMERGECONFLICT

Fix MSVC Clar compilation warnings

Fix MSVC compilation warnings

checkout: Deploy EMERGECONFLICT usage

errors: Introduce EMERGECONFLICT error code