I'm skeptical about whether these add any perf benefit, and am suspicious that they might actually incur a loss. We should perf test removing these in turn and seeing how that affects performance.

In particular the implicit conversion constructor from Status -> StatusWith could be called on accident by passing Status to a function that takes StatusWith as a parameter and user would not know they were doing something "cold".