refactor: use std::make_shared instead of raw new in RestCatalog

Replace raw new with std::make_shared for better exception safety
and modern C++ best practices in RestCatalog::Make().

Using std::make_shared provides:
- Better exception safety (no leak if constructor throws)
- More efficient memory allocation (single allocation for control block and object)
- More concise and readable code
- Consistent with modern C++ guidelines

Author: shangxinli

src/iceberg/catalog/rest/rest_catalog.cc

@@ -107,9 +107,8 @@ Result<std::shared_ptr<RestCatalog>> RestCatalog::Make(
   ICEBERG_ASSIGN_OR_RAISE(auto final_uri, final_config->Uri());
   ICEBERG_RETURN_UNEXPECTED(paths->SetBaseUri(std::string(TrimTrailingSlash(final_uri))));
 
-  return std::shared_ptr<RestCatalog>(
-      new RestCatalog(std::move(final_config), std::move(file_io), std::move(paths),
-                      std::move(endpoints)));
+  return std::make_shared<RestCatalog>(std::move(final_config), std::move(file_io),
+                                       std::move(paths), std::move(endpoints));
 }
 
 RestCatalog::RestCatalog(std::unique_ptr<RestCatalogProperties> config,