Fix travis issues and remove cel_operations

.travis.yml

@@ -9,8 +9,8 @@ addons:
       - pkg-config
 
 before_install:
-  - wget https://github.com/bazelbuild/bazel/releases/download/0.9.0/bazel_0.9.0-linux-x86_64.deb
-  - sudo dpkg -i bazel_0.9.0-linux-x86_64.deb
+  - wget https://github.com/bazelbuild/bazel/releases/download/0.17.2/bazel_0.17.2-linux-x86_64.deb
+  - sudo dpkg -i bazel_0.17.2-linux-x86_64.deb
 
 script:
   - bazel test ...

WORKSPACE

@@ -6,8 +6,8 @@ http_archive(
 
 http_archive(
     name = "com_google_protobuf",
-    strip_prefix = "protobuf-3.5.0",
-    urls = ["https://github.com/google/protobuf/archive/v3.5.0.zip"],
+    strip_prefix = "protobuf-3.5.1",
+    urls = ["https://github.com/google/protobuf/archive/v3.5.1.zip"],
 )
 
 http_archive(

common/BUILD.bazel

@@ -184,23 +184,10 @@ cc_library(
 cc_test(
     name = "converters_test",
     srcs = ["converters_test.cc"],
-    data = [
-        "@com_google_cel_spec//testdata",
-    ],
     deps = [
         ":converters",
         ":value",
         "@com_google_absl//absl/memory",
         "@com_google_googletest//:gtest_main",
     ],
 )
-
-cc_library(
-    name = "cel_operators",
-    srcs = [
-        "cel_operators.cc",
-    ],
-    hdrs = [
-        "cel_operators.h",
-    ],
-)

common/enum.h

@@ -33,8 +33,8 @@ class UnnamedEnumValue final : public internal::RefCountable {
  public:
   ~UnnamedEnumValue() = default;
 
-  inline constexpr int32_t value() const { return value_; }
-  inline constexpr EnumType type() const { return type_; }
+  inline int32_t value() const { return value_; }
+  inline EnumType type() const { return type_; }
 
   std::string ToString() const;
 

common/type.cc

@@ -10,7 +10,7 @@ namespace expr {
 namespace {
 
 constexpr const std::size_t kBasicTypeNamesSize = 10;
-const auto* kBasicTypeNames = new std::array<std::string, kBasicTypeNamesSize>{
+const auto* kBasicTypeNames = new std::array<std::string, kBasicTypeNamesSize>({
     "null_type",  // kNull
     "bool",       // kBool
     "int",        // kInt
@@ -21,7 +21,7 @@ const auto* kBasicTypeNames = new std::array<std::string, kBasicTypeNamesSize>{
     "type",       // kType
     "map",        // kMap
     "list",       // kList
-};
+});
 
 static_assert(kBasicTypeNamesSize ==
                   static_cast<int>(BasicTypeValue::DO_NOT_USE),

common/type.h

@@ -137,10 +137,10 @@ class Type final {
   inline EnumType enum_type() const { return absl::get<EnumType>(data_); }
   inline ObjectType object_type() const { return absl::get<ObjectType>(data_); }
 
-  constexpr inline bool operator==(const Type& rhs) const {
+  inline bool operator==(const Type& rhs) const {
     return data_ == rhs.data_;
   }
-  constexpr inline bool operator!=(const Type& rhs) const {
+  inline bool operator!=(const Type& rhs) const {
     return data_ != rhs.data_;
   }
 

internal/hash_util.h

@@ -95,7 +95,7 @@ std::size_t HashImpl(const T& value, specialize_for<typename T::Hasher>) {
 
 // Specialization for enums.
 template <typename T>
-specialize_ift<std::is_enum<T>, std::size_t> HashImpl(T&& v) {
+std::size_t HashImpl(T&& v, specialize_ift<std::is_enum<T>>) {
   return Hash(static_cast<typename std::underlying_type<T>::type>(v));
 }