diff options
author | Bojan Durdevic <bojan88@gmail.com> | 2021-08-22 11:59:24 -0400 |
---|---|---|
committer | David Peter <sharkdp@users.noreply.github.com> | 2021-08-23 19:55:41 +0200 |
commit | c86a1794121ab3460d496a55556abc17b03b71a2 (patch) | |
tree | bde64432568977194c220935d2ee92a8070613af | |
parent | 07487834044a1e24247d4c1631447419fdd463d2 (diff) |
PR comments addressed for line range +delta syntax
-rw-r--r-- | CHANGELOG.md | 2 | ||||
-rw-r--r-- | src/line_range.rs | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md index ec4d9e08..8908b7cc 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -3,7 +3,7 @@ ## Features - `$BAT_CONFIG_DIR` is now a recognized environment variable. It has precedence over `$XDG_CONFIG_HOME`, see #1727 (@billrisher) -- Support for `x:+x` syntax in line ranges (e.g. `20:+10`). See #1810 (@bojan88) +- Support for `x:+delta` syntax in line ranges (e.g. `20:+10`). See #1810 (@bojan88) ## Bugfixes diff --git a/src/line_range.rs b/src/line_range.rs index d014d688..1cd3a20d 100644 --- a/src/line_range.rs +++ b/src/line_range.rs @@ -48,10 +48,10 @@ impl LineRange { 2 => { new_range.lower = line_numbers[0].parse()?; - new_range.upper = if line_numbers[1].bytes().next().unwrap() == b'+' { + new_range.upper = if line_numbers[1].bytes().next() == Some(b'+') { let more_lines = &line_numbers[1][1..] .parse() - .map_err(|_| "Invalid line number after +")?; + .map_err(|_| "Invalid character after +")?; new_range.lower + more_lines } else { line_numbers[1].parse()? |