PR comments addressed for line range +delta syntax
commitc86a1794121ab3460d496a55556abc17b03b71a2
authorBojan Durdevic <bojan88@gmail.com>
dateSun 22 Aug 2021, 11:59 (5 years ago)
parent0748783
CHANGELOG.md+1−1
@@ -3,7 +3,7 @@
33## Features
44
55- `$BAT_CONFIG_DIR` is now a recognized environment variable. It has precedence over `$XDG_CONFIG_HOME`, see #1727 (@billrisher)
6−- Support for `x:+x` syntax in line ranges (e.g. `20:+10`). See #1810 (@bojan88)
6+- Support for `x:+delta` syntax in line ranges (e.g. `20:+10`). See #1810 (@bojan88)
77
88## Bugfixes
99
src/line_range.rs+2−2
@@ -48,10 +48,10 @@ impl LineRange {
4848 2 => {
4949 new_range.lower = line_numbers[0].parse()?;
5050
51− new_range.upper = if line_numbers[1].bytes().next().unwrap() == b'+' {
51+ new_range.upper = if line_numbers[1].bytes().next() == Some(b'+') {
5252 let more_lines = &line_numbers[1][1..]
5353 .parse()
54− .map_err(|_| "Invalid line number after +")?;
54+ .map_err(|_| "Invalid character after +")?;
5555 new_range.lower + more_lines
5656 } else {
5757 line_numbers[1].parse()?