mirror of
https://git.proxmox.com/git/proxmox-backup
synced 2025-08-07 20:04:35 +00:00
shorten code a tiny bit
Ends up with a rather long pattern guard but does avoid the return case duplication... Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
This commit is contained in:
parent
70bca12324
commit
a9f5360b82
@ -565,10 +565,9 @@ impl Extractor {
|
|||||||
|
|
||||||
match nix::unistd::symlinkat(link, Some(parent), file_name) {
|
match nix::unistd::symlinkat(link, Some(parent), file_name) {
|
||||||
Ok(()) => {}
|
Ok(()) => {}
|
||||||
Err(err @ nix::errno::Errno::EEXIST) => {
|
Err(nix::errno::Errno::EEXIST)
|
||||||
if !self.overwrite_flags.contains(OverwriteFlags::SYMLINK) {
|
if self.overwrite_flags.contains(OverwriteFlags::SYMLINK) =>
|
||||||
return Err(err.into());
|
{
|
||||||
}
|
|
||||||
// Never unlink directories
|
// Never unlink directories
|
||||||
let flag = nix::unistd::UnlinkatFlags::NoRemoveDir;
|
let flag = nix::unistd::UnlinkatFlags::NoRemoveDir;
|
||||||
nix::unistd::unlinkat(Some(parent), file_name, flag)?;
|
nix::unistd::unlinkat(Some(parent), file_name, flag)?;
|
||||||
@ -605,10 +604,9 @@ impl Extractor {
|
|||||||
|
|
||||||
match dolink() {
|
match dolink() {
|
||||||
Ok(()) => {}
|
Ok(()) => {}
|
||||||
Err(err @ nix::errno::Errno::EEXIST) => {
|
Err(nix::errno::Errno::EEXIST)
|
||||||
if !self.overwrite_flags.contains(OverwriteFlags::HARDLINK) {
|
if self.overwrite_flags.contains(OverwriteFlags::HARDLINK) =>
|
||||||
return Err(err.into());
|
{
|
||||||
}
|
|
||||||
// Never unlink directories
|
// Never unlink directories
|
||||||
let flag = nix::unistd::UnlinkatFlags::NoRemoveDir;
|
let flag = nix::unistd::UnlinkatFlags::NoRemoveDir;
|
||||||
nix::unistd::unlinkat(Some(parent), file_name, flag)?;
|
nix::unistd::unlinkat(Some(parent), file_name, flag)?;
|
||||||
|
Loading…
Reference in New Issue
Block a user