on_success and on_error are documented as returning void

[krwalker]

As a follow-on to #491, the probe_on_success and probe_on_error calls use std::is_same<> return-type checks. They should be updated to check for void, or the documentation should be changed to say they return error_code.