notes: slight documentation enhancements

Fix compilation warning

tests: fix tree walking test

Return -1 to stop the iteration instead of not-0

Merge pull request #873 from carlosmn/tree-walk

git_tree_walk callback return value semantic does not match documentation

Merge pull request #875 from arrbee/fix-message-prettify-length-check

Fix message prettify length check

Make git_message_prettify return bytes written

If you want to be absolutely safe with git_message_prettify, you
can now pass a NULL pointer for the buffer and get back the number
of bytes that would be copied into the buffer.

This means that an error is a non-negative return code and a
success will be greater than zero from this function.

Merge pull request #871 from joshtriplett/fix-note_foreach-docs

git_note_foreach: Fix documentation for notes_ref parameter

Merge pull request #870 from joshtriplett/fix-note_create-docs

git_note_oid: Fix the documentation to reference parameters using the correct names

Merge pull request #872 from joshtriplett/fix-note_remove-docs

git_note_create: Copyediting on documentation for the oid parameter

Merge pull request #867 from joshtriplett/fix-array-size-for-git_config_get_mapped

Fix incorrect array size in example for git_config_get_mapped

Merge pull request #866 from arrbee/fix-config-file-parsing

Config file parser includes = in name if no space around it

tree: allow the user to skip an entry or cancel the walk

Returning a negative cancels the walk, and returning a positive one
causes us to skip an entry, which was previously done by a negative
value.

This allows us to stay consistent with the rest of the functions that
take a callback and keeps the skipping functionality.

tree: bring back the documented behaviour for a walk

However, there should be a way to cancel the walk and another to skip
the entry.

Clean up code

Okay, this is probably cleaner and it is also less net change
from the original version

git_note_remove: Copyediting on documentation for the oid parameter

Check prettify message output buffer after cleanup

This makes the message prettify buffer length check accurate.

Fix config parser boundary logic

The config file parser was not working right if there was no
whitespace between the value name and the equals sign.  This
fixes that.

git_note_foreach: Fix documentation for notes_ref parameter

git_note_oid: Fix the documentation to reference parameters using the correct names

Fix incorrect array size in example for git_config_get_mapped

In the documentation for git_config_get_mapped, the sample mapping
array uses [3] but has 4 entries.  Fix by dropping the size entirely and
letting the compiler figure it out.

Merge pull request #863 from joshtriplett/export-git_attr_value

Export git_attr_value

Export git_attr_value

Commit 0c9eacf3d2c83256736a5bb2a240e73afd13d55f introduced the function
git_attr_value and switched the GIT_ATTR_* macros to use it, but
attempting to use that function leads to a linker error (undefined
reference to `git_attr_value').  Export git_attr_value so programs can
actually call it.

sha1: add missing header guards

oid: Explicitly include `oid.h` for the inlined CMP

Merge pull request #861 from josh/parse-chomped-oid

Parse ref oids without trailing newline

Merge remote-tracking branch 'arrbee/rtrim-loose-refs' into parse-chomped-oid

Revert implementation changes

Ignore ref oid terminator

Test trailing space after ref oid

trim whitespace when parsing loose refs

Parse ref oids without trailing newline

Fix iterator check and return value

There is a little cleanup necessary from PR #843.  Since the
new callbacks return `GIT_EUSER` we have to be a little careful
about return values when they are used internally to the library.

Also, callbacks should be checked for non-zero return values,
not just less than zero.

test: Open ODB on each test suite

remotes: Proper return for `git_remote_ls`

Merge remote-tracking branch 'arrbee/tree-walk-fixes' into development

Conflicts:
src/notes.c
src/transports/git.c
src/transports/http.c
src/transports/local.c
tests-clar/odb/foreach.c

remote: add missing include git2/remote.h

Otherwise we get an incomplete type error, since git_remote_callbacks
isn't declared yet.

Add new iteration behavior to git_tree_walk

Missed this one, ironically enough.

Update iterators for consistency across library

This updates all the `foreach()` type functions across the library
that take callbacks from the user to have a consistent behavior.
The rules are:

* A callback terminates the loop by returning any non-zero value
* Once the callback returns non-zero, it will not be called again
  (i.e. the loop stops all iteration regardless of state)
* If the callback returns non-zero, the parent fn returns GIT_EUSER
* Although the parent returns GIT_EUSER, no error will be set in
  the library and `giterr_last()` will return NULL if called.

This commit makes those changes across the library and adds tests
for most of the iteration APIs to make sure that they follow the
above rules.

filebuf: Check the return value for `close`

win32: set errno to ENOENT or ENOTDIR when appropriate in do_lstat

Merge remote-tracking branch 'nulltoken/topic/amd64-compat' into development

Conflicts:
src/netops.c
src/netops.h
src/oid.c

Merge pull request #850 from libgit2/attr-export

attr: Do not export variables externally

repository: Indentation

Merge pull request #848 from carlosmn/pending-message

repository: add a getter and remove function for git's prepared message

attr: Do not export variables externally

Fixes #824

Exporting variables in a dynamic library is a PITA. Let's keep
these values internally and wrap them through a helper method.

This doesn't break the external API. @arrbee, aren't you glad I turned
the `GIT_ATTR_` macros into function macros? :sparkles:

Merge pull request #849 from scunz/git_caps

Add function to query for compile time settings.

Add function to query for compile time settings.

repository: add a getter and remove function for git's prepared message

The 'git revert/cherry-pick/merge -n' commands leave .git/MERGE_MSG
behind so that git-commit can find it. As we don't yet support these
operations, users who are shelling out to let git perform these
operations haven't had a convenient way to get this message.

These functions allow the user to retrieve the message and remove it
when she's created the commit.

Merge pull request #826 from carlosmn/config-find-error

git_config_find_* does not set a git error

Merge pull request #837 from carlosmn/travis

travis: be more idiomatic with the environment

Merge pull request #833 from carlosmn/odb-one

odb: allow creating an ODB backend from a packfile index

Merge pull request #840 from carlosmn/remote-unify

Unify the transport code

transport: rename encrypt to use_ssl

SSL isn't the only way that a transport can be encrypted. The new name
will make it easier to merge the SSH support.

fetch: remove timeout code

ssl: use the callback instead of ifs to determine how to get data

Using the callbacks makes it clearer and reduces the amount of #ifdefs
and ifs and we need in the code.

examples: fix warnings in network/

transport: store the refs in a common area

Instad of each transport having its own function and logic to get to
its refs, store them directly in transport.

Leverage the new gitno_buffer to make the parsing and storing of the
refs use common code and get rid of the git_protocol struct.

remote: use the same code to control git and http

This allows us to add capabilitites to both at the same time, keeps
them in sync and removes a lot of code.

gitno_buffer now uses a callback to fill its buffer, allowing us to
use the same interface for git and http (which uses callbacks).

network: implement multi_ack for the git transport

remote: start moving the protocol to a common area

For the transition, http is going to keep its own logic until the
git/common code catches up with the implied multi_ack that http
has. This also has the side-effect of making the code cleaner and more
correct regardingt he protocol.

Merge pull request #847 from schu/inline-oid-cmp

git_oid_cmp: inline memcmp by hand to optimize

git_oid_cmp: inline memcmp by hand to optimize

git.git uses an inlined hashcmp function instead of memcmp, since it
performes much better when comparing hashes (most hashes compared
diverge within the first byte).

Measurements and rationale for the curious reader:
http://thread.gmane.org/gmane.comp.version-control.git/172286

Fix -Wuninitialized warning

Merge pull request #839 from nulltoken/topic/odb-foreach-documentation

odb: add some documentation to the foreach() test

Merge pull request #834 from carlosmn/network-callbacks

Add a struct for network callbacks

branch: Add `repository` argument to `create`

Yes, we can get the repository from the owner of the object, but having
it marked explicitly makes the API more consistent.

Merge remote-tracking branch 'nulltoken/topic/branch-rework' into development

reflog: Rename `entry_drop` to `drop`

Merge remote-tracking branch 'nulltoken/topic/reflog-delete' into development

remote: fix C99-ism

Merge pull request #838 from scunz/remote_push_url

Add support for push-urls

Fix git_tree_walk to return user error

This makes sure that an error code returned by the callback function
of `git_tree_walk` will stop the iteration and get propagated back
to the caller verbatim.

Also, this adds a minor helper function `git_tree_entry_byoid` that
searches a `git_tree` for an entry with the given OID.  This isn't
a fast function, but it's easier than writing the loop yourself as
an external user of the library.

Remotes: Use correct url in git_remote_connect

Remotes: Save a cleaned pushurl (by deleting it from the config)

Tests: Test remote's pushurl

Remotes: Setter for url+pushurl; Getter for pushurl

Remotes: Load/Save for fetch.foo.pushurl

Remove unneccessary string transformation

tests-clar/status: fix missing-prototype warning

odb: add some documentation to the foreach() test

reflog: keep the reflog name in sync with the reference name

reflog: prevent git_reflog_append() from persisting the reflog back to disk

reflog: prevent git_reflog_read() from chocking when no log exists yet

reflog: introduce git_reflog_write()

reflog: rename git_reflog_write() to git_reflog_append()

reflog: introduce git_reflog_entry_drop()

reflog: add GIT_OID_HEX_ZERO constant

tests: reorganize reflog tests

Merge pull request #799 from yorah/fix/issue-787

Fix/issue 787

travis: be more idiomatic with the environment

Instead of putting the compilers in CC, use the travis configuration
to specify them.

Also ask it to send reports to the IRC channel.

portability: Improve x86/amd64 compatibility

branch: enforce git_branch_delete() parameter checking

refs: drop git_reference_remote_tracking_from_branch()

branch: introduce git_branch_tracking()

branch: make git_branch_move() reference based

branch: slight git_branch_create() doc improvement

branch: introduce git_reference_is_branch()

branch: introduce git_branch_lookup()

branch: change git_branch_create() to make it return a reference

branch: remove useless header