Skip to content

Commit 54ea689

Browse files
authored
Merge pull request #1816 from joto/flex-cleanup
Consistently use uppercase "Table" as Lua "class" name
2 parents 85a657b + 473793a commit 54ea689

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/output-flex.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1230,7 +1230,7 @@ flex_table_t const &output_flex_t::get_table_from_param()
12301230
{
12311231
if (lua_gettop(lua_state()) != 1) {
12321232
throw std::runtime_error{
1233-
"Need exactly one parameter of type osm2pgsql.table."};
1233+
"Need exactly one parameter of type osm2pgsql.Table."};
12341234
}
12351235

12361236
auto const &table = m_tables->at(table_idx_from_param(lua_state()));
@@ -1242,7 +1242,7 @@ int output_flex_t::table_tostring()
12421242
{
12431243
auto const &table = get_table_from_param();
12441244

1245-
std::string const str{"osm2pgsql.table[{}]"_format(table.name())};
1245+
std::string const str{"osm2pgsql.Table[{}]"_format(table.name())};
12461246
lua_pushstring(lua_state(), str.c_str());
12471247

12481248
return 1;
@@ -1342,14 +1342,14 @@ int output_flex_t::table_add_row()
13421342
auto const num_params = lua_gettop(lua_state());
13431343
if (num_params < 1 || num_params > 2) {
13441344
throw std::runtime_error{
1345-
"Need two parameters: The osm2pgsql.table and the row data."};
1345+
"Need two parameters: The osm2pgsql.Table and the row data."};
13461346
}
13471347

13481348
auto &table_connection =
13491349
m_table_connections.at(table_idx_from_param(lua_state()));
13501350
auto const &table = table_connection.table();
13511351

1352-
// It there is a second parameter, it must be a Lua table.
1352+
// If there is a second parameter, it must be a Lua table.
13531353
if (num_params == 2) {
13541354
luaL_checktype(lua_state(), 2, LUA_TTABLE);
13551355
}
@@ -1423,7 +1423,7 @@ int output_flex_t::table_insert()
14231423
auto const num_params = lua_gettop(lua_state());
14241424
if (num_params != 2) {
14251425
throw std::runtime_error{
1426-
"Need two parameters: The osm2pgsql.table and the row data."};
1426+
"Need two parameters: The osm2pgsql.Table and the row data."};
14271427
}
14281428

14291429
// The first parameter is the table object.
@@ -1651,7 +1651,7 @@ void output_flex_t::add_row(table_connection_t *table_connection,
16511651
// must be present.
16521652
if (lua_gettop(lua_state()) == 0) {
16531653
throw std::runtime_error{
1654-
"Need two parameters: The osm2pgsql.table and the row data."};
1654+
"Need two parameters: The osm2pgsql.Table and the row data."};
16551655
}
16561656

16571657
auto const geom_transform = get_transform(lua_state(), table.geom_column());

0 commit comments

Comments
 (0)