Skip to content
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

Clone args for t! #98

Merged
merged 3 commits into from
Feb 22, 2024
Merged

Clone args for t! #98

merged 3 commits into from
Feb 22, 2024

Conversation

Baptistemontan
Copy link
Owner

When a non-Copy arg is passed to the t! macro, the resulting closure is only FnOnce, even if the type of the arg impl Clone.

Now clone the args making the function Fn and not just FnOnce.

@Baptistemontan Baptistemontan merged commit 060da58 into master Feb 22, 2024
16 checks passed
@Baptistemontan Baptistemontan deleted the clone_t_args branch February 22, 2024 20:23
Baptistemontan added a commit that referenced this pull request Feb 24, 2024
commit d3db6f6
Author: Baptiste de Montangon <[email protected]>
Date:   Thu Feb 22 22:19:06 2024 +0100

    v0.3.3

commit 4b5ed89
Author: Baptiste de Montangon <[email protected]>
Date:   Thu Feb 22 21:43:34 2024 +0100

    updated doc

commit 060da58
Merge: de9bd6b 4bb2ce4
Author: Baptiste <[email protected]>
Date:   Thu Feb 22 21:23:01 2024 +0100

    Merge pull request #98 from Baptistemontan/clone_t_args

    Clone args for `t!`

commit de9bd6b
Author: Baptiste de Montangon <[email protected]>
Date:   Thu Feb 22 21:22:47 2024 +0100

    better error if multiple files format features are provided

commit 4bb2ce4
Author: Baptiste de Montangon <[email protected]>
Date:   Thu Feb 22 19:50:01 2024 +0100

    updated code output doc

commit 2b4be7f
Author: Baptiste de Montangon <[email protected]>
Date:   Thu Feb 22 19:45:50 2024 +0100

    added test for non copy args

commit 93e10e7
Author: Baptiste de Montangon <[email protected]>
Date:   Thu Feb 22 19:31:01 2024 +0100

    clone args for t!, making the closure Fn instead of FnOnce
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.

1 participant