refactor: Removed Position from Token, made fields private

Turns out the position was never read from `Token`,
it is always picked up from the `RcStringView`
with `with_pos()`.

Author: ngeor

rusty_basic/src/bin/rusty_basic.rs

@@ -2,9 +2,9 @@ use std::env;
 use std::fs::File;
 
 use rusty_basic::instruction_generator::{generate_instructions, unwrap_linter_context};
-use rusty_basic::interpreter::{new_default_interpreter, InterpreterTrait};
-use rusty_linter::{lint, Context};
-use rusty_parser::{parse_main_file, Program};
+use rusty_basic::interpreter::{InterpreterTrait, new_default_interpreter};
+use rusty_linter::{Context, lint};
+use rusty_parser::{Program, parse_main_file};
 
 fn main() {
     let is_running_in_apache = is_running_in_apache();

rusty_basic/src/instruction_generator/instruction_generator.rs

@@ -5,11 +5,11 @@ use rusty_parser::{
 };
 use rusty_variant::Variant;
 
+use crate::RuntimeError;
 use crate::instruction_generator::label_resolver::LabelResolver;
 use crate::instruction_generator::subprogram_info::{
     SubprogramInfoCollector, SubprogramInfoRepository
 };
-use crate::RuntimeError;
 
 pub fn unwrap_linter_context(linter_context: Context) -> (Names, UserDefinedTypes) {
     let (pre_linter_result, linter_names) = linter_context.into();

rusty_basic/src/instruction_generator/test_utils.rs

@@ -1,9 +1,9 @@
 use rusty_common::NoPosContainer;
 use rusty_linter::lint;
-use rusty_parser::{parse, UserDefinedTypes};
+use rusty_parser::{UserDefinedTypes, parse};
 
 use crate::instruction_generator::{
-    generate_instructions, unwrap_linter_context, Instruction, InstructionGeneratorResult, InstructionPos
+    Instruction, InstructionGeneratorResult, InstructionPos, generate_instructions, unwrap_linter_context
 };
 
 pub fn generate_instructions_str_with_types(

rusty_basic/src/interpreter/built_ins/beep.rs

@@ -1,5 +1,5 @@
-use crate::interpreter::interpreter_trait::InterpreterTrait;
 use crate::RuntimeError;
+use crate::interpreter::interpreter_trait::InterpreterTrait;
 
 #[cfg(windows)]
 pub fn run<S: InterpreterTrait>(_interpreter: &mut S) -> Result<(), RuntimeError> {

rusty_basic/src/interpreter/built_ins/chr.rs

@@ -1,8 +1,8 @@
 use rusty_linter::QBNumberCast;
 use rusty_parser::BuiltInFunction;
 
-use crate::interpreter::interpreter_trait::InterpreterTrait;
 use crate::RuntimeError;
+use crate::interpreter::interpreter_trait::InterpreterTrait;
 
 pub fn run<S: InterpreterTrait>(interpreter: &mut S) -> Result<(), RuntimeError> {
     let i: i32 = interpreter.context()[0].try_cast()?;

rusty_basic/src/interpreter/built_ins/close.rs

@@ -1,8 +1,8 @@
 use rusty_parser::FileHandle;
 
+use crate::RuntimeError;
 use crate::interpreter::interpreter_trait::InterpreterTrait;
 use crate::interpreter::variant_casts::VariantCasts;
-use crate::RuntimeError;
 
 pub fn run<S: InterpreterTrait>(interpreter: &mut S) -> Result<(), RuntimeError> {
     let file_handles: Vec<FileHandle> = interpreter

rusty_basic/src/interpreter/built_ins/cls.rs

@@ -1,6 +1,6 @@
+use crate::RuntimeError;
 use crate::interpreter::interpreter_trait::InterpreterTrait;
 use crate::interpreter::io::Printer;
-use crate::RuntimeError;
 
 pub fn run<S: InterpreterTrait>(interpreter: &mut S) -> Result<(), RuntimeError> {
     match interpreter.screen().get_view_print() {

rusty_basic/src/interpreter/built_ins/color.rs

@@ -1,7 +1,7 @@
 use rusty_linter::QBNumberCast;
 
-use crate::interpreter::interpreter_trait::InterpreterTrait;
 use crate::RuntimeError;
+use crate::interpreter::interpreter_trait::InterpreterTrait;
 
 pub fn run<S: InterpreterTrait>(interpreter: &mut S) -> Result<(), RuntimeError> {
     let flags: i32 = interpreter.context()[0].try_cast()?;

rusty_basic/src/interpreter/built_ins/cvd.rs

@@ -1,10 +1,10 @@
 use rusty_parser::BuiltInFunction;
 use rusty_variant::bytes_to_f64;
 
+use crate::RuntimeError;
 use crate::interpreter::interpreter_trait::InterpreterTrait;
 use crate::interpreter::string_utils::to_ascii_bytes;
 use crate::interpreter::variant_casts::VariantCasts;
-use crate::RuntimeError;
 
 pub fn run<S: InterpreterTrait>(interpreter: &mut S) -> Result<(), RuntimeError> {
     let s = interpreter.context()[0].to_str_unchecked();

rusty_basic/src/interpreter/built_ins/data.rs

@@ -1,7 +1,7 @@
 use rusty_variant::Variant;
 
-use crate::interpreter::interpreter_trait::InterpreterTrait;
 use crate::RuntimeError;
+use crate::interpreter::interpreter_trait::InterpreterTrait;
 
 pub fn run<S: InterpreterTrait>(interpreter: &mut S) -> Result<(), RuntimeError> {
     let variables: Vec<Variant> = interpreter