Skip to content

Split recursive_wrapper and recursive_wrapper_alloca#44

Merged
saki7 merged 10 commits intomainfrom
shorten-recursive-wrapper
Mar 3, 2026
Merged

Split recursive_wrapper and recursive_wrapper_alloca#44
saki7 merged 10 commits intomainfrom
shorten-recursive-wrapper

Conversation

@saki7
Copy link
Member

@saki7 saki7 commented Mar 3, 2026

fixes #43

"alloca" stands for "allocator-aware", not "allocating".

This PR effectively shortens the type name into half.

Also includes some documentation updates and refactoring.

workaround for #42
@saki7 saki7 self-assigned this Mar 3, 2026
@saki7 saki7 force-pushed the shorten-recursive-wrapper branch from f8c75f7 to 9512653 Compare March 3, 2026 00:01
@cppwarningnotifier
Copy link

EnvironmentC++23C++26
irisClang21Debug✅success✅success
Release✅success✅success
GCC14Debug✅success✅success
Release✅success✅success
MSVC2022Debug✅success✅success
Release✅success✅success
2026Debug✅success✅success
Release✅success✅success

@saki7 saki7 added enhancement New feature or request documentation Improvements or additions to documentation labels Mar 3, 2026
@saki7 saki7 force-pushed the shorten-recursive-wrapper branch from 9e29256 to 3233c28 Compare March 3, 2026 00:25
@saki7 saki7 marked this pull request as ready for review March 3, 2026 01:38
@saki7 saki7 merged commit 93187f6 into main Mar 3, 2026
34 checks passed
@saki7 saki7 deleted the shorten-recursive-wrapper branch March 3, 2026 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

recursive_wrapper<T> should not contain allocator type in its full signature

1 participant