-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[circular queue] fixing index of callback(s) (preparatory work) #5
Merged
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
…s now `intptr_t` (was `size_t`) so can connote "unknown"
synesissoftware
approved these changes
Feb 9, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
synesissoftware
approved these changes
Feb 9, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
synesissoftware
added a commit
that referenced
this pull request
Feb 10, 2025
commit 04952f8ce9741f08f7852a4fbcdad54dab1a9337 Author: Matt Wilson <[email protected]> Date: Tue Feb 11 08:57:48 2025 +1100 ~ CMake fix commit 9223976 Author: Matt Wilson <[email protected]> Date: Tue Feb 11 08:55:01 2025 +1100 V0.1 (#15) * [0.1] : ~ container versions * ~ final version preparation commit 4328cbb Author: Matt Wilson <[email protected]> Date: Tue Feb 11 08:42:34 2025 +1100 Function pointers (#14) * ~ tidying * ~ refactoring support macros into **collect-c/common.h** * [documentation] : ~ language * [documentation] : ~ clearing up preconditions declarations * [documentation] : + adding missing documentation elements * [refac] : ~ function pointers * [refac] : ~ doc fix commit 68355ef Author: Matt Wilson <[email protected]> Date: Tue Feb 11 08:35:40 2025 +1100 Documentation (#13) * ~ tidying * ~ refactoring support macros into **collect-c/common.h** * [documentation] : ~ language * [documentation] : ~ clearing up preconditions declarations * [documentation] : + adding missing documentation elements commit d529112 Author: Matt Wilson <[email protected]> Date: Tue Feb 11 08:31:09 2025 +1100 refactor (1) (#12) * ~ tidying * ~ refactoring support macros into **collect-c/common.h** commit c59892f Author: Matt Wilson <[email protected]> Date: Mon Feb 10 20:00:34 2025 +1100 Vec (#11) * [vector] : initial (and very partial) implementation * [vector] : + added `#clear()` and `#push_front_by_ref()` --------- Co-authored-by: Matt Wilson <[email protected]> commit 20f5e72 Author: Matt Wilson <[email protected]> Date: Mon Feb 10 19:47:35 2025 +1100 ~ tidying (#10) Co-authored-by: Matt Wilson <[email protected]> commit 9204ce6 Author: Matt Wilson <[email protected]> Date: Mon Feb 10 13:38:30 2025 +1100 library version commit ee951a6 Author: Matt Wilson <[email protected]> Date: Mon Feb 10 13:03:17 2025 +1100 [doubly-linked list] (#9) * [double-linked list] : initial (and very partial) implementation * [double-linked list] : + `#push_front()` * [double-linked list] : ~ `#free_storage()` now ensures `#spares` list is freed * [double-linked list] : + `#clear()` * [double-linked list] : + `#find_node()` and `#rfind_node()` * [double-linked list] : + `#erase_node)` * [double-linked list] : ~ macro improvements, such that most macros can now take `collect_c_dlist_t*` (and `collect_c_dlist_t const*`) as well as `collect_c_dlist_t` * [double-linked list] : + added `#front()` and `#back()` * [double-linked list] : + added `#insert_after()` and `#insert_before()` --------- Co-authored-by: Matt Wilson <[email protected]> commit cf99c64 Author: Matt Wilson <[email protected]> Date: Mon Feb 10 09:07:39 2025 +1100 [circular queue] deprecated elements (#8) * ~ naming consistency * **test.unit.cq** : ~ improved test case names * [circular queue] : ~ `collect_c_cq_t#pfn_element_free`'s `el_index` is now `intptr_t` (was `size_t`) so can connote "unknown" * [circular queue] : ~ tidying * ~ CMake fixes * [circular queue] : ~ fixed callback index for `collect_c_cq_free_storage()` and `#clear()` * [circular queue] : ~ fixing callback indexes, no properly indicates the logical index of the element * [circular queue] : ~ macro fixes * [circular queue] + added `collect_c_cq_push_back_n_by_ref()` * [circular queue] : ~ improved various API names * [circular queue] : ~ sorting deprecated constructs * [circular queue] : ~ replacing deprecated constructs commit b9af539 Author: Matt Wilson <[email protected]> Date: Mon Feb 10 09:04:20 2025 +1100 [circular queue] : + added `collect_c_cq_push_back_by_ref()` (#7) * ~ naming consistency * **test.unit.cq** : ~ improved test case names * [circular queue] : ~ `collect_c_cq_t#pfn_element_free`'s `el_index` is now `intptr_t` (was `size_t`) so can connote "unknown" * [circular queue] : ~ tidying * ~ CMake fixes * [circular queue] : ~ fixed callback index for `collect_c_cq_free_storage()` and `#clear()` * [circular queue] : ~ fixing callback indexes, no properly indicates the logical index of the element * [circular queue] : ~ macro fixes * [circular queue] + added `collect_c_cq_push_back_n_by_ref()` * [circular queue] : ~ improved various API names commit fe1d8e0 Author: Matt Wilson <[email protected]> Date: Mon Feb 10 08:48:19 2025 +1100 [circular queue] fixing index of callback(s) (#6) * ~ naming consistency * **test.unit.cq** : ~ improved test case names * [circular queue] : ~ `collect_c_cq_t#pfn_element_free`'s `el_index` is now `intptr_t` (was `size_t`) so can connote "unknown" * [circular queue] : ~ tidying * ~ CMake fixes * [circular queue] : ~ fixed callback index for `collect_c_cq_free_storage()` and `#clear()` * [circular queue] : ~ fixing callback indexes, no properly indicates the logical index of the element * [circular queue] : ~ macro fixes commit 04fa084 Author: Matt Wilson <[email protected]> Date: Sun Feb 9 14:12:46 2025 +1100 ~ minor mods commit c030cf9 Author: Matt Wilson <[email protected]> Date: Sun Feb 9 14:09:27 2025 +1100 [circular queue] fixing index of callback(s) (preparatory work) (#5) * ~ naming consistency * **test.unit.cq** : ~ improved test case names * [circular queue] : ~ `collect_c_cq_t#pfn_element_free`'s `el_index` is now `intptr_t` (was `size_t`) so can connote "unknown" * [circular queue] : ~ tidying * ~ CMake fixes commit dcc7b72 Author: Matt Wilson <[email protected]> Date: Fri Feb 7 11:20:06 2025 +1100 [circular queue] : Added element-overwrite functionality (#4) * cq : initial * [circular queue] : initial implementation * [circular queue] : + added erase/remove/overwrite callback functionality * [circular queue] : `COLLECT_C_CIRCQ_clear()` now takes optional second argument * [circular queue] : ~ minor optimisation of `#clear()`, and resetting `b` = `e` = 0 when clear / deallocate * [circular queue] : + added **test.performance.cq** * [circular queue] : ~ documentation markup * [circular queue] : ~ fix * [circular queue] : + added and applied `COLLECT_C_CIRCQ_F_OVERWRITE_FRONT_WHEN_FULL` * ~ settings * [circular queue] : ~ renamed `COLLECT_C_CIRCQ_push_by_ref()` / `COLLECT_C_CIRCQ_push_by_value()` => `CLC_CQ_push_by_ref()` / `CLC_CQ_push_by_value()` * [circular queue] : ~ tidying / simplifying test programs * [circular queue] : ~ fix * [circular queue] : + added performance test of use of `COLLECT_C_CIRCQ_F_OVERWRITE_FRONT_WHEN_FULL` * [circular queue] : + added and applied `COLLECT_C_CIRCQ_define_empty_with_callback()` * [circular queue] : ~ refactoring commit a2e93fd Author: Matt Wilson <[email protected]> Date: Fri Feb 7 11:02:49 2025 +1100 Circular Queue (#3) * cq : initial * [circular queue] : initial implementation * [circular queue] : + added erase/remove/overwrite callback functionality * [circular queue] : `COLLECT_C_CIRCQ_clear()` now takes optional second argument * [circular queue] : ~ minor optimisation of `#clear()`, and resetting `b` = `e` = 0 when clear / deallocate * [circular queue] : + added **test.performance.cq** * [circular queue] : ~ documentation markup * [circular queue] : ~ fix * ~ settings * [circular queue] : ~ renamed `COLLECT_C_CIRCQ_push_by_ref()` / `COLLECT_C_CIRCQ_push_by_value()` => `CLC_CQ_push_by_ref()` / `CLC_CQ_push_by_value()` * [circular queue] : ~ tidying / simplifying test programs * [circular queue] : ~ fix * [circular queue] : ~ tidying * [circular queue] : ~ updating **test.unit.version** to incorporate circular queue * [circular queue] : ~ tidying * [circular queue] : ~ fixing assertion failures * [circular queue] : ~ minor refactoring commit e1d612c Author: Matt Wilson <[email protected]> Date: Fri Feb 7 10:39:29 2025 +1100 + added **test.unit.version** (#2) Co-authored-by: Matt Wilson <[email protected]>
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.
No description provided.