Summary

ParmParse::add("key", "value") will result in adding a boolean value (1) due to C++ converting pointer to bool. This is almost certainly not intended. Users must write
add("key", std::string("value") which is cumbersome and easy to forget. Adding explicit const char * -> std::string overload removes this pitfall

Additional background

Checklist

The proposed changes:

• fix a bug or incorrect behavior in AMReX
• add new capabilities to AMReX
• changes answers in the test suite to more than roundoff level
• are likely to significantly affect the results of downstream AMReX users
• include documentation in the code and/or rst files, if appropriate