diff --git a/src/parser.rs b/src/parser.rs index 6bb1830fa..020929055 100644 --- a/src/parser.rs +++ b/src/parser.rs @@ -35,7 +35,6 @@ use printf::sprintf; use std::cell::{Ref, RefCell, RefMut}; use std::ffi::{CStr, OsStr}; -use std::num::NonZeroU16; use std::os::fd::{AsRawFd, OwnedFd, RawFd}; use std::os::unix::prelude::OsStrExt; use std::rc::{Rc, Weak}; @@ -76,13 +75,8 @@ pub struct Block { /// Name of the file that created this block pub src_filename: Option>, - /// Line number where this block was created, starting from 1. - /// - /// We count lines from 1, so we can use niche optimization to avoid increasing the size when - /// this is wrapped in an `Option`. - /// - /// This will saturate at the 65,535th line of a single fish script. I think that's ok! - pub src_lineno: Option, + /// Line number where this block was created. + pub src_lineno: Option, } impl Block { @@ -131,7 +125,7 @@ pub fn description(&self) -> WString { .to_owned(); if let Some(src_lineno) = self.src_lineno { - result.push_utfstr(&sprintf!(" (line %d)", src_lineno.get())); + result.push_utfstr(&sprintf!(" (line %d)", src_lineno)); } if let Some(src_filename) = &self.src_filename { result.push_utfstr(&sprintf!(" (file %ls)", src_filename)); @@ -827,10 +821,7 @@ pub fn sync_uvars_and_fire(&self, always: bool) { /// Pushes a new block. Returns a pointer to the block, stored in the parser. pub fn push_block(&self, mut block: Block) -> BlockId { - // It's extremely unrealistic to assume a fish script will have more than 65,535 lines. - block.src_lineno = self - .get_lineno() - .and_then(|num| NonZeroU16::new(num.try_into().unwrap_or(u16::MAX))); + block.src_lineno = self.get_lineno(); block.src_filename = self.current_filename(); if block.typ() != BlockType::top { let new_scope = block.typ() == BlockType::function_call { shadows: true }; @@ -1212,7 +1203,7 @@ fn append_block_description_to_stack_trace(parser: &Parser, b: &Block, trace: &m if let Some(file) = b.src_filename.as_ref() { trace.push_utfstr(&sprintf!( "\tcalled on line %d of file %ls\n", - b.src_lineno.map(|n| n.get()).unwrap_or(0), + b.src_lineno.unwrap_or(0), user_presentable_path(file, parser.vars()) )); } else if parser.libdata().pods.within_fish_init {