mirror of
https://github.com/bootandy/dust.git
synced 2026-06-08 11:29:05 +03:00
style: Fix clippy
New clippy has new standards. Clean up get_metadata by removing unused return
This commit is contained in:
+24
-24
@@ -29,34 +29,34 @@ pub fn get_metadata<P: AsRef<Path>>(
|
|||||||
Ok(md) => {
|
Ok(md) => {
|
||||||
let file_size = md.len();
|
let file_size = md.len();
|
||||||
if use_apparent_size {
|
if use_apparent_size {
|
||||||
return Some((
|
Some((
|
||||||
file_size,
|
file_size,
|
||||||
Some((md.ino(), md.dev())),
|
Some((md.ino(), md.dev())),
|
||||||
(md.mtime(), md.atime(), md.ctime()),
|
(md.mtime(), md.atime(), md.ctime()),
|
||||||
));
|
))
|
||||||
}
|
|
||||||
|
|
||||||
// On NTFS mounts, the reported block count can be unexpectedly large.
|
|
||||||
// To avoid overestimating disk usage, cap the allocated size to what the
|
|
||||||
// file should occupy based on the file system I/O block size (blksize).
|
|
||||||
// Related: https://github.com/bootandy/dust/issues/295
|
|
||||||
let blksize = md.blksize();
|
|
||||||
let target_size = file_size.div_ceil(blksize) * blksize;
|
|
||||||
let reported_size = md.blocks() * get_block_size();
|
|
||||||
|
|
||||||
// File systems can pre-allocate more space for a file than what would be necessary
|
|
||||||
let pre_allocation_buffer = blksize * 65536;
|
|
||||||
let max_size = target_size + pre_allocation_buffer;
|
|
||||||
let allocated_size = if reported_size > max_size {
|
|
||||||
target_size
|
|
||||||
} else {
|
} else {
|
||||||
reported_size
|
// On NTFS mounts, the reported block count can be unexpectedly large.
|
||||||
};
|
// To avoid overestimating disk usage, cap the allocated size to what the
|
||||||
Some((
|
// file should occupy based on the file system I/O block size (blksize).
|
||||||
allocated_size,
|
// Related: https://github.com/bootandy/dust/issues/295
|
||||||
Some((md.ino(), md.dev())),
|
let blksize = md.blksize();
|
||||||
(md.mtime(), md.atime(), md.ctime()),
|
let target_size = file_size.div_ceil(blksize) * blksize;
|
||||||
))
|
let reported_size = md.blocks() * get_block_size();
|
||||||
|
|
||||||
|
// File systems can pre-allocate more space for a file than what would be necessary
|
||||||
|
let pre_allocation_buffer = blksize * 65536;
|
||||||
|
let max_size = target_size + pre_allocation_buffer;
|
||||||
|
let allocated_size = if reported_size > max_size {
|
||||||
|
target_size
|
||||||
|
} else {
|
||||||
|
reported_size
|
||||||
|
};
|
||||||
|
Some((
|
||||||
|
allocated_size,
|
||||||
|
Some((md.ino(), md.dev())),
|
||||||
|
(md.mtime(), md.atime(), md.ctime()),
|
||||||
|
))
|
||||||
|
}
|
||||||
}
|
}
|
||||||
Err(_e) => None,
|
Err(_e) => None,
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user