Remove FileContents::first_index() and FilePosition::clip().
Neither is necessary any more. FileContents contains an ordinary Vec now, with no movable origin, so its first index is always 0. That makes the lower-bound checking half of FilePosition::clip() meaningless (the usize index is never going to be less than 0), and in fact _all_ of it is unnecessary, because it's only ever called in the constructor of File, which has only just _invented_ the FilePosition's array index field, and it did it based on looking up a string id, which won't have returned an out-of-range value in any case.
Loading
Please register or sign in to comment