TudbuT/helix-mods
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Fix panic when starting helix tutor (#11352)

Browse source 
Closes #11351

Also fixed some minor issues related to log
message contents, and removed unnecessary use
of `.as_mut()` as per code review comments on
the PR.
This commit is contained in:
dnaq 2024-07-28 18:29:26 +02:00 • committed by GitHub
parent fade4b218c
commit f5950196d9
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 14 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
helix-view/src/document.rs
View file

@ -1080,22 +1080,25 @@ impl Document {
} }
pub fn pickup_last_saved_time(&mut self) { pub fn pickup_last_saved_time(&mut self) {
self.last_saved_time = match self.path.as_mut().unwrap().metadata() { self.last_saved_time = match self.path().as_mut() {
Ok(metadata) => match metadata.modified() { Some(path) => match path.metadata() {
Ok(mtime) => mtime, Ok(metadata) => match metadata.modified() {
Err(_) => { Ok(mtime) => mtime,
Err(_) => {
log::error!(
"Use a system time instead of fs' mtime not supported on this platform"
);
SystemTime::now()
}
},
Err(e) => {
log::error!( log::error!(
"Use a system time instead of fs' mtime not supported on this platform" "Use a system time instead of fs' mtime: failed to file's metadata: {e}"
); );
SystemTime::now() SystemTime::now()
} }
}, },
Err(e) => { None => SystemTime::now(),
log::error!(
"Use a system time instead of fs' mtime: failed to file's metadata: {e}"
);
SystemTime::now()
}
}; };
} }