cleanup

src/main.rs

@@ -167,30 +167,35 @@ impl Navigator {
 	fn enter(&mut self) {
 		if self.selected < self.editors.len() {
 			self.editors[self.selected].enter();
-		} else {
+			return;
+		}
+
 		let i = self.selected - self.editors.len();
+		// top entry is hardcoded to be ../
 		if i == 0 {
 			if let Some(parent) = self.path.parent() {
 				self.set_path(self.path.join(parent));
 			}
-			} else {
+			return;
+		}
+
 		let path = &self.files[i];
 		if path.is_dir() {
 			self.set_path(self.path.join(path));
-				} else if path.is_file() {
+			return;
+		}
+		if path.is_file() {
 			let path = path.canonicalize().unwrap();
-					let mut editor_index = self.editors.len();
+			let mut selected = self.editors.len();
 			for (i, editor) in self.editors.iter().enumerate() {
 				if editor.path() == Some(&path) {
-							editor_index = i;
+					selected = i;
 					break;
 				}
 			}
-					if editor_index == self.editors.len() {
-						match Editor::open_file(
-							self.clipboard.clone(),
-							path.canonicalize().unwrap(),
-						) {
+			// no editor exists with this path
+			if selected == self.editors.len() {
+				match Editor::open_file(self.clipboard.clone(), path) {
 					Ok(editor) => self.editors.push(editor),
 					Err(err) => {
 						self.message(format!("Could not open file: {err}"));
@@ -198,12 +203,10 @@ impl Navigator {
 					}
 				}
 			}
-					self.selected = editor_index;
+			self.selected = selected;
 			self.open_selected();
 		}
 	}
-		}
-	}
 
 	fn set_path(&mut self, new_path: PathBuf) {
 		match env::set_current_dir(&new_path) {