String vs str

[vimalloc]

I have a feeling I'm doing a lot of pass by values and copying strings around using String::from() all over the place, that could be better handled by passing string references around. Need to look into this more