bendns' repos / helix
Unnamed repository; edit this file 'description' to name the repository.
Diffstat (limited to 'helix-term/src/args.rs')
-rw-r--r--helix-term/src/args.rs144
1 files changed, 15 insertions, 129 deletions
diff --git a/helix-term/src/args.rs b/helix-term/src/args.rs
index 090c1192..f0ef09eb 100644
--- a/helix-term/src/args.rs
+++ b/helix-term/src/args.rs
@@ -1,95 +1,32 @@
-use anyhow::Result;
-use helix_core::Position;
-use helix_view::tree::Layout;
-use indexmap::IndexMap;
-use std::path::{Path, PathBuf};
+use anyhow::{Error, Result};
+use std::path::PathBuf;
#[derive(Default)]
pub struct Args {
pub display_help: bool,
pub display_version: bool,
- pub health: bool,
- pub health_arg: Option<String>,
- pub load_tutor: bool,
- pub fetch_grammars: bool,
- pub build_grammars: bool,
- pub split: Option<Layout>,
pub verbosity: u64,
- pub log_file: Option<PathBuf>,
- pub config_file: Option<PathBuf>,
- pub files: IndexMap<PathBuf, Vec<Position>>,
- pub working_directory: Option<PathBuf>,
+ pub files: Vec<PathBuf>,
}
impl Args {
pub fn parse_args() -> Result<Args> {
let mut args = Args::default();
- let mut argv = std::env::args().peekable();
- let mut line_number = 0;
+ let argv: Vec<String> = std::env::args().collect();
+ let mut iter = argv.iter();
- let mut insert_file_with_position = |file_with_position: &str| {
- let (filename, position) = parse_file(file_with_position);
+ iter.next(); // skip the program, we don't care about that
- // Before setting the working directory, resolve all the paths in args.files
- let filename = helix_stdx::path::canonicalize(filename);
-
- args.files
- .entry(filename)
- .and_modify(|positions| positions.push(position))
- .or_insert_with(|| vec![position]);
- };
-
- argv.next(); // skip the program, we don't care about that
-
- while let Some(arg) = argv.next() {
+ for arg in &mut iter {
match arg.as_str() {
"--" => break, // stop parsing at this point treat the remaining as files
"--version" => args.display_version = true,
"--help" => args.display_help = true,
- "--tutor" => args.load_tutor = true,
- "--vsplit" => match args.split {
- Some(_) => anyhow::bail!("can only set a split once of a specific type"),
- None => args.split = Some(Layout::Vertical),
- },
- "--hsplit" => match args.split {
- Some(_) => anyhow::bail!("can only set a split once of a specific type"),
- None => args.split = Some(Layout::Horizontal),
- },
- "--health" => {
- args.health = true;
- args.health_arg = argv.next_if(|opt| !opt.starts_with('-'));
- }
- "-g" | "--grammar" => match argv.next().as_deref() {
- Some("fetch") => args.fetch_grammars = true,
- Some("build") => args.build_grammars = true,
- _ => {
- anyhow::bail!("--grammar must be followed by either 'fetch' or 'build'")
- }
- },
- "-c" | "--config" => match argv.next().as_deref() {
- Some(path) => args.config_file = Some(path.into()),
- None => anyhow::bail!("--config must specify a path to read"),
- },
- "--log" => match argv.next().as_deref() {
- Some(path) => args.log_file = Some(path.into()),
- None => anyhow::bail!("--log must specify a path to write"),
- },
- "-w" | "--working-dir" => match argv.next().as_deref() {
- Some(path) => {
- args.working_directory = if Path::new(path).is_dir() {
- Some(PathBuf::from(path))
- } else {
- anyhow::bail!(
- "--working-dir specified does not exist or is not a directory"
- )
- }
- }
- None => {
- anyhow::bail!("--working-dir must specify an initial working directory")
- }
- },
arg if arg.starts_with("--") => {
- anyhow::bail!("unexpected double dash argument: {}", arg)
+ return Err(Error::msg(format!(
+ "unexpected double dash argument: {}",
+ arg
+ )))
}
arg if arg.starts_with('-') => {
let arg = arg.get(1..).unwrap().chars();
@@ -98,70 +35,19 @@ impl Args {
'v' => args.verbosity += 1,
'V' => args.display_version = true,
'h' => args.display_help = true,
- _ => anyhow::bail!("unexpected short arg {}", chr),
+ _ => return Err(Error::msg(format!("unexpected short arg {}", chr))),
}
}
}
- "+" => line_number = usize::MAX,
- arg if arg.starts_with('+') => {
- match arg[1..].parse::<usize>() {
- Ok(n) => line_number = n.saturating_sub(1),
- _ => insert_file_with_position(arg),
- };
- }
- arg => insert_file_with_position(arg),
+ arg => args.files.push(PathBuf::from(arg)),
}
}
// push the remaining args, if any to the files
- for arg in argv {
- insert_file_with_position(&arg);
- }
-
- if line_number != 0 {
- if let Some(first_position) = args
- .files
- .first_mut()
- .and_then(|(_, positions)| positions.first_mut())
- {
- first_position.row = line_number;
- }
+ for filename in iter {
+ args.files.push(PathBuf::from(filename));
}
Ok(args)
}
}
-
-/// Parse arg into [`PathBuf`] and position.
-pub(crate) fn parse_file(s: &str) -> (PathBuf, Position) {
- let def = || (PathBuf::from(s), Position::default());
- if Path::new(s).exists() {
- return def();
- }
- split_path_row_col(s)
- .or_else(|| split_path_row(s))
- .unwrap_or_else(def)
-}
-
-/// Split file.rs:10:2 into [`PathBuf`], row and col.
-///
-/// Does not validate if file.rs is a file or directory.
-fn split_path_row_col(s: &str) -> Option<(PathBuf, Position)> {
- let mut s = s.trim_end_matches(':').rsplitn(3, ':');
- let col: usize = s.next()?.parse().ok()?;
- let row: usize = s.next()?.parse().ok()?;
- let path = s.next()?.into();
- let pos = Position::new(row.saturating_sub(1), col.saturating_sub(1));
- Some((path, pos))
-}
-
-/// Split file.rs:10 into [`PathBuf`] and row.
-///
-/// Does not validate if file.rs is a file or directory.
-fn split_path_row(s: &str) -> Option<(PathBuf, Position)> {
- let (path, row) = s.trim_end_matches(':').rsplit_once(':')?;
- let row: usize = row.parse().ok()?;
- let path = path.into();
- let pos = Position::new(row.saturating_sub(1), 0);
- Some((path, pos))
-}