More remote rename fixes

Win32: Fix object::cache::threadmania test on x64

Win32 test fixes

This was a bad idea. Don't free except in the free function.

A symref inside the namespace gets renamed, we should make it point to
the target's new name.

This is for the origin/HEAD -> origin/master type of situations.

There is no reason why we need to use a callback here. A string array
fits better with the usage, as this is not an event and we don't need
anything from the user.

We must make sure that the name pointer remains valid, so make sure to
allocate the new one before freeing the old one and swap them so the
user never sees an invalid pointer.

We don't allow renames of anonymous remotes, so there's no need to
handle them.

A remote is always associated with a repository, so there's no need to
check for that.

Remote rename fixes

Update AUTHORS

Add me. :)

Tighten up which references we consider for renaming so we don't try to
rename unrelated ones and end up with unexplained references.

If there is a reference on the target namespace, git overwrites it, so
let's do the same.

When there is a reference in the target namespace, we should overwrite
it. Instead it gets a different name under the current code.

clone: re-use the local transport's path resolution

Concurrent ref iterator access

Whe already worked out the kinks with the function used in the local
transport. Expose it and make use of it in the local clone method
instead of trying to work it out again.

Remote: Set an error when a remote cannot be found.

Documentation formatting fixes

The documentation has shown this as a single enum for a long time. These
should in fact be two enums. One with the bits for the flags and another
with the bits for the extended flags.

These are some issues I found while playing around with the new parser
for docurium.

This lets us work without worrying about what's happening but work on a
snapshot.

If we remove a reference while we're iterating through the packed refs,
the position in the iterator will be off.

remote: build up the list of refs to remove

When removing the remote-tracking branches, build up the list and remove
in two steps, working around an issue with the iterator. Removing while
we're iterating over the refs can cause us to miss references.

Minor fixes to binary diff code

Inside `git_remote_load`, the calls to `get_optional_config` use
`giterr_clear` to unset any errors that are set due to missing config
keys. If neither a fetch nor a push url config was found for a remote,
we should set an error again.

Ignore core.safecrlf=warn until we have a warn infrastructure

Respect core.precomposeunicode = true in more cases.

This adds another assertion to ensure that the reference name inside
the git_reference struct returned by `git_branch_create` is returned as
precomposed if `core.precomposeunicode` is enabled.

This tests that decomposed branch names are correctly precomposed when
passed to `git_branch_move` and `core.precomposeunicode` is enabled.