A better place for checking for missing outputs

Author: joto

src/osmdata.cpp

@@ -22,11 +22,7 @@ osmdata_t::osmdata_t(std::shared_ptr<middle_t> mid,
 : m_mid(mid), m_outs(outs)
 {
     assert(m_mid);
-
-    if (m_outs.empty()) {
-        throw std::runtime_error{"Must have at least one output, "
-                                 "but none have been configured."};
-    }
+    assert(!m_outs.empty());
 
     // Get the "extra_attributes" option from the first output. We expect
     // all others to be the same.

src/output.cpp

@@ -187,6 +187,11 @@ output_t::create_outputs(std::shared_ptr<middle_query_t> const &mid,
             "gazetteer, null, multi].\n"_format(options.output_backend)};
     }
 
+    if (outputs.empty()) {
+        throw std::runtime_error{"Must have at least one output, "
+                                 "but none have been configured."};
+    }
+
     return outputs;
 }