From 7ec99e116a5d3736636a33e769863021df495c61 Mon Sep 17 00:00:00 2001
From: Richard Cohen <richard@daijobu.co.uk>
Date: Wed, 1 Mar 2023 18:22:10 +0000
Subject: [PATCH] Fix  CoreApp() constructor misuse

- spotted by [bugprone-undelegated-constructor] [bugprone-unused-raii]

gnucash/gnucash-core-app.cpp:218:5: warning: did you intend to call a delegated constructor? A temporary object is created here instead [bugprone-undelegated-constructor]
    CoreApp();
    ^
gnucash/gnucash-core-app.cpp:218:5: warning: object destroyed immediately after creation; did you mean to name the object? [bugprone-unused-raii]
    CoreApp();
    ^      ~~
            give_me_a_name
---
 gnucash/gnucash-core-app.cpp | 10 +---------
 gnucash/gnucash-core-app.hpp |  1 -
 2 files changed, 1 insertion(+), 10 deletions(-)

diff --git a/gnucash/gnucash-core-app.cpp b/gnucash/gnucash-core-app.cpp
index 1687293b14..f307e6ab94 100644
--- a/gnucash/gnucash-core-app.cpp
+++ b/gnucash/gnucash-core-app.cpp
@@ -167,7 +167,7 @@ gnc_log_init (const std::vector <std::string> log_flags,
     }
 }
 
-Gnucash::CoreApp::CoreApp ()
+Gnucash::CoreApp::CoreApp (const char* app_name) : m_app_name {app_name}
 {
     #ifdef ENABLE_BINRELOC
     {
@@ -210,14 +210,6 @@ Gnucash::CoreApp::CoreApp ()
     std::cerr.imbue (gnc_get_boost_locale());
     std::cout.imbue (gnc_get_boost_locale());
     g_free(localedir);
-}
-
-Gnucash::CoreApp::CoreApp (const char* app_name)
-{
-
-    CoreApp();
-
-    m_app_name = std::string(app_name);
 
     // Now that gettext is properly initialized, set our help tagline.
     m_tagline = _("- GnuCash, accounting for personal and small business finance");
diff --git a/gnucash/gnucash-core-app.hpp b/gnucash/gnucash-core-app.hpp
index b7b1bf40dc..4a4ded5102 100644
--- a/gnucash/gnucash-core-app.hpp
+++ b/gnucash/gnucash-core-app.hpp
@@ -39,7 +39,6 @@ namespace bpo = boost::program_options;
 class CoreApp
 {
 public:
-    CoreApp ();
     CoreApp (const char* app_name);
 
     void parse_command_line (int argc, char **argv);