Should use std::string