From: Joel Holdsworth <redacted>
Date: Fri, 2 Nov 2012 00:11:21 +0000 (+0000)
Subject: Change main() to have only one exit code flow, used also on errors
X-Git-Tag: pulseview-0.1.0~224
X-Git-Url: https://sigrok.org/gitaction?a=commitdiff_plain;h=be03620eb7b765ff01f52d0f8b9e7dac308524c5;p=pulseview.git

Change main() to have only one exit code flow, used also on errors
---

diff --git a/main.cpp b/main.cpp
index b7ba5d89..56b4dd5f 100644
--- a/main.cpp
+++ b/main.cpp
@@ -47,6 +47,7 @@ void usage()
 
 int main(int argc, char *argv[])
 {
+	int ret = 0;
 	QApplication a(argc, argv);
 
 	// Set some application metadata
@@ -87,33 +88,38 @@ int main(int argc, char *argv[])
 	}
 
 	// Initialise libsigrokdecode
-	if (srd_init(NULL) != SRD_OK) {
-		qDebug() << "ERROR: libsigrokdecode init failed.";
-		return 1;
-	}
+	if (srd_init(NULL) == SRD_OK) {
+
+		// Load the protocol decoders
+		srd_decoder_load_all();
+
+		// Initialize all libsigrok drivers
+		sr_dev_driver **const drivers = sr_driver_list();
+		for (sr_dev_driver **driver = drivers; *driver; driver++) {
+			if (sr_driver_init(*driver) != SR_OK) {
+				qDebug("Failed to initialize driver %s",
+					(*driver)->name);
+				ret = 1;
+				break;
+			}
+		}
 
-	// Load the protocol decoders
-	srd_decoder_load_all();
+		if(ret == 0) {
+			// Initialise the main window
+			pv::MainWindow w;
+			w.show();
 
-	// Initialize all libsigrok drivers
-	sr_dev_driver **const drivers = sr_driver_list();
-	for (sr_dev_driver **driver = drivers; *driver; driver++) {
-		if (sr_driver_init(*driver) != SR_OK) {
-			qDebug("Failed to initialize driver %s",
-				(*driver)->name);
-			return 1;
+			// Run the application
+			ret = a.exec();
 		}
-	}
 
-	// Initialise the main window
-	pv::MainWindow w;
-	w.show();
+		// Destroy libsigrokdecode and libsigrok
+		srd_exit();
 
-	// Run the application
-	const int ret = a.exec();
+	} else {
+		qDebug() << "ERROR: libsigrokdecode init failed.";
+	}
 
-	// Destroy libsigrokdecode and libsigrok
-	srd_exit();
 	sr_exit();
 
 	return ret;