New helper function luaX_pushstring()

Author: joto

src/lua-utils.cpp

@@ -58,6 +58,11 @@ void *luaX_get_context(lua_State *lua_state) noexcept
 
 #endif
 
+void luaX_pushstring(lua_State *lua_state, std::string_view str) noexcept
+{
+    lua_pushlstring(lua_state, str.data(), str.size());
+}
+
 void luaX_add_table_str(lua_State *lua_state, char const *key,
                         char const *value) noexcept
 {

src/lua-utils.hpp

@@ -17,11 +17,14 @@
 
 #include <cassert>
 #include <cstdint>
+#include <string_view>
 #include <utility>
 
 void luaX_set_context(lua_State *lua_state, void *ptr) noexcept;
 void *luaX_get_context(lua_State *lua_state) noexcept;
 
+void luaX_pushstring(lua_State *lua_state, std::string_view str) noexcept;
+
 void luaX_add_table_str(lua_State *lua_state, char const *key,
                         char const *value) noexcept;
 void luaX_add_table_str(lua_State *lua_state, char const *key,

src/output-flex.cpp

@@ -530,7 +530,7 @@ int output_flex_t::table_tostring()
     auto const &table = get_table_from_param();
 
     std::string const str{fmt::format("osm2pgsql.Table[{}]", table.name())};
-    lua_pushstring(lua_state(), str.c_str());
+    luaX_pushstring(lua_state(), str);
 
     return 1;
 }
@@ -689,8 +689,8 @@ int output_flex_t::table_insert()
     } catch (not_null_exception const &e) {
         copy_mgr->rollback_line();
         lua_pushboolean(lua_state(), false);
-        lua_pushstring(lua_state(), "null value in not null column.");
-        lua_pushstring(lua_state(), e.column().name().c_str());
+        lua_pushliteral(lua_state(), "null value in not null column.");
+        luaX_pushstring(lua_state(), e.column().name());
         push_osm_object_to_lua_stack(lua_state(), object);
         table_connection.increment_not_null_error_counter();
         return 4;
@@ -730,14 +730,14 @@ int output_flex_t::table_columns()
 int output_flex_t::table_name()
 {
     auto const &table = get_table_from_param();
-    lua_pushstring(lua_state(), table.name().c_str());
+    luaX_pushstring(lua_state(), table.name());
     return 1;
 }
 
 int output_flex_t::table_schema()
 {
     auto const &table = get_table_from_param();
-    lua_pushstring(lua_state(), table.schema().c_str());
+    luaX_pushstring(lua_state(), table.schema());
     return 1;
 }
 
@@ -758,7 +758,7 @@ int output_flex_t::expire_output_tostring()
                     expire_output.minzoom(), expire_output.maxzoom(),
                     expire_output.filename(), expire_output.schema(),
                     expire_output.table());
-    lua_pushstring(lua_state(), str.c_str());
+    luaX_pushstring(lua_state(), str);
 
     return 1;
 }
@@ -783,23 +783,23 @@ int output_flex_t::expire_output_filename()
 {
     auto const &expire_output = get_expire_output_from_param();
 
-    lua_pushstring(lua_state(), expire_output.filename().c_str());
+    luaX_pushstring(lua_state(), expire_output.filename());
     return 1;
 }
 
 int output_flex_t::expire_output_schema()
 {
     auto const &expire_output = get_expire_output_from_param();
 
-    lua_pushstring(lua_state(), expire_output.schema().c_str());
+    luaX_pushstring(lua_state(), expire_output.schema());
     return 1;
 }
 
 int output_flex_t::expire_output_table()
 {
     auto const &expire_output = get_expire_output_from_param();
 
-    lua_pushstring(lua_state(), expire_output.table().c_str());
+    luaX_pushstring(lua_state(), expire_output.table());
     return 1;
 }
 
@@ -1338,7 +1338,7 @@ void output_flex_t::init_lua(std::string const &filename,
 
     luaX_add_table_int(lua_state(), "stage", 1);
 
-    lua_pushstring(lua_state(), "properties");
+    lua_pushliteral(lua_state(), "properties");
     lua_createtable(lua_state(), 0, (int)properties.size());
     for (auto const &property : properties) {
         luaX_add_table_str(lua_state(), property.first.c_str(),