chore: improve error, use ++depth

Author: diegomarquezp

google/cloud/bigtable/value.cc

@@ -340,7 +340,8 @@ std::ostream& operator<<(std::ostream& os, Value const& v) {
 }
 
 Status MakeDepthExceededError() {
-  return internal::InternalError("Nested value depth exceeds 10 levels");
+  return internal::InternalError("Nested value depth exceeds 10 levels",
+                                 GCP_ERROR_INFO());
 }
 
 // NOLINTNEXTLINE(misc-no-recursion)
@@ -357,7 +358,7 @@ Status TypeAndArrayValuesMatch(google::bigtable::v2::Type const& type,
   auto const& vals = value.array_value().values();
   for (auto const& val : vals) {
     auto const element_match_result = Value::TypeAndValuesMatch(
-        type.array_type().element_type(), val, depth + 1);
+        type.array_type().element_type(), val, ++depth);
     if (!element_match_result.ok()) {
       return element_match_result;
     }
@@ -388,13 +389,13 @@ Status TypeAndMapValuesMatch(google::bigtable::v2::Type const& type,
     auto map_value = val.array_value().values(1);
     // NOLINTNEXTLINE(misc-no-recursion)
     auto key_match_result =
-        Value::TypeAndValuesMatch(key_type, map_key, depth + 1);
+        Value::TypeAndValuesMatch(key_type, map_key, ++depth);
     if (!key_match_result.ok()) {
       return key_match_result;
     }
     // NOLINTNEXTLINE(misc-no-recursion)
     auto value_match_result =
-        Value::TypeAndValuesMatch(value_type, map_value, depth + 1);
+        Value::TypeAndValuesMatch(value_type, map_value, ++depth);
     if (!value_match_result.ok()) {
       return value_match_result;
     }
@@ -424,7 +425,7 @@ Status TypeAndStructValuesMatch(google::bigtable::v2::Type const& type,
   for (int i = 0; i < fields.size(); ++i) {
     auto const& f1 = fields.Get(i);
     auto const& v = values[i];
-    auto match_result = Value::TypeAndValuesMatch(f1.type(), v, depth + 1);
+    auto match_result = Value::TypeAndValuesMatch(f1.type(), v, ++depth);
     if (!match_result.ok()) {
       return match_result;
     }
@@ -458,13 +459,13 @@ Status Value::TypeAndValuesMatch(google::bigtable::v2::Type const& type,
   Status result;
   switch (type.kind_case()) {
     case Type::kArrayType:
-      result = TypeAndArrayValuesMatch(type, value, depth + 1);
+      result = TypeAndArrayValuesMatch(type, value, ++depth);
       break;
     case Type::kMapType:
-      result = TypeAndMapValuesMatch(type, value, depth + 1);
+      result = TypeAndMapValuesMatch(type, value, ++depth);
       break;
     case Type::kStructType:
-      result = TypeAndStructValuesMatch(type, value, depth + 1);
+      result = TypeAndStructValuesMatch(type, value, ++depth);
       break;
     case Type::kBoolType:
       if (!value.has_bool_value()) {