Conversation
|
mv_multiple_to_dir mv_force_overwrite Is |
a0b2b3f to
6934263
Compare
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
| // remember the moved destinations for further usage | ||
| let mut moved_destinations: HashSet<PathBuf> = HashSet::with_capacity(files.len()); | ||
| let mut moved_destinations: FxHashSet<PathBuf> = | ||
| FxHashSet::with_capacity_and_hasher(files.len(), rustc_hash::FxBuildHasher); |
There was a problem hiding this comment.
Annoying we can't use FxHashSet::with_capacity
There was a problem hiding this comment.
You can open an issue at https://github.com/rust-lang/rustc-hash . (also .with_capacity_auto()?)
5bb6648 to
de7325d
Compare
|
GNU testsuite comparison: |
Merging this PR will not alter performance
Comparing Footnotes
|
src/uu/mv/Cargo.toml
Outdated
| path = "src/mv.rs" | ||
|
|
||
| [dependencies] | ||
| rustc-hash = { workspace = true } |
There was a problem hiding this comment.
Why does not CI detect this?
There was a problem hiding this comment.
We use taplo to lint .toml files. I don't know if this is a supported feature.
|
GNU testsuite comparison: |
No description provided.