Always return true if AppInitMain got to the end

TheBlueMatt · TheBlueMatt · commit 07c483895d21 · 2017-12-05T10:39:44.000-05:00
This should fix a very rare travis failure in zapwallettxes, but
is also more correct, as you can currently race
ReacceptWalletTransactions with stop RPC calls to get bitcoind to
(IMO) eroneously return a non-0 exit code.
diff --git a/src/init.cpp b/src/init.cpp
@@ -1726,5 +1726,5 @@ bool AppInitMain(boost::thread_group& threadGroup, CScheduler& scheduler)
     StartWallets(scheduler);
 #endif
 
-    return !fRequestShutdown;
+    return true;
 }

Original file line number	Diff line number	Diff line change
`@@ -1726,5 +1726,5 @@ bool AppInitMain(boost::thread_group& threadGroup, CScheduler& scheduler)`
`1726`	`1726`	`StartWallets(scheduler);`
`1727`	`1727`	`#endif`
`1728`	`1728`
`1729`		`- return !fRequestShutdown;`
	`1729`	`+ return true;`
`1730`	`1730`	`}`