From 1ecf690259c3d8ba6c34d0bd18178129bd2b9344 Mon Sep 17 00:00:00 2001 From: rsgowman Date: Tue, 27 Feb 2018 14:33:46 -0500 Subject: Add FieldValue.boolean_value() (#862) --- Firestore/core/src/firebase/firestore/model/field_value.h | 5 +++++ Firestore/core/src/firebase/firestore/remote/serializer.cc | 7 +------ 2 files changed, 6 insertions(+), 6 deletions(-) (limited to 'Firestore/core/src') diff --git a/Firestore/core/src/firebase/firestore/model/field_value.h b/Firestore/core/src/firebase/firestore/model/field_value.h index 15945b9..72b9481 100644 --- a/Firestore/core/src/firebase/firestore/model/field_value.h +++ b/Firestore/core/src/firebase/firestore/model/field_value.h @@ -96,6 +96,11 @@ class FieldValue { return tag_; } + bool boolean_value() const { + FIREBASE_ASSERT(tag_ == Type::Boolean); + return boolean_value_; + } + int64_t integer_value() const { FIREBASE_ASSERT(tag_ == Type::Integer); return integer_value_; diff --git a/Firestore/core/src/firebase/firestore/remote/serializer.cc b/Firestore/core/src/firebase/firestore/remote/serializer.cc index 02013fa..79ed98d 100644 --- a/Firestore/core/src/firebase/firestore/remote/serializer.cc +++ b/Firestore/core/src/firebase/firestore/remote/serializer.cc @@ -131,12 +131,7 @@ void Serializer::EncodeFieldValue(const FieldValue& field_value, // TODO(rsgowman): figure out error handling abort(); } - if (field_value == FieldValue::TrueValue()) { - EncodeBool(&stream, true); - } else { - FIREBASE_DEV_ASSERT(field_value == FieldValue::FalseValue()); - EncodeBool(&stream, false); - } + EncodeBool(&stream, field_value.boolean_value()); break; case FieldValue::Type::Integer: -- cgit v1.2.3