mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix: revert strange usage of ? operator

Browse Source
This commit is contained in:
Basti Ortiz 2021-08-10 22:44:15 +08:00
parent 27605b402e
commit 0986632c04
No known key found for this signature in database
GPG Key ID: 1402D5CB17F48E1B
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
crates/vfs/src/lib.rs
View File

@ -111,9 +111,7 @@ pub fn len(&self) -> usize {
/// Id of the given path if it exists in the `Vfs` and is not deleted.
pub fn file_id(&self, path: &VfsPath) -> Option<FileId> {
let it = self.interner.get(path)?;
let _ = self.get(it).as_ref()?;
Some(it)
self.interner.get(path).filter(|&it| self.get(it).is_some())
}
/// File path corresponding to the given `file_id`.
@ -139,12 +137,13 @@ pub fn file_contents(&self, file_id: FileId) -> &[u8] {
///
/// This will skip deleted files.
pub fn iter(&self) -> impl Iterator<Item = (FileId, &VfsPath)> + '_ {
(0..self.data.len()).filter_map(move |it| {
let file_id = FileId(it as u32);
let _ = self.get(file_id).as_ref()?;
let path = self.interner.lookup(file_id);
Some((file_id, path))
})
(0..self.data.len())
.map(|it| FileId(it as u32))
.filter(move |&file_id| self.get(file_id).is_some())
.map(move |file_id| {
let path = self.interner.lookup(file_id);
(file_id, path)
})
}
/// Update the `path` with the given `contents`. `None` means the file was deleted.