rust/comprehensive-rust
1
0
Fork 0
mirror of https://github.com/google/comprehensive-rust.git synced 2025-10-31 08:37:45 +02:00
Code Issues Releases Activity

Use Arc::clone instead of .clone() (#1023)

Browse Source 
The fully qualified syntax is recommended for `Arc` which implements all methods as associated methods.
This commit is contained in:
Martin Geisler
2023-07-24 20:19:06 +02:00
committed by GitHub
parent a2367b4d93
commit e8ea741cea
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/exercises/concurrency/dining-philosophers.rs
View File

@@ -62,8 +62,8 @@ fn main() {
for i in 0..forks.len() {
let tx = tx.clone();
let mut left_fork = forks[i].clone();
let mut right_fork = forks[(i + 1) % forks.len()].clone();
let mut left_fork = Arc::clone(&forks[i]);
let mut right_fork = Arc::clone(&forks[(i + 1) % forks.len()]);
// To avoid a deadlock, we have to break the symmetry
// somewhere. This will swap the forks without deinitializing