fix export and switch case

Author: wgtmac

src/iceberg/expression/expression.cc

@@ -173,9 +173,18 @@ Result<Expression::Operation> Negate(Expression::Operation op) {
       return Expression::Operation::kNotStartsWith;
     case Expression::Operation::kNotStartsWith:
       return Expression::Operation::kStartsWith;
-    default:
+    case Expression::Operation::kTrue:
+    case Expression::Operation::kFalse:
+    case Expression::Operation::kNot:
+    case Expression::Operation::kCountStar:
+    case Expression::Operation::kMax:
+    case Expression::Operation::kMin:
+    case Expression::Operation::kAnd:
+    case Expression::Operation::kOr:
+    case Expression::Operation::kCount:
       return InvalidArgument("No negation for operation: {}", op);
   }
+  std::unreachable();
 }
 
 }  // namespace iceberg

src/iceberg/expression/expression.h

@@ -198,6 +198,6 @@ class ICEBERG_EXPORT Or : public Expression {
 ICEBERG_EXPORT std::string_view ToString(Expression::Operation op);
 
 /// \brief Returns the negated operation.
-Result<Expression::Operation> Negate(Expression::Operation op);
+ICEBERG_EXPORT Result<Expression::Operation> Negate(Expression::Operation op);
 
 }  // namespace iceberg