From d439bbccd4b90583a89d209d2cc81308aabca8ac Mon Sep 17 00:00:00 2001 From: Gil Date: Tue, 22 May 2018 13:21:08 -0700 Subject: Add a HARD_ASSERT C++ assertion macro (#1304) * Add HARD_ASSERT * Use HARD_ASSERT * Remove FIREBASE_ASSERT * Remove StringPrintf --- Firestore/core/src/firebase/firestore/util/statusor.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'Firestore/core/src/firebase/firestore/util/statusor.cc') diff --git a/Firestore/core/src/firebase/firestore/util/statusor.cc b/Firestore/core/src/firebase/firestore/util/statusor.cc index be1e03a..bbd5781 100644 --- a/Firestore/core/src/firebase/firestore/util/statusor.cc +++ b/Firestore/core/src/firebase/firestore/util/statusor.cc @@ -15,6 +15,7 @@ */ #include "Firestore/core/src/firebase/firestore/util/statusor.h" +#include "Firestore/core/src/firebase/firestore/util/hard_assert.h" namespace firebase { namespace firestore { @@ -24,15 +25,14 @@ namespace internal_statusor { void Helper::HandleInvalidStatusCtorArg(Status* status) { const char* kMessage = "An OK status is not a valid constructor argument to StatusOr"; - FIREBASE_DEV_ASSERT_MESSAGE(false, kMessage); + HARD_FAIL("%s", kMessage); // Fall back to Internal for non-debug builds *status = Status(FirestoreErrorCode::Internal, kMessage); } void Helper::Crash(const Status& status) { - FIREBASE_ASSERT_MESSAGE( - false, "Attempting to fetch value instead of handling error ", - status.ToString().c_str()); + HARD_FAIL("Attempting to fetch value instead of handling error %s", + status.ToString()); } } // namespace internal_statusor -- cgit v1.2.3