If you are checking out NONE, then don't remove.

The checkout code used to defer removal of "blocking" files in
checkouts until the blocked item was actually being written (since
we have already checked that the removing the block is acceptable
according to the update rules).  Unfortunately, this resulted in
an intermediate index state where both the blocking and new items
were in the index which is no longer allowed.  Now we just remove
the blocking item in the first pass so it never needs to coexist.

In cases where there are typechanges, this could result in a bit
more churn of removing and recreating intermediate directories,
but I'm going to assume that is an unusual case and the churn will
not be too costly.

When three-way merging indexes, we previously changed each path
as we read them, which would lead to us adding an index entry for
'foo', then removing an index entry for 'foo/file'.  With the new
index requirements, this is not allowed.  Removing entries in the
merged index, then adding them, resolves this.  In the previous
example, we now remove 'foo/file' before adding 'foo'.

In case insensitive index mode, we would stop at a prefixed entry,
treating the provided search key length as a substring, not the
length of the string to match.

Don't strcmp a git_buf, strcmp its char *

Add `git_commit_descendant_of`.

Add built in diff drivers

Writing a sample Javascript driver pointed out some extra
whitespace handling that needed to be done in the diff driver.
This adds some tests with some sample javascript code that I
pulled off of GitHub just to see what would happen.  Also, to
clean up the userdiff test data, I did a "git gc" and packed
up the test objects.

Leftover OID -> ID changes

Buff up returning strings

MSVC doesn't like modern code neither

Make blame example compile on MSVC

A lot of the tests were checking for overflow, which we don't have
anymore, so we can remove them.

Again, we already did this internally, so simply remove the conversions.

Internally we already did everything with git_bufs, so this is just
exposing those functions with public names.

Since we now export that type, we can avoid making the user guess a
size.

Fix git_remote_dup & add missing const

Fix write_object.

Fix local push to file:// URL.

Following the rest of the series, use 'id' when refering to the value.

Same as the other commits in the series, we use 'id' when talking about
thing rather than the datatype.

In the same vein as the previous commits in this series.

Rename git_tree_entry_byoid() to _byid() as per the convention.