test: completely ignored diff is empty

Merge branch 'pr/5853'

Merge pull request #5892 from libgit2/ethomson/memleak

tests: clean up memory leak, fail on leak for win32

diff: test ignore-blank-lines

tests: clean up refs::races zero oid test

tests: exit with error on win32 leakcheck

Merge pull request #5852 from implausible/httpclient/skip-entire-body

Fix issues with Proxy Authentication after httpclient refactor

Merge pull request #5850 from punkymaniac/comment-format

Fix documentation formatting

Merge pull request #5831 from todaysoftware/bindings/libgit2-delphi

Update README.md for additional Delphi bindings

Merge branch 'zero_oid_in_old'

Manually merging #5842

refs: test git_reference_create_matching failure for zero oid

Merge pull request #5839 from staktrace/find_similar

Add testcase

diff: Fix bug in git_diff_find_similar.

When a split src gets turned into a rename, it should
also lose the IS_RENAME_TARGET flag, so that it doesn't
get processed in a subsequent iteration as a rename target.
Doing so can cause an assertion failure because it no
longer has the SPLIT flag.

Fixes #5811.

httpclient: no proxy creds in requests if proxy is CONNECT type

httpclient: git_http_client_skip_body should drain socket of body

Merge pull request #5860 from libgit2/ethomson/buf_text

buf: remove unnecessary buf_text namespace

Merge pull request #5859 from libgit2/ethomson/filter_buf

filter: stop taking git_buf as user input

buf: bom enum is in the buf namespace

Instead of a `git_bom_t` that a `git_buf` function returns, let's keep
it `git_buf_bom_t`.

buf: remove internal `git_buf_text` namespace

The `git_buf_text` namespace is unnecessary and strange.  Remove it,
just keep the functions prefixed with `git_buf`.

filter: internal git_buf filter handling function

Introduce `git_filter_list__convert_buf` which behaves like the old
implementation of `git_filter_list__apply_data`, where it might move the
input data buffer over into the output data buffer space for efficiency.

This new implementation will do so in a more predictible way, always
freeing the given input buffer (either moving it to the output buffer or
filtering it into the output buffer first).

Convert internal users to it.

filter: deprecate git_filter_list_apply_to_data

Deprecate `git_filter_list_apply_to_data` as it takes user input as a
`git_buf`.  Users should use `git_filter_list_apply_to_buffer` instead.

filter: introduce git_filter_list_apply_to_buffer

Provide a filter application mechanism that takes a user-provided string
and length, instead of a `git_buf`.

filter: remove git_buf sharing in `git_filter_list_apply_to_data`

The API `git_filter_list_apply_to_data` shares data between its out and
in parameters to avoid unnecessarily copying it when there are no
filters to apply.  However, it does so in a manner that is potentially
confusing, leaving both `git_buf`s populated with data.  This is risky
for end-users who have to know how to deal with this.  Instead, we
remove this optimization - users who want to avoid unnecessary copies
can use the longstanding streaming API or check the filter status before
invoking the filters.

filter: deprecate git_filter_list_stream_data

`git_filter_list_stream_data` takes user input in a `git_buf`.
`git_buf` should only be used when libgit2 itself needs to allocate data
and returned to a user that they can free when they wish.  Replace it
with `git_filter_list_stream_buffer` that takes a data buffer and
length.

Merge pull request #5858 from libgit2/ethomson/memleaks

test: clean up memory leaks

test: clean up memory leaks

Merge pull request #5857 from libgit2/ethomson/longpaths

Support `core.longpaths` on Windows

win32: add file and get status on a long path

win32: ensure core.longpaths is honored for workdir paths

path: bump most Win32 unicode buffer sizes from MAX_PATH to GIT_PATH_MAX

clar: support long paths on Windows

Merge pull request #5855 from libgit2/ethomson/ci

ci: use GitHub for storing mingw-w64 build dependency

ci: use GitHub for storing mingw-w64 build dependency

Merge pull request #5823 from libgit2/ethomson/path_validation

Working directory path validation

diff:add option to ignore blank line changes

repo: validate repository paths

Ensure that a repository's path (at initialization or open time) is
valid.  On Windows systems, this means that the longest known path
beneath the repository will fit within MAX_PATH: this is a lock file for
a loose object within the repository itself.

Other paths, like a very long loose reference, may fail to be opened
after the repository is opened.  These variable length paths will be
checked when they are accessed themselves.  This new functionality is
done at open to prevent needlessly checking every file in the gitdir
(eg, `MERGE_HEAD`) for its length when we could instead check once at
repository open time.

repo: factor the commondir detection

refs: ensure loose refs adhere to path validation

On Windows, we need to enforce MAX_PATH for loose references and their
reflogs.  Ensure that any path - including the lock file - would fit
within the 260 character maximum.

We do not honor core.longpaths for loose reference files or reflogs.
core.longpaths only applies to paths in the working directory.

path: don't join paths in git_path_find_dir

Let `git_path_find_dir` simply take a `git_buf` that contains a
directory or a file, instead of trying to both join a path AND then deal
with prettifying it or its basename.  This allows consumers to join
paths themselves (and apply any necessary rules - like fitting within
MAX_PATH).

worktree: validate worktree paths

Worktree paths need to fix within MAX_PATH always, regardless of
`core.longpaths` setting.

submodule: validate path lengths

Validate that working directory paths honor `core.longpaths` where
appropriate.  Paths to the submodule gitdirs must always honor the
operating system length restrictions; `core.longpaths` does not affect
gitdir paths.

iterator: validate workdir paths

Supply the repository for the filesystem and workdir iterators - for
workdir iterators, this is non-null and we can lookup the core.longpaths
configuration option.  (For regular filesystem iterators, this is NULL,
so core.longpaths does not apply.)

ignore: validate workdir paths for ignore files

attr: validate workdir paths for attribute files

We should allow attribute files - inside working directories - to have
names longer than MAX_PATH when core.longpaths is set.
`git_attr_path__init` takes a repository to validate the path with.

repository: validate workdir path lengths

workdir: validate working directory entry path length

diff: use git_repository_workdir_path

The new git_repository_workdir_path function does error checking on
working directory inputs on Windows; use it to construct paths within
working directories.

use git_repository_workdir_path to generate paths

Use `git_repository_workdir_path` to generate workdir paths since it
will validate the length.

checkout: validate path length

Ensure that we are validating working directory paths before we try to
write to them.

checkout: use target path; don't assume workdir

We're not necessarily checking out into the working directory.  We could
be checking out into an arbitrary location.  Ensure that when we are
writing conflict data that we do it in the checkout target.

repo: introduce git_repository_workdir_path

Add a simple accessor for workdir paths to get an absolute on-disk path
given a repository and a relative path within it.  This is useful to
avoid copy-pasta `git_buf_joinpath` and to ensure that we validate
working directory paths while honoring `core.longpaths` settings.

apply: ensure we validate paths

There was no test ensuring that we validate `.git` paths.  We do, but
let's add a test to make sure that we never regress this.

path: introduce ondisk and workdir path validation

Introduce `git_path_validate_filesystem` which validates (absolute) on-disk
paths and `git_path_validate_workdir` to perform validations on (absolute)
working directory paths.  These functions are useful as there may be system
limitations on on-disk paths, particularly on Windows (for example,
enforcing MAX_PATH).

For working directory paths, these limitations may be per-repository, based
on the `core.longpaths` configuration setting.

Fix typo

Uniformise documentation return code separator

Fix documentation formatting

The return code description use '-' char as list of return value.
But with the generation of the documentation the char '-' create an
ambiguous return code value who seem an negative value.

git_reference_create_matching: Treat all-zero OID as "must be absent"

This is pretty useful in avoiding races: I want to create a ref only if
it doesn't already exist.  I can't check first because of TOCTOU -- by
the time I finish the check, someone else might have already created
the ref.  And I can't take a lock because then I can't do the create,
since the create expects to take the lock.

The semantics are inspired by git update-ref, which allows an all-zero old
value to mean that the ref must not exist.

Merge pull request #5843 from tniessen/src-fix-typos-in-headers

src: fix typos in header files

src: fix typos in header files

path: git_path_isvalid -> git_path_validate

If we want to validate more and different types of paths, the name
`git_path_validate` makes that easier and more expressive.  We can add,
for example, `git_path_validate_foo` while the current name makes that
less ergonomic.

docs: document core.longpaths support

utf8: introduce git_utf8_char_length

Introduce a function to determine the number of Unicode characters in a
given UTF-8 string.

utf8: refactor utf8 functions

Move the utf8 functions into a proper namespace `git_utf8` instead of
being in the namespaceless `git__` function group.  Update them to
have out-params first and use `char *` instead of `uint8_t *` to match
our API treating strings as `char *` (even if they truly contain `uchar`s
inside).

Merge pull request #5834 from libgit2/cmn/repo-no-passthrough

repo: remove an inappropriate use of PASSTHROUGH

Merge pull request #5835 from libgit2/cmn/branch-or-ref

repo: specify init.defaultbranch is meant to be a branch name

Merge pull request #5836 from tniessen/tests-var-name-list-c

tests: fix variable name in list.c

Merge pull request #5838 from mjsir911/msirabella/entrycount_typo_fix

Fix diff_entrycount -> diff_num_deltas doc typo

Fix diff_entrycount -> diff_num_deltas doc typo

This just fixes a typo in the documentation, actual rename change was
done in 5f69a31f

tests: fix variable name in list.c

repo: specify init.defaultbranch is meant to be a branch name

We don't want the default branch's refname here but its name as branch.
Returning an error saying it's not a valid reference here suggests we want the
value of `init.defaultbranch` to be something like `refs/heads/default` which is
not the case.

repo: remove an inappropriate use of PASSTHROUGH

This error code is for callbacks where we should act as though the callback was
not set. This is not something that makes sense for a `_foreach` and checking
for that error message to bubble up mostly is happenstance precisely because
this is not an error code we expect in the callback.

As part of removing this, let's also remove a use of foreach as we can open-code
this check.

Merge pull request #5832 from ianhattendorf/fix/repo-is-empty-empty-config-defaultbranch

Default to GIT_BRANCH_DEFAULT if init.defaultBranch is empty string

Default to GIT_BRANCH_DEFAULT if init.defaultBranch is empty string

We already do this in repo_init_head

Update README.md for additional Delphi bindings

Delphi/Free pascal bindings targeting the latest version of libgit2

Merge pull request #5821 from punkymaniac/example-duplicate-line

Remove duplicate line, in example code

Remove duplicate line, in example code

Merge pull request #5815 from libgit2/ethomson/treebuilder_write

tree: deprecate `git_treebuilder_write_with_buffer`

Merge pull request #5814 from ianhattendorf/fix/winhttp-proxy-https

winhttp: skip certificate check if unable to send request

README: it's actually our main branch

[skip ci]

Merge pull request #5764 from lhchavez/cgraph-needs-refresh

commit-graph: Introduce `git_commit_graph_needs_refresh()`

Merge pull request #5763 from lhchavez/cgraph-lookup

commit-graph: Support lookups of entries in a commit-graph

ci: enable UBSan stacktraces

tree: deprecate `git_treebuilder_write_with_buffer`

The function `git_treebuilder_write_with_buffer` is unnecessary; it
is used internally as part of treebuilder writing, but it has little
use to external callers.  For callers that repeatedly write a
treebuilder, we can supply them with a buffer in the treebuilder struct
instead of recreating it.  For ourselves, when we want a single buffer
in our write loop, we can use an internal function.

Merge pull request #5818 from mamapanda/patch-3

merge: Check insert_head_ids error in create_virtual_base

Merge pull request #5817 from mamapanda/patch-2

Check git_signature_dup failure

merge: Check insert_head_ids error in create_virtual_base

insert_head_ids can fail due to allocation error

Check git_signature_dup failure

git_signature_dup can have an allocation failure

winhttp: skip certificate check if unable to send request

In some circumstances (e.g. when proxies are involved), winhttp will fail to reach the WINHTTP_CALLBACK_STATUS_SENDING_REQUEST phase. If this occurs, we'll error with ERROR_WINHTTP_INCORRECT_HANDLE_STATE when attempting to query the server certificate context (see https://docs.microsoft.com/en-us/windows/win32/api/winhttp/nf-winhttp-winhttpsendrequest#remarks).

To avoid this, verify that WinHttpSendRequest has reached the WINHTTP_CALLBACK_STATUS_SENDING_REQUEST phase before checking the certificate. Since we're using WinHTTP in synchronous mode, we know for sure that once WinHttpSendRequest returns we've either sent it successfully or not.

NOTE: WINHTTP_CALLBACK_STATUS_SENDING_REQUEST appears to be deprecated with no direct replacement. WINHTTP_CALLBACK_STATUS_SENDREQUEST_COMPLETE is only available in async mode, and there doesn't appear to be a method of querying this flag outside of the status callback.

winhttp: test proxy https clone

Merge pull request #5797 from aaronfranke/typos

Fix some typos

Merge pull request #5805 from tniessen/include-typos

include: fix typos in comments

Merge pull request #5806 from punkymaniac/comment-format

Fix documentation formating on repository.h

include: fix typos in comments

Fix documentation formating on repository.h

The enum 'git_repository_init_flag_t', 'git_repository_init_mode_t' and
the structure 'git_repository_init_options' does not follow the format
used by docurium.

Merge pull request #5801 from mamapanda/patch-1

index: Check git_vector_dup error in write_entries

index: Initialize case_sorted to GIT_VECTOR_INIT

This is for extra safety within write_entries

Merge pull request #5800 from mamapanda/check-sortedcache-lock

index: Check git_vector_dup error in write_entries

If allocating case_sorted.contents fails, git_vector_sort will segfault.

refdb_fs: Check git_sortedcache wlock/rlock errors

To prevent a race condition, these wlock/rlock calls should probably be checked
for errors.