Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Document the cloning behavior for Batch #643

Merged
merged 2 commits into from
Jun 2, 2023

Conversation

marmistrz
Copy link
Contributor

@marmistrz marmistrz commented May 31, 2022

We could consider adding something like

impl<T: BatchTransport> Batch<T> {
    pub fn deep_clone(&self) -> Self {
        Self::new(self.transport.clone())
    }
}

or perhaps this is what we'd like to use for Clone, actually?

@tomusdrw tomusdrw enabled auto-merge (squash) March 27, 2023 20:37
@tomusdrw tomusdrw disabled auto-merge June 2, 2023 19:55
@tomusdrw tomusdrw merged commit b4095d2 into tomusdrw:master Jun 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants