Add _t suffix to class binary_param

Author: joto

src/gen/gen-rivers.cpp

@@ -345,7 +345,8 @@ SELECT "{id_column}", "{width_column}", "{name_column}", "{geom_column}"
         geom::geometry_t const geom{std::move(edge.points), PROJ_SPHERE_MERC};
         auto const wkb = geom_to_ewkb(geom);
         connection().exec_prepared("ins", edge.id, edge.width,
-                                   get_name(names, edge.id), binary_param(wkb));
+                                   get_name(names, edge.id),
+                                   binary_param_t(wkb));
     }
     connection().exec("COMMIT");
     timer(m_timer_write).stop();

src/gen/gen-tile-raster.cpp

@@ -237,7 +237,7 @@ void gen_tile_raster_union_t::process(tile_t const &tile)
         timer(m_timer_write).start();
         for (auto const &geom : geometries) {
             auto const wkb = geom_to_ewkb(geom);
-            connection().exec_prepared("insert_geoms", binary_param{wkb},
+            connection().exec_prepared("insert_geoms", binary_param_t{wkb},
                                        tile.x(), tile.y(), param);
         }
         timer(m_timer_write).stop();

src/pgsql.hpp

@@ -138,12 +138,12 @@ class pg_result_t
  * Wrapper class for query parameters that should be sent to the database
  * as binary parameter.
  */
-class binary_param : public std::string_view
+class binary_param_t : public std::string_view
 {
 public:
     using std::string_view::string_view;
 
-    explicit binary_param(std::string const &str)
+    explicit binary_param_t(std::string const &str)
     : std::string_view(str.data(), str.size())
     {}
 };
@@ -272,7 +272,7 @@ class pg_conn_t
     {
         if constexpr (std::is_same_v<T, char const *> ||
                       std::is_same_v<T, std::string> ||
-                      std::is_same_v<T, binary_param>) {
+                      std::is_same_v<T, binary_param_t>) {
             return 0;
         }
         return 1;
@@ -293,7 +293,7 @@ class pg_conn_t
         } else if constexpr (std::is_same_v<T, std::string>) {
             *length = static_cast<int>(param.size());
             return param.c_str();
-        } else if constexpr (std::is_same_v<T, binary_param>) {
+        } else if constexpr (std::is_same_v<T, binary_param_t>) {
             *length = static_cast<int>(param.size());
             *bin = 1;
             return param.data();

tests/test-pgsql.cpp

@@ -105,7 +105,7 @@ TEST_CASE("exec_prepared with binary parameter should work")
     auto const conn = db.db().connect();
     conn.exec("PREPARE test(bytea) AS SELECT length($1)");
 
-    binary_param const p{"foo \x01 bar"};
+    binary_param_t const p{"foo \x01 bar"};
     auto const result = conn.exec_prepared("test", p);
     REQUIRE(result.status() == PGRES_TUPLES_OK);
     REQUIRE(result.num_fields() == 1);
@@ -120,7 +120,7 @@ TEST_CASE("exec_prepared with mixed parameter types should work")
               " SELECT length($1) + length($2) + $3");
 
     std::string const p1{"foo bar"};
-    binary_param const p2{"foo \x01 bar"};
+    binary_param_t const p2{"foo \x01 bar"};
     int const p3 = 17;
     auto const result = conn.exec_prepared("test", p1, p2, p3);
     REQUIRE(result.status() == PGRES_TUPLES_OK);