Skip to content

Remove safe_fn, refactor fn/unique_fn, and add classic_fn. - #117

Merged
Kim-J-Smith merged 7 commits into
dev/v2.2.0/trunkfrom
v2.2.0/dev/classic_fn
Aug 1, 2026
Merged

Remove safe_fn, refactor fn/unique_fn, and add classic_fn.#117
Kim-J-Smith merged 7 commits into
dev/v2.2.0/trunkfrom
v2.2.0/dev/classic_fn

Conversation

@Kim-J-Smith

Copy link
Copy Markdown
Owner

No description provided.

@Kim-J-Smith Kim-J-Smith added new feature This is a new feature. careful review Complex pull request, which needs more careful review. labels Jul 31, 2026
Comment thread include/embed/embed_function.hpp Outdated
Comment thread include/embed/embed_function.hpp

@Kim-J-Smith Kim-J-Smith left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Kim-J-Smith
Kim-J-Smith merged commit 13fee5e into dev/v2.2.0/trunk Aug 1, 2026
6 checks passed
@Kim-J-Smith
Kim-J-Smith deleted the v2.2.0/dev/classic_fn branch August 1, 2026 16:56
Kim-J-Smith added a commit that referenced this pull request Aug 2, 2026
)

* refactor: WIP: refactor fn, fn_ref. deprecated safe_fn and add classic_fn.

* feat: propagate noexcept qualifiers through make_fn and allow noexcept signatures for fn/unique_fn

* chore: remove view_buf and update documents.

* chore: remove outdate comments.

* docs: update documents.

* resolve review comments.

* style: rename noexcept_qualify_like as get_correct_signature
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

careful review Complex pull request, which needs more careful review. new feature This is a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant