From 6595c329a89242ca68619b9a32b048dc7f76e65a Mon Sep 17 00:00:00 2001
From: Bill Somerville <g4wjs@classdesign.com>
Date: Tue, 5 Jan 2016 17:29:24 +0000
Subject: [PATCH] Do not try to open a missing Hamlib settings file

git-svn-id: svn+ssh://svn.code.sf.net/p/wsjt/wsjt/branches/wsjtx@6349 ab8295b8-cf94-4d9e-aec4-7959e3be5d79
---
 HamlibTransceiver.cpp | 68 +++++++++++++++++++++++--------------------
 1 file changed, 36 insertions(+), 32 deletions(-)

diff --git a/HamlibTransceiver.cpp b/HamlibTransceiver.cpp
index 17a58ab39..344882db9 100644
--- a/HamlibTransceiver.cpp
+++ b/HamlibTransceiver.cpp
@@ -240,45 +240,49 @@ HamlibTransceiver::HamlibTransceiver (int model_number, TransceiverFactory::Para
     //
     // user defined Hamlib settings
     //
-    QFile settings_file {QStandardPaths::locate (
+    auto settings_file_name = QStandardPaths::locate (
 #if QT_VERSION >= 0x050500
                                                  QStandardPaths::AppConfigLocation
 #else
                                                  QStandardPaths::ConfigLocation
 #endif
-                                                 , "hamlib_settings.json")};
-    if (settings_file.open (QFile::ReadOnly))
+                                                 , "hamlib_settings.json");
+    if (!settings_file_name.isEmpty ())
       {
-        QJsonParseError status;
-        auto settings_doc = QJsonDocument::fromJson (settings_file.readAll (), &status);
-        if (status.error)
-          {
-            throw error {tr ("Hamlib settings file error: %1 at character offset %2")
-                .arg (status.errorString ()).arg (status.offset)};
-          }
-        if (!settings_doc.isObject ())
-          {
-            throw error {tr ("Hamlib settings file error: top level must be a JSON object")};
-          }
-        auto const& settings = settings_doc.object ();
+	QFile settings_file {settings_file_name};
+	if (settings_file.open (QFile::ReadOnly))
+	  {
+	    QJsonParseError status;
+	    auto settings_doc = QJsonDocument::fromJson (settings_file.readAll (), &status);
+	    if (status.error)
+	      {
+		throw error {tr ("Hamlib settings file error: %1 at character offset %2")
+		    .arg (status.errorString ()).arg (status.offset)};
+	      }
+	    if (!settings_doc.isObject ())
+	      {
+		throw error {tr ("Hamlib settings file error: top level must be a JSON object")};
+	      }
+	    auto const& settings = settings_doc.object ();
 
-        //
-        // configuration settings
-        //
-        auto const& config = settings["config"];
-        if (!config.isUndefined ())
-          {
-            if (!config.isObject ())
-              {
-                throw error {tr ("Hamlib settings file error: config must be a JSON object")};
-              }
-            auto const& config_list = config.toObject ();
-            for (auto item = config_list.constBegin (); item != config_list.constEnd (); ++item)
-              {
-                set_conf (item.key ().toLocal8Bit ().constData ()
-                          , (*item).toVariant ().toString ().toLocal8Bit ().constData ());
-              }
-          }
+	    //
+	    // configuration settings
+	    //
+	    auto const& config = settings["config"];
+	    if (!config.isUndefined ())
+	      {
+		if (!config.isObject ())
+		  {
+		    throw error {tr ("Hamlib settings file error: config must be a JSON object")};
+		  }
+		auto const& config_list = config.toObject ();
+		for (auto item = config_list.constBegin (); item != config_list.constEnd (); ++item)
+		  {
+		    set_conf (item.key ().toLocal8Bit ().constData ()
+			      , (*item).toVariant ().toString ().toLocal8Bit ().constData ());
+		  }
+	      }
+	  }
       }
   }