Small code cleanups in flex output

Author: joto

src/output-flex.cpp

@@ -456,10 +456,9 @@ flex_table_t &output_flex_t::create_flex_table()
         throw fmt_error("Table with name '{}' already exists.", table_name);
     }
 
-    m_tables->emplace_back(table_name);
-    auto &new_table = m_tables->back();
+    auto &new_table = m_tables->emplace_back(table_name);
 
-    lua_pop(lua_state(), 1);
+    lua_pop(lua_state(), 1); // "name"
 
     // optional "schema" field
     lua_getfield(lua_state(), -1, "schema");
@@ -1445,7 +1444,7 @@ output_flex_t::output_flex_t(std::shared_ptr<middle_query_t> const &mid,
         log_debug("- TABLE {}", qualified_name(table.schema(), table.name()));
         log_debug("  - columns:");
         for (auto const &column : table) {
-            log_debug("    - \"{}\" {} ({}) not_null={} create_only={}",
+            log_debug(R"(    - "{}" {} ({}) not_null={} create_only={})",
                       column.name(), column.type_name(), column.sql_type_name(),
                       column.not_null(), column.create_only());
         }

src/output-flex.hpp

@@ -190,6 +190,7 @@ class output_flex_t : public output_t
     void setup_flex_table_columns(flex_table_t *table);
     void setup_indexes(flex_table_t *table);
 
+    // Get the flex table that is as first parameter on the Lua stack.
     flex_table_t const &get_table_from_param();
 
     void check_context_and_state(char const *name, char const *context,