StringUtils::Tokenize: clarify, simplify and optimize

Variables names were wrong, no need to check "pos" for "string::npos", comments were incorrect (notice `// Find first "non-delimiter"`).
@jmarshallnz ?
This is a change in behaviour, but on checking the callers, those that re-use the vector clear it out before the call, so all good.