Avoid constructing a string in make_success_code

[hkaiser]

We use make_success_code() all over the place to create an error_code object representing a successful outcome of a function. This currently constructs a std::string (holding "success") which should be avoided as it does not add any information.