delta/libgit2.git/tests/threads, branch ethomson/diff_enum

Homogenize semantics for atomic-related functions

2021-08-26T12:34:17+00:00

There were some subtle semantic differences between the various
implementations of atomic functions. Now they behave the same, have
tests and are better documented to avoid this from happening again in
the future.

Of note:

* The semantics chosen for `git_atomic_compare_and_swap` match
  `InterlockedCompareExchangePointer`/`__sync_cal_compare_and_swap` now.
* The semantics chosen for `git_atomic_add` match
  `InterlockedAdd`/`__atomic_add_fetch`.
* `git_atomic_swap` and `git_atomic_load` still have a bit of semantic
  difference with the gcc builtins / msvc interlocked operations, since
  they require an l-value (not a pointer). If desired, this can be
  homogenized.

threads: git_tls_data to git_tlsdata

2020-12-08T14:53:13+00:00

Use a no-allocation approach to the TLS data abstraction.

threads: rename thread files to thread.[ch]

2020-12-06T01:08:22+00:00

threads: rename git_atomic to git_atomic32

2020-12-06T01:08:22+00:00

Clarify the `git_atomic` type and functions now that we have a 64 bit
version as well (`git_atomic64`).

thread: restore the git_thread_exit tests

2020-10-11T19:06:34+00:00

We were never properly testing git_thread_exit.  Do so.

git_error: use new names in internal APIs and usage

2019-01-22T22:30:35+00:00

Move to the `git_error` name in the internal API for error-related
functions.

threads::iterator: use separate repository objects

2018-08-19T11:57:29+00:00

Our thread policies state that we cannot re-use the `git_repository`
across threads.  Our tests cannot deviate from that.

Courtesy of Ximin Luo, https://github.com/infinity0:

https://github.com/libgit2/libgit2/issues/4753#issuecomment-412247757

threads::diff: use separate git_repository objects

2018-08-05T10:01:14+00:00

Our thread policies state that we cannot re-use the `git_repository`
across threads.  Our tests cannot deviate from that.

treewide: remove use of C++ style comments

2018-07-13T06:25:12+00:00

C++ style comment ("//") are not specified by the ISO C90 standard and
thus do not conform to it. While libgit2 aims to conform to C90, we did
not enforce it until now, which is why quite a lot of these
non-conforming comments have snuck into our codebase. Do a tree-wide
conversion of all C++ style comments to the supported C style comments
to allow us enforcing strict C90 compliance in a later commit.

tests: threads::basic: remove unused function `exit_abruptly`

2017-05-15T05:34:04+00:00