cleanup or comment char* types in option definition code

The option definition code uses a lot of char* types so either they can be replaced by a std::string or they can't and a comment should be added to warn developers.