Version 0.1.1 #40
Merged
Version 0.1.1 #40
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
David-Haim
commented
Mar 26, 2021
David-Haim
commented
- fully move to CTest as the testing framwork
- unified shutdown exception for executors and the timer queue
- test folder cleanup
- timer queue has idling mechanism for its worker thread
- optimizations
* Add CLion ignores * Use Modern CMake * Restructure the project files * Move test files around * Remove Unnecessary CMakeLists.txt * Restructure the examples * Move thread sanitizer tests to the test directory * Put coroutine options in a separate CMake module * Rewrite the example lists files * Add the option to build the library with thread sanitizer (This only works when the compiler is Clang based.) * Rewrite the sandbox lists file * Add coroutine options to tests as well * Make the add_test function forward properties * Stop script execution early to reduce indentation * Include coroutine options using a relative path Remembering to always pass the CMAKE_MODULE_PATH for every cmake invocation might become troublesome otherwise. Co-authored-by: friendlyanon <friendlyanon@users.noreply.github.com>
* Run clang-format * Missing <algorithm> header * Rework the test folder * Fix include paths in library and example code * Add a lists file that builds all the examples * Rework coroutineOptions.cmake * Fix ignores for CLion * Add CI support There are 4 builds in CI, on 2 providers: Github Actions: * Ubuntu Bionic, Clang 10, with TSAN * Ubuntu Bionic, Clang 10, without TSAN * Windows, VS2019 Travis CI: * macOS, Xcode 10.3 * Allow failures in the Travis CI * Add missing newlines at EOF Co-authored-by: friendlyanon <friendlyanon@users.noreply.github.com>
* fix include path in the sandbox and tests
* Simplify CI process (Remove Travis.ci, move macOS steps to github actions) Co-authored-by: friendlyanon <friendlyanon@users.noreply.github.com>
* Modify the imported target directly * Use a single variable for TSAN Co-authored-by: friendlyanon <friendlyanon@users.noreply.github.com>
* clang-format fine tuning
* better readme, fix include path in examples
* Make TSAN compile correctly by adding missing cmake properties * remove buggish assert from thread-pool test * compile everything in release mode with debug info
* result_core refactor * renamed to result_state * broken to consumer_context and producer_context * await_via behaviuor fine tune - resolve_via throws executor_exception if the coroutine is supended and enqueueing its resumption throws * await/await_via/resolve/resolve_via tests were re-written
* task object replaces raw usage of std::coroutine_handle * calls to executor::post try to inline the memory by using the task inline buffer (the callable doesn't run as a coroutine anymore) * await(_via)/relove(_via) throw concurrencpp::broken_task if the executor destroyed the associated task, for example, by throwing * await small refactor
* complete executors-refactor * timer::cancel also cancels scheduled-not-executed tasks, ongoing tasks are uneffected * test utils refactor * complete tests-re-write
* support shared_result objects
* result_state and result_coro_promise were merged together (finally!) * thread sanitizer tests were rewritten * small improvements and bug fixes
Co-authored-by: NN <NN---@users.noreply.github.com>
* timer queue idling mechanism * unified shutdown exception * test folder cleaning up * clang format changes * optimizations
Ladence
pushed a commit
to Ladence/concurrencpp
that referenced
this pull request
Jul 24, 2021
* full move to ctest * timer queue idling mechanism * unified shutdown exception * test folder cleaning up * clang format changes * optimizations Co-authored-by: friendlyanon <1736896+friendlyanon@users.noreply.github.com> Co-authored-by: friendlyanon <friendlyanon@users.noreply.github.com> Co-authored-by: NN <580536+NN---@users.noreply.github.com> Co-authored-by: NN <NN---@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.