From d9d0d63572b328c428d780dda7adf0a2927aedf2 Mon Sep 17 00:00:00 2001
From: Jochen Topf <jochen@topf.org>
Date: Thu, 14 Dec 2023 15:50:15 +0100
Subject: [PATCH 1/2] Remove need for style_set boolean

---
 src/command-line-parser.cpp | 1 -
 src/options.hpp             | 1 -
 src/osm2pgsql.cpp           | 2 +-
 3 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/command-line-parser.cpp b/src/command-line-parser.cpp
index 79c1749f1..d49733501 100644
--- a/src/command-line-parser.cpp
+++ b/src/command-line-parser.cpp
@@ -617,7 +617,6 @@ options_t parse_command_line(int argc, char *argv[])
             break;
         case 'S': // --style
             options.style = optarg;
-            options.style_set = true;
             break;
         case 'i': // --tablespace-index
             options.tblsmain_index = optarg;
diff --git a/src/options.hpp b/src/options.hpp
index dea7367a0..a0e455f3f 100644
--- a/src/options.hpp
+++ b/src/options.hpp
@@ -164,7 +164,6 @@ struct options_t
     bool pass_prompt = false;
 
     bool output_backend_set = false;
-    bool style_set = false;
 }; // struct options_t
 
 #endif // OSM2PGSQL_OPTIONS_HPP
diff --git a/src/osm2pgsql.cpp b/src/osm2pgsql.cpp
index 656954454..075b2d77f 100644
--- a/src/osm2pgsql.cpp
+++ b/src/osm2pgsql.cpp
@@ -336,7 +336,7 @@ static void check_for_nodes_table(options_t const &options)
 
 static void check_and_set_style(options_t *options)
 {
-    if (!options->style_set) {
+    if (options->style.empty()) {
         if (options->output_backend == "flex" ||
             options->output_backend == "gazetteer") {
             throw std::runtime_error{"You have to set the config file "

From aa4507fad72bb01c939c1784e0868d06f5c91aa7 Mon Sep 17 00:00:00 2001
From: Jochen Topf <jochen@topf.org>
Date: Thu, 14 Dec 2023 16:28:33 +0100
Subject: [PATCH 2/2] Remove need for output_backend_set boolean

---
 src/command-line-parser.cpp       |  1 -
 src/options.hpp                   |  4 +---
 src/osm2pgsql.cpp                 | 12 ++++++++----
 tests/common-options.hpp          |  1 +
 tests/test-options-projection.cpp |  5 +++--
 5 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/src/command-line-parser.cpp b/src/command-line-parser.cpp
index d49733501..6e63f5c4f 100644
--- a/src/command-line-parser.cpp
+++ b/src/command-line-parser.cpp
@@ -646,7 +646,6 @@ options_t parse_command_line(int argc, char *argv[])
             break;
         case 'O': // --output
             options.output_backend = optarg;
-            options.output_backend_set = true;
             break;
         case 'x': // --extra-attributes
             options.extra_attributes = true;
diff --git a/src/options.hpp b/src/options.hpp
index a0e455f3f..b9f886587 100644
--- a/src/options.hpp
+++ b/src/options.hpp
@@ -92,7 +92,7 @@ struct options_t
     /// File name to output expired tiles list to
     std::string expire_tiles_filename{"dirty_tiles"};
 
-    std::string output_backend{"pgsql"};
+    std::string output_backend;
     std::string input_format; ///< input file format (default: autodetect)
 
     osmium::Box bbox;
@@ -162,8 +162,6 @@ struct options_t
     bool parallel_indexing = true;
     bool create = false;
     bool pass_prompt = false;
-
-    bool output_backend_set = false;
 }; // struct options_t
 
 #endif // OSM2PGSQL_OPTIONS_HPP
diff --git a/src/osm2pgsql.cpp b/src/osm2pgsql.cpp
index 075b2d77f..676a06186 100644
--- a/src/osm2pgsql.cpp
+++ b/src/osm2pgsql.cpp
@@ -256,7 +256,7 @@ static void check_output(properties_t const &properties, options_t *options)
 {
     auto const output = properties.get_string("output", "pgsql");
 
-    if (!options->output_backend_set) {
+    if (options->output_backend.empty()) {
         options->output_backend = output;
         log_info("Using output '{}' (same as on import).", output);
         return;
@@ -334,8 +334,12 @@ static void check_for_nodes_table(options_t const &options)
     }
 }
 
-static void check_and_set_style(options_t *options)
+static void set_option_defaults(options_t *options)
 {
+    if (options->output_backend.empty()) {
+        options->output_backend = "pgsql";
+    }
+
     if (options->style.empty()) {
         if (options->output_backend == "flex" ||
             options->output_backend == "gazetteer") {
@@ -375,7 +379,7 @@ int main(int argc, char *argv[])
             if (properties.load()) {
                 check_and_update_properties(&properties, &options);
             } else {
-                check_and_set_style(&options);
+                set_option_defaults(&options);
                 check_for_nodes_table(options);
             }
 
@@ -393,7 +397,7 @@ int main(int argc, char *argv[])
                 }
             }
         } else {
-            check_and_set_style(&options);
+            set_option_defaults(&options);
             store_properties(&properties, options);
             auto const finfo = run(options);
             store_data_properties(&properties, finfo);
diff --git a/tests/common-options.hpp b/tests/common-options.hpp
index 7c0bb81d3..05050af8d 100644
--- a/tests/common-options.hpp
+++ b/tests/common-options.hpp
@@ -22,6 +22,7 @@ class opt_t
 public:
     opt_t()
     {
+        m_opt.output_backend = "pgsql";
         m_opt.prefix = "osm2pgsql_test";
         m_opt.style = OSM2PGSQLDATA_DIR "default.style";
         m_opt.num_procs = 1;
diff --git a/tests/test-options-projection.cpp b/tests/test-options-projection.cpp
index 90d75a9b6..265770ea0 100644
--- a/tests/test-options-projection.cpp
+++ b/tests/test-options-projection.cpp
@@ -22,8 +22,9 @@ TEST_CASE("Projection setup")
 {
     char const* const style_file = OSM2PGSQLDATA_DIR "default.style";
 
-    std::vector<char const *> option_params = {"osm2pgsql", "-S", style_file,
-                                               "--number-processes", "1"};
+    std::vector<char const *> option_params = {"osm2pgsql", "--output=pgsql",
+                                               "-S", style_file,
+                                               "--number-processes=1"};
 
     std::string proj_name;
     char const *srid = "";