summaryrefslogtreecommitdiff
path: root/absl/status
diff options
context:
space:
mode:
Diffstat (limited to 'absl/status')
-rw-r--r--absl/status/internal/status_internal.h8
-rw-r--r--absl/status/status.cc5
2 files changed, 6 insertions, 7 deletions
diff --git a/absl/status/internal/status_internal.h b/absl/status/internal/status_internal.h
index fc1e78bc..19a4a7aa 100644
--- a/absl/status/internal/status_internal.h
+++ b/absl/status/internal/status_internal.h
@@ -15,6 +15,7 @@
#define ABSL_STATUS_INTERNAL_STATUS_INTERNAL_H_
#include <string>
+#include <utility>
#include "absl/base/attributes.h"
#include "absl/container/inlined_vector.h"
@@ -70,12 +71,11 @@ struct StatusRep {
absl::StatusCode MapToLocalCode(int value);
-// If `status` is not OK, returns a pointer to a newly-allocated string with the
-// given `prefix`, suitable for output as an error message in assertion/CHECK()
-// failures. Otherwise returns nullptr.
+// Returns a pointer to a newly-allocated string with the given `prefix`,
+// suitable for output as an error message in assertion/`CHECK()` failures.
//
// This is an internal implementation detail for Abseil logging.
-std::string* MakeCheckFailString(const absl::Status& status,
+std::string* MakeCheckFailString(const absl::Status* status,
const char* prefix);
} // namespace status_internal
diff --git a/absl/status/status.cc b/absl/status/status.cc
index 89749a52..88e8eda9 100644
--- a/absl/status/status.cc
+++ b/absl/status/status.cc
@@ -603,12 +603,11 @@ Status ErrnoToStatus(int error_number, absl::string_view message) {
namespace status_internal {
-std::string* MakeCheckFailString(const absl::Status& status,
+std::string* MakeCheckFailString(const absl::Status* status,
const char* prefix) {
- if (status.ok()) { return nullptr; }
return new std::string(
absl::StrCat(prefix, " (",
- status.ToString(StatusToStringMode::kWithEverything), ")"));
+ status->ToString(StatusToStringMode::kWithEverything), ")"));
}
} // namespace status_internal