For consistency, the parameter should like `FileHandler* handler`, or `const string& str`, because `const string&` is the type, `str` is the name. Same as all pointers and const references.