Stop using StringVectorPtr

Use vector<string> instead, because there is no need to use a shared_ptr to maintain a vector