Clarify parsing issues and errors

Streamline url-parsing logic.

Merge pull request #1916 from libgit2/simplify-examples

Fix examples to make the important stuff more obvious

Merge pull request #1933 from libgit2/vmg/gcc-warnings

Warnings for Windows x64 (MSVC) and GCC on Linux

clar: Fix warnings in GCC/Linux

iconv: Do not fake an API when iconv is not available

checkout: Remove unused vector

array: Wrap `array_alloc` as a single statement

pack: `__object_header` always returns unsigned values

Fix warning on win64

Merge pull request #1918 from libgit2/cmn/indexer-naming

indexer: remove the stream infix

config_file: Style fixes

Merge remote-tracking branch 'drodriguez/fix-remote-save' into development

Fix typos

Merge pull request #1928 from libgit2/parse-bad-urls

Improve bad URL handling

A few formatting changes for rocco

I'm not too happy about manually inserting < and > but those get
output as html tags otherwise.

Set new multivar values using unmatcheable regexp.

Seems that regexp in Mac OS X and Linux were behaving
differently: while in OS X the empty string didn't
match any value, in Linux it was matching all of them,
so the the second fetch refspec was overwritting the
first one, instead of creating a new one.

Using an unmatcheable regular expression solves the
problem (and seems to be portable).

Fix memory leaks.

Fix saving remotes with several fetch/push ref specs.

At some moment git_config_delete_entry lost the ability to delete one entry of
a multivar configuration. The moment you had more than one fetch or push
ref spec for a remote you will not be able to save that remote anymore. The
changes in network::remote::remotes::save show that problem.

I needed to create a new git_config_delete_multivar because I was not able to
remove one or several entries of a multivar config with the current API.
Several tries modifying how git_config_set_multivar(..., NULL) behaved were
not successful.

git_config_delete_multivar is very similar to git_config_set_multivar, and
delegates into config_delete_multivar of config_file. This function search
for the cvar_t that will be deleted, storing them in a temporal array, and
rebuilding the linked list. After calling config_write to delete the entries,
the cvar_t stored in the temporal array are freed.

There is a little fix in config_write, it avoids an infinite loop when using
a regular expression (case for the multivars). This error was found by the
test network::remote::remotes::tagopt.

Update examples/README.md

Format comments for use with docco

Prevent another segfault from bad URL

Test another bad URL

Prevent segfault with a badly-formed URL

Add tests for badly-formed URLs

Merge pull request #1924 from linquize/gmtime

Use gmtime() instead of gmtime_t()

Use gmtime() instead of gmtime_t()

The latter is not available on Windows

Test more kinds of bad url

Initialize variables

Clean up showindex sample

Reorganize rev-parse example

rev-list.c example: use common utils, reorganize

Merge pull request #1919 from libgit2/cmn/multi-ack-detailed

protocol: basic support for multi_ack_detailed

merge: any non-zero return from the user is an error

This fixes #1703.

protocol: basic support for multi_ack_detailed

This tells the server that we speak it, but we don't make use of its
extra information to determine if there's a better place to stop
negotiating.

In a somewhat-related change, reorder the capabilities so we ask for
them in the same order as git does.

Also take this opportunity to factor out a fairly-indented portion of
the negotiation logic.

cat-file.c example: deploy helpers, reorg

add.c: proper frontmatter

Remove leftover function declaration

indexer: remove the stream infix

It was there to keep it apart from the one which read in from a file on
disk. This other indexer does not exist anymore, so there is no need for
anything other than git_indexer to refer to it.

While here, rename _add() function to _append() and _finalize() to
_commit(). The former change is cosmetic, while the latter avoids
talking about "finalizing", which OO languages use to mean something
completely different.

init.c example: deploy more helpers

add.c example: deploy helpers, reorg

pack-objects: Depth can be negative

init example: deploy helpers, reorg

Extract common example helpers and reorg examples

This reorganizes a few of the examples so that the main function
comes first with the argument parsing extracted into a helper
that can come at the end of the file (so the example focuses more
on the use of libgit2 instead of command line support).  This also
creates a shared examples/common.[ch] so that useful helper funcs
can be shared across examples instead of repeated.

Merge pull request #1891 from libgit2/cmn/fix-thin-packs

Add support for thin packs

Merge pull request #1912 from ethomson/push_err_messages

Test for failure, but don't test the error message

Merge pull request #1904 from libgit2/cmn/ssh-naming

Rename the ssh credentials

Merge pull request #1802 from libgit2/cmn/reflog-backend

Make reflog part of refdb

Merge pull request #1915 from zcbenz/common-h-before-config-h

The "common.h" should be included before "config.h".

The "common.h" should be included before "config.h".

When building libgit2 for ia32 architecture on a x64 machine, including
"config.h" without a "common.h" would result the following error:
C:\Program Files\Microsoft SDKs\Windows\v7.1\include\winbase.h(2288): error C2373: 'InterlockedIncrement' : redefinition; different type modifiers [C:\cygwin\home\zcbenz\codes\git-utils\build\libgit2.vcxproj]
C:\Program Files\Microsoft SDKs\Windows\v7.1\include\winbase.h(2295): error C2373: 'InterlockedDecrement' : redefinition; different type modifiers [C:\cygwin\home\zcbenz\codes\git-utils\build\libgit2.vcxproj]
C:\Program Files\Microsoft SDKs\Windows\v7.1\include\winbase.h(2303): error C2373: 'InterlockedExchange' : redefinition; different type modifiers [C:\cygwin\home\zcbenz\codes\git-utils\build\libgit2.vcxproj]
C:\Program Files\Microsoft SDKs\Windows\v7.1\include\winbase.h(2314): error C2373: 'InterlockedExchangeAdd' : redefinition; different type modifiers [C:\cygwin\home\zcbenz\codes\git-utils\build\libgit2.vcxproj]

Merge pull request #1913 from nvloff/remove_fedora_spec

remove rpm spec file and readme

remove rpm spec

Fedora provides a rpm package for libgit2 -
https://apps.fedoraproject.org/packages/libgit2.

Test for failure, but don't test the error message

Merge pull request #1910 from ethomson/disable_winhttp

Allow -DWINHTTP=OFF to disable WinHTTP

Merge pull request #1911 from ethomson/http_headers

Use two calls to set two headers

Use two calls to set two headers

Allow -DWINHTTP=OFF to disable WinHTTP

Merge pull request #1907 from nikai3d/patch-1

fix typos in README.md

fix typos in README.md

Really fix the pc file

It turns out that variables have function scope by default. Let's
really set -liconv and add a few libraries that were forgotten in
the previous commit.

We also need to special-case OSX, as they ship zlib but do not provide
a pkg-config file for it.

Merge pull request #1905 from libgit2/cmn/list-deps

Be explicit about dependencies

Merge pull request #1906 from libgit2/cmn/net-cb-errors

transport: let the progress output return an error

transport: let the progress output return an error

There are any number of issues that can come up in the progress
callback, and we should let the user cancel at that point as well.

indexer: include the delta stats

The user is unable to derive the number of deltas in the pack, as that
would require them to capture the stats exactly in the moment between
download and final processing, which is abstracted away in the fetch.

Capture these numbers for the user and expose them in the progress
struct. The clone and fetch examples now also present this information
to the user.

examples: show used local objects in fetch

Show how many local objects were used to fix the thin pack in our fetch
example.

README: add a note about the optional dependencies

Set the correct dependencies in the pkg-config file

When linking statically, the including project needs to know what the
current library build depends on so they can link to it. Store this
information in the pkg-config file.

While here, remove claims that users need to link to zlib or libcrypto.

Rename the ssh credentials

The names from libssh2 are somewhat obtuse for us. We can simplify the
usual key/passphrase credential's name, as well as make clearer what the
custom signature function is.

Merge pull request #1897 from libgit2/split-patch-from-diff

RFC: Proposed reworking of diff APIs

Fix typo

Implement `git_odb_object_dup`

Implement patience and minimal diff flags

It seems that to implement these options, we just have to pass
the appropriate flags through to the libxdiff code taken from
core git.  So let's do it (and add a test).

Fix warnings for win64

Create git_diff_line and extend git_diff_hunk

Instead of having functions with so very many parameters to pass
hunk and line data, this takes the existing git_diff_hunk struct
and extends it with more hunk data, plus adds a git_diff_line.
Those structs are used to pass back hunk and line data instead of
the old APIs that took tons of parameters.

Some work that was previously only being done for git_diff_patch
creation (scanning the diff content for exact line counts) is now
done for all callbacks, but the performance difference should not
be noticable.

Merge pull request #1903 from ethomson/ssh

Allowed credential types should be a bitfield

Allowed credential types should be a bitfield

Tweak to git_diff_delta structure for nfiles

While the base git_diff_delta structure always contains two files,
when we introduce conflict data, it will be helpful to have an
indicator when an additional file is involved.

Merge pull request #1900 from haneefmubarak/patch-1

license clarification + formatting all over

slightly clarified the "license" portion of `README.md" + formatting all over

fixed typo `can can` -> `can`

slightly clarified the "license" portion of `README.md" + formatting all over

added the second change suggested by @nulltoken (with two extra words :) )

slightly clarified the "license" portion of `README.md" + formatting all over

There are commercial open source applications, hence proprietary seems to be a better term.

Also:
 - Trimmed header `=`'s to match text length
 - made libgit2 stand out everywhere
 - pretty printing links
 - irc link for those with the correct setup

Merge pull request #1702 from ethomson/checkout_merge

Checkout merge

Move path prefixed help to path.h

Move functions in checkout_conflicts to checkout.c

It seemed exceptionally silly to have a split there
where no split needed to be.

Two-step conflict checkout (load / perform)

Move conflict handling into two steps: load the conflicts and
then apply the conflicts.  This is more compatible with the
existing checkout implementation and makes progress reporting
more sane.

Honor UPDATE_ONLY bit when checking out conflicts

Fix warning, fix memory leak

Don't overwrite ~ files checking out conflicts

If a D/F conflict or rename 2->1 conflict occurs,
we write the file sides as filename~branchname.  If
a file with that name already exists in the working
directory, write as filename~branchname_0 instead.
(Incrementing 0 until a unique filename is found.)

Get rid of some quick hacks

checkout rename conflicts

rename conflict tests for checkout conflicts, don't suffix filenames
when checking out with USE_OURS or USE_THEIRS

Prevent checkout_tree when conflicts exist, clear NAME on checkout tree

Prevent checkout tree when unresolved changes exist (unless FORCE flag
is specified).  Clear NAME table when checking out, to avoid
checkout_conflicts from attempting to manipulate it.  Ensure that NAME
is also cleared at reset.

checkout (from index) can write conflicts

tests for checkout index with conflicts

Diff API cleanup

This lays groundwork for separating formatting options from diff
creation options.  This groups the formatting flags separately
from the diff list creation flags and reorders the options.  This
also tweaks some APIs to further separate code that uses patches
from code that just looks at git_diffs.

Rename diff objects and split patch.h

This makes no functional change to diff but renames a couple of
the objects and splits the new git_patch (formerly git_diff_patch)
into a new header file.

Merge pull request #1562 from libgit2/cmn/refs-namespace-lookup

Provide the user with a more useful error code when a looking up a reference which name points to a namepace

indexer: clearer stats for thin packs

Don't increase the number of total objects, as it can produce
suprising progress output. The only addition compared to pre-thin is
the addition of local_objects to allow an output similar to git's
"completed with %d local objects".

futils: return GIT_ENOTFOUND when trying to read a directory

This lets the reference code return not-found when the user asks to
look up a reference when in fact they pass a namespace.

Merge pull request #1898 from arthurschreiber/patch-2

Fix a typo in the rpm build instructions.

Fix a typo in the rpm build instructions.