Yet another refactor

Created by: stratact

For the builtin functions, Ion was converting existing &[String] values into Vec<&str> by collecting each item and converting them with as_ref() which makes them &str values, to avoid from more collecting, just pass the &[String] borrows to save from more allocation and when a function needs to index a &[String] just Deref and borrow it.