Fix all current warnings

schala-lang/language/src/ast.rs

@@ -152,9 +152,11 @@ impl Expression {
     Expression { id, kind, type_anno: None }
   }
 
+  /* - commented out because unused
   pub fn with_anno(id: ItemId, kind: ExpressionKind, type_anno: TypeIdentifier) -> Expression {
     Expression { id, kind, type_anno: Some(type_anno) }
   }
+  */
 }
 
 #[derive(Debug, PartialEq, Clone)]

schala-lang/language/src/eval.rs

@@ -2,10 +2,10 @@ use std::rc::Rc;
 use std::fmt::Write;
 use std::io;
 
-use crate::schala::SymbolTableHandle;
+//use crate::schala::SymbolTableHandle;
 use crate::util::ScopeStack;
 use crate::reduced_ast::{BoundVars, ReducedAST, Stmt, Expr, Lit, Func, Alternative, Subpattern};
-use crate::symbol_table::{SymbolSpec, Symbol, SymbolTable, FullyQualifiedSymbolName};
+//use crate::symbol_table::{SymbolSpec, Symbol, SymbolTable, FullyQualifiedSymbolName};
 use crate::builtin::Builtin;
 
 mod test;

schala-lang/language/src/reduced_ast.rs

@@ -155,7 +155,6 @@ impl<'a> Reducer<'a> {
 
   fn expression(&mut self, expr: &Expression) -> Expr {
     use crate::ast::ExpressionKind::*;
-    let symbol_table = self.symbol_table;
     let ref input = expr.kind;
     match input {
       NatLiteral(n) => Expr::Lit(Lit::Nat(*n)),

schala-lang/language/src/schala.rs

@@ -18,6 +18,7 @@ pub type SourceMapHandle = Rc<RefCell<source_map::SourceMap>>;
 /// All the state necessary to parse and execute a Schala program are stored in this struct.
 /// `state` represents the execution state for the AST-walking interpreter, the other fields
 /// should be self-explanatory.
+#[allow(dead_code)]
 pub struct Schala {
   source_reference: SourceReference,
   source_map: SourceMapHandle,

schala-lang/language/src/symbol_table.rs

@@ -84,6 +84,7 @@ impl fmt::Display for ScopeSegment {
 }
 
 impl ScopeSegment {
+  #[allow(dead_code)]
   pub fn new(name: Rc<String>) ->  ScopeSegment {
     ScopeSegment { name }
   }
@@ -134,6 +135,7 @@ impl SymbolTable {
   }
 }
 
+#[allow(dead_code)]
 #[derive(Debug)]
 pub struct Symbol {
   pub local_name: Rc<String>, //TODO does this need to be pub?