cleanup
use reference when passing string