Disable caching in Clar

Merge pull request #1344 from arrbee/fix-static-analyzer-issues

Fix static analyzer issues

Clear up warnings from cppcheck

The cppcheck static analyzer generates warnings for a bunch of
places in the libgit2 code base.  All the ones fixed in this
commit are actually false positives, but I've reorganized the
code to hopefully make it easier for static analysis tools to
correctly understand the structure.  I wouldn't do this if I
felt like it was making the code harder to read or worse for
humans, but in this case, these fixes don't seem too bad and will
hopefully make it easier for better analysis tools to get at any
real issues.

Fix memory leak in p_getaddrinfo on Amiga

If gethostbyname() fails on platforms with NO_ADDRINFO, the code
leaks the struct addrinfo that was allocated.  This fixes that
(and a number of code formatting issues in that area of code in
src/posix.c).

Add rudimentary error checks and reformat comments

There were a number of functions assigning their return value to
`error` without much explanation.  I added in some rudimentary
error checking to help flesh out the example.

Also, I reformatted all of the comments down to 80 cols (and in
some cases, slightly updated the wording).

Merge pull request #1342 from ghedo/development

push: fix typo in git_push_finish() doc

push: fix typo in git_push_finish() doc

Merge pull request #1318 from nulltoken/topic/diff-tree-coverage

Topic/diff tree coverage

Merge pull request #1340 from schu/push-docs

push: improve docs on success / failure of git_push_finish

push: improve docs on success / failure of git_push_finish

Fix a leak when canceling a network operation

Fix a git_filebuf leak (fixes Win32 clone::can_cancel)

Merge pull request #1335 from phkelley/development

Improve MSVC compiler, linker flags

Change git2.rc to identify git.dll as VOS_NT_WINDOWS32

Improve MSVC compiler, linker flags

Merge pull request #1333 from phkelley/push_options

Add git_push_options, to set packbuilder parallelism

Merge pull request #1246 from arrbee/fix-force-text-for-diff-blobs

Add FORCE_TEXT check into git_diff_blobs code path

Merge pull request #1316 from ben/clone-cancel

Allow network operations to cancel

More diff tests with binary data

Reorganize FORCE_TEXT diff flag checks

Add FORCE_TEXT check into git_diff_blobs code path

`git_diff_blobs` and `git_diff_blob_to_buffer` skip the step
where we check file attributes because they don't have a filename
associated with the data. Unfortunately, this meant they were also
skipping the check for the GIT_DIFF_FORCE_TEXT option and so you
could not force a diff of an apparent binary file.  This adds the
force text check into their code path.

Merge pull request #1324 from nulltoken/topic/remote_isvalidname

Topic/remote isvalidname

remote: Introduce git_remote_is_valid_name()

Fix libgit2/libgit2sharp#318

refs: No component of a refname can end with '.lock'

remote: reorganize tests

Merge pull request #1190 from nulltoken/topic/reset-paths

reset: Allow the selective reset of pathspecs

Merge pull request #1323 from jamill/resolve_remote

Resolve a remote branch's remote

Teach refspec to transform destination reference to source reference

Teach remote branch to return its remote

Add git_push_options, to set packbuilder parallelism

Fix some incorrect MSVC #ifdef's. Fixes #1305

Fix a bug introduced in df93a681 'Merge the push...'

Fix -Wmaybe-uninitialized warning

Merge pull request #1320 from cscheid/development

removed other references to api.html

Merge pull request #1327 from phkelley/development

Merge the push report into the refs to avoid a 3rd network call

tests: fix whitespace in refs/rename.c

Merge the push report into the refs to avoid a 3rd network call

Fix Windows symlinks

Merge pull request #1249 from yorah/topic/diff-notify-unmatched-pathspecs

diff: Add a callback to notify of diffed files

Merge pull request #1325 from SHyx0rmZ/fix-windows-symlinks

Fix Windows symlinks

Moved braces to conform to code style

Replace LoadLibrary with GetModuleHandle, since kernel32 is loaded by default

As requested

Replace call to strnlen with call to strlen

Include <string.h>

Fix call to readlink

diff: add a notify callback to `git_diff__from_iterators`

The callback will be called for each file, just before the `git_delta_t` gets inserted into the diff list.

When the callback:
- returns < 0, the diff process will be aborted
- returns > 0, the delta will not be inserted into the diff list, but the diff process continues
- returns 0, the delta is inserted into the diff list, and the diff process continues

Return the matched pathspec pattern in `git_pathspec_match_path`

Instead of returning directly the pattern as the return value, I used an
out parameter, because the function also tests if the passed pathspecs
vector is empty. If yes, it considers that the path "matches", but in
that case there is no matched pattern per se.

Merge pull request #1322 from phkelley/development

No bitfields in public headers b/c packing is compiler-specific

No bitfields in public headers b/c packing is compiler-specific

tests: fix indentation in repo/message.c

tests: fix indentation in repo/init.c

tests: fix code style in threads/basic.c

Call p_readlink to determine symlink size

removed other references to api.html

Merge pull request #1319 from cscheid/development

removed obsolete reference to api.html

removed obsolete reference to api.html

Fetchhead: don't expect a tag that isn't there

Fix fetchhead tests to expect nearly-dangling

Clone: fetch all tags

Expect standard error code from internal calls

diff: Enhance tree-to-tree diff test coverage

These tests are related to issue libgit2/libgit2sharp#196

diff: refactor git_diff_tree_to_tree() tests

Add unsymlinked.git test repository

Allow all non-zero returns to cancel transfers

Enhance test coverage for transfer cancellation

Document callback-triggered cancellation

Fix MSVC compilation warnings

Fix #1308

tests: Remove useless code

reset: Introduce git_reset_default()

Allow progress callback to cancel fetch

This works by having the indexer watch the return
code of the callback, so will only take effect
on object boundaries.

reset: Enhance documentation

index: Fix indentations

index: Enhance documentation

Merge pull request #1315 from nulltoken/development

cMakeList: Prevent MSVCR1x0.dll dependency

cMakeList: Prevent MSVCR1x0.dll dependency

Deploys the libgit2/libgit2@9041250 fix to RELWITHDEBINFO
and MINSIZEREL build flavors

Fix #255

Merge pull request #1298 from ben/user-at

Handle "user@" prefix for credentials partially included in URLs

Address feedback

Merge pull request #1307 from nulltoken/fix/revparse_describe

revparse: Lookup branch before described tag

revparse: Lookup sha before branch

revparse: Lookup branch before described tag

Fix #1306

Use malloc rather than calloc

Merge pull request #1303 from csware/win32_consistent_error_encoding

Win32: Make sure error messages are consistently UTF-8 encoded

Deduplicate FormatMessage UTF-16 to UTF-8 conversion code

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Improved error handling

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Merge pull request #1299 from csware/support_local_msysgit_install

Support local msysgit installations

Merge pull request #1302 from jamill/global_config_lookup

Try harder to find global config file

Get utf8_size from WideCharToMultiByte instead of guessing it

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Win32: Make sure error messages are consistently UTF-8 encoded

W/o this a libgit2 error message could have a mixed encoding:
e.g. a filename in UTF-8 combined with a native Windows error message
encoded with the local code page.

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Try harder to find global config file

Introduce git__substrdup

Merge pull request #1286 from lznuaa/master

Fix clone fail if repo head detached

Add test case for clone head detached repo

Signed-off-by: Frank Li <lznuaa@gmail.com>
Signed-off-by: nulltoken <emeric.fermas@gmail.com>

Stick to coding style: Move up braces

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Remove double-free segfaults

WinHttp: use cred in url if provided

HTTP: use creds in url if available

Plug test leaks

Enhance url parsing to include passwords

Merge pull request #1300 from carlosmn/ref-leading-slash

Crash when resolving a ref starting with "/"

refs: handle ALLOW_ONELEVEL normalization with leading slash

A leading slash confuses the name normalization code when the flags
include ALLOW_ONELEVEL. Catch this case in particular to avoid
triggering an assertion in the uppercase check which expects us not to
pass it an empty string.

The existing tests don't catch this as they simply use the NORMAL
flag.

This fixes #1300.