Use the T() instead of NULL for the default value.

The template can be specialized on primitives, e.g. double, where
converting NULL will trigger a warning.
This commit is contained in:
Jisi Liu 2016-03-29 15:25:49 -07:00
parent 261ee021f6
commit e164f1083f

View File

@ -224,7 +224,7 @@ inline StatusOr<T>& StatusOr<T>::operator=(const StatusOr<T>& other) {
template<typename T>
template<typename U>
inline StatusOr<T>::StatusOr(const StatusOr<U>& other)
: status_(other.status_), value_(other.status_.ok() ? other.value_ : NULL) {
: status_(other.status_), value_(other.status_.ok() ? other.value_ : T()) {
}
template<typename T>