Remove assignment operators

PiperOrigin-RevId: 738086887

Author: jcking

common/values/bool_value.h

@@ -20,7 +20,6 @@
 
 #include <ostream>
 #include <string>
-#include <type_traits>
 
 #include "absl/base/nullability.h"
 #include "absl/status/status.h"
@@ -50,13 +49,7 @@ class BoolValue final : private common_internal::ValueMixin<BoolValue> {
   BoolValue& operator=(const BoolValue&) = default;
   BoolValue& operator=(BoolValue&&) = default;
 
-  constexpr explicit BoolValue(bool value) noexcept : value_(value) {}
-
-  template <typename T, typename = std::enable_if_t<std::is_same_v<T, bool>>>
-  BoolValue& operator=(T value) noexcept {
-    value_ = value;
-    return *this;
-  }
+  explicit BoolValue(bool value) noexcept : value_(value) {}
 
   // NOLINTNEXTLINE(google-explicit-constructor)
   operator bool() const noexcept { return value_; }
@@ -110,9 +103,9 @@ inline std::ostream& operator<<(std::ostream& out, BoolValue value) {
   return out << value.DebugString();
 }
 
-constexpr BoolValue FalseValue() { return BoolValue(false); }
+inline BoolValue FalseValue() noexcept { return BoolValue(false); }
 
-constexpr BoolValue TrueValue() { return BoolValue(true); }
+inline BoolValue TrueValue() noexcept { return BoolValue(true); }
 
 }  // namespace cel
 

common/values/double_value.h

@@ -20,7 +20,6 @@
 
 #include <ostream>
 #include <string>
-#include <type_traits>
 
 #include "absl/base/nullability.h"
 #include "absl/status/status.h"
@@ -45,14 +44,6 @@ class DoubleValue final : private common_internal::ValueMixin<DoubleValue> {
 
   explicit DoubleValue(double value) noexcept : value_(value) {}
 
-  template <typename T,
-            typename = std::enable_if_t<std::conjunction_v<
-                std::is_floating_point<T>, std::is_convertible<T, double>>>>
-  DoubleValue& operator=(T value) noexcept {
-    value_ = value;
-    return *this;
-  }
-
   DoubleValue() = default;
   DoubleValue(const DoubleValue&) = default;
   DoubleValue(DoubleValue&&) = default;

common/values/duration_value.h

@@ -55,12 +55,6 @@ class DurationValue final : private common_internal::ValueMixin<DurationValue> {
     ABSL_DCHECK_OK(internal::ValidateDuration(value));
   }
 
-  DurationValue& operator=(absl::Duration value) noexcept {
-    ABSL_DCHECK_OK(internal::ValidateDuration(value));
-    value_ = value;
-    return *this;
-  }
-
   DurationValue() = default;
   DurationValue(const DurationValue&) = default;
   DurationValue(DurationValue&&) = default;

common/values/int_value.h

@@ -21,7 +21,6 @@
 #include <cstdint>
 #include <ostream>
 #include <string>
-#include <type_traits>
 
 #include "absl/base/nullability.h"
 #include "absl/status/status.h"
@@ -47,15 +46,6 @@ class IntValue final : private common_internal::ValueMixin<IntValue> {
 
   explicit IntValue(int64_t value) noexcept : value_(value) {}
 
-  template <typename T,
-            typename = std::enable_if_t<std::conjunction_v<
-                std::is_integral<T>, std::negation<std::is_same<T, bool>>,
-                std::is_convertible<T, int64_t>>>>
-  IntValue& operator=(T value) noexcept {
-    value_ = value;
-    return *this;
-  }
-
   IntValue() = default;
   IntValue(const IntValue&) = default;
   IntValue(IntValue&&) = default;

common/values/timestamp_value.h

@@ -56,12 +56,6 @@ class TimestampValue final
     ABSL_DCHECK_OK(internal::ValidateTimestamp(value));
   }
 
-  TimestampValue& operator=(absl::Time value) noexcept {
-    ABSL_DCHECK_OK(internal::ValidateTimestamp(value));
-    value_ = value;
-    return *this;
-  }
-
   TimestampValue() = default;
   TimestampValue(const TimestampValue&) = default;
   TimestampValue(TimestampValue&&) = default;

common/values/uint_value.h

@@ -21,7 +21,6 @@
 #include <cstdint>
 #include <ostream>
 #include <string>
-#include <type_traits>
 
 #include "absl/base/nullability.h"
 #include "absl/status/status.h"
@@ -47,15 +46,6 @@ class UintValue final : private common_internal::ValueMixin<UintValue> {
 
   explicit UintValue(uint64_t value) noexcept : value_(value) {}
 
-  template <typename T,
-            typename = std::enable_if_t<std::conjunction_v<
-                std::is_integral<T>, std::negation<std::is_same<T, bool>>,
-                std::is_convertible<T, int64_t>>>>
-  UintValue& operator=(T value) noexcept {
-    value_ = value;
-    return *this;
-  }
-
   UintValue() = default;
   UintValue(const UintValue&) = default;
   UintValue(UintValue&&) = default;