Skip to content

feat: impl Deref for Exn#37

Merged
tisonkun merged 4 commits into
mainfrom
exts
Jan 19, 2026
Merged

feat: impl Deref for Exn#37
tisonkun merged 4 commits into
mainfrom
exts

Conversation

@tisonkun

Copy link
Copy Markdown
Contributor

Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Comment thread exn/src/impls.rs
Comment thread exn/src/ext.rs Outdated
Comment on lines +52 to +62
/// Raise this error as a new exception, with `sources` as causes.
#[track_caller]
fn raise_all<T, I>(self, sources: I) -> Exn<Self>
where
Self: Sized,
T: Error + Send + Sync + 'static,
I: IntoIterator,
I::Item: Into<Exn<T>>,
{
Exn::raise_all(self, sources)
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Looks the direction of "raise" is wrong. Perhaps we should remove this extension.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed for now. We need some time to work on the vocabulary.

@tisonkun tisonkun changed the title feat: impl Deref for Exn + add ErrorExt::raise_all feat: impl Deref for Exn Jan 19, 2026
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
@tisonkun tisonkun enabled auto-merge (squash) January 19, 2026 05:13
@tisonkun tisonkun merged commit 85b12a4 into main Jan 19, 2026
9 checks passed
@tisonkun tisonkun deleted the exts branch January 19, 2026 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants