Simplify signature parsing
signature: Shut up MSVC, you silly goose
signature: Small cleanup
Merge pull request #1348 from libgit2/signatures-2

Simplify signature parsing
Store treebuilder length separately from entries vec

The treebuilder entries vector flags removed items which means
we can't rely on the entries vector length to accurately get the
number of entries.  This adds an entrycount value and maintains it
while updating the treebuilder entries.
Some code cleanups in tree.c

This replaces most of the explicit vector iteration with calls
to git_vector_foreach, adds in some git__free and giterr_clear
calls to clean up during some error paths, and a couple of
other code simplifications.
Merge pull request #1350 from arrbee/fix-1292

Add explicit entrycount to tree builder
Add more treebuilder tests

The recent changes with git_treebuilder_entrycount point out that
the test coverage for git_treebuilder_remove and
git_treebuilder_entrycount is completely absent.  This adds tests.
Merge pull request #1351 from arrbee/moar-treebuilder-tests

Add more treebuilder tests