Check the log save settings before emitting save log signal
Disallow close if active connections

Also moved log file settings save check to mavlink protocol
Test disallow of close if active connections
MainWindow now disallows close if active connections

So no need for connect window close test here
Fix unused variable warning
Merge pull request #1088 from DonLakeFlyer/BugFixes

Bug fixes