rust/comprehensive-rust
1
0
Fork 0
mirror of https://github.com/google/comprehensive-rust.git synced 2025-03-06 00:44:23 +02:00
Code Issues Releases Activity

Remove Copy bound from binary tree exercise (#1606)

Browse Source 
There's absolutely no need for the element type of the binary tree to be
`Copy`. Let's remove this bound hence.
This commit is contained in:
Martin Huschenbett 2023-12-20 16:41:56 +01:00 committed by GitHub
parent 52bfb0c4a3
commit 8e4bb60023
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/smart-pointers/exercise.rs
View File

@ -18,7 +18,7 @@ use std::cmp::Ordering;
// ANCHOR: types
/// A node in the binary tree.
#[derive(Debug)]
struct Node<T: Ord + Copy> {
struct Node<T: Ord> {
value: T,
left: Subtree<T>,
right: Subtree<T>,
@ -26,18 +26,18 @@ struct Node<T: Ord + Copy> {
/// A possibly-empty subtree.
#[derive(Debug)]
struct Subtree<T: Ord + Copy>(Option<Box<Node<T>>>);
struct Subtree<T: Ord>(Option<Box<Node<T>>>);
/// A container storing a set of values, using a binary tree.
///
/// If the same value is added multiple times, it is only stored once.
#[derive(Debug)]
pub struct BinaryTree<T: Ord + Copy> {
pub struct BinaryTree<T: Ord> {
root: Subtree<T>,
}
// ANCHOR_END: types
impl<T: Ord + Copy> BinaryTree<T> {
impl<T: Ord> BinaryTree<T> {
fn new() -> Self {
Self {
root: Subtree::new(),
@ -57,7 +57,7 @@ impl<T: Ord + Copy> BinaryTree<T> {
}
}
impl<T: Ord + Copy> Subtree<T> {
impl<T: Ord> Subtree<T> {
fn new() -> Self {
Self(None)
}
@ -92,7 +92,7 @@ impl<T: Ord + Copy> Subtree<T> {
}
}
impl<T: Ord + Copy> Node<T> {
impl<T: Ord> Node<T> {
fn new(value: T) -> Self {
Self {
value,