Fix LGTM issues

Stellarium · Mar 8, 2021 · fc3f1ec · fc3f1ec
1 parent b06c897
commit fc3f1ec
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/plugins/Oculars/src/Oculars.cpp b/plugins/Oculars/src/Oculars.cpp
@@ -2732,7 +2732,7 @@ void Oculars::handleMagnitudeLimits(PluginMode newMode)
 	}
 	else if ((pluginMode==OcOcular) || (pluginMode==OcFinder))
 	{
-		Q_ASSERT(newMode=OcNone);
+		Q_ASSERT(newMode==OcNone);
 	}
 	StelCore *core = StelApp::getInstance().getCore();
 	StelSkyDrawer *skyDrawer = core->getSkyDrawer();

diff --git a/plugins/Oculars/src/Oculars.hpp b/plugins/Oculars/src/Oculars.hpp
@@ -509,7 +509,7 @@ private slots:
 
 	QFont font;			//!< The font used for drawing labels.
 	PluginMode pluginMode;          //!< Current operational mode
-	// The next 4 are mutually exclusive "slave mode" flags to keep the buttons in the GUI show active/inactive highlight state.
+	// The next 4 are mutually exclusive "slave mode" flags to keep the buttons in the GUI showing active/inactive highlight state.
 	bool flagModeOcular;		//!< flag used to track if we are in ocular mode.
 	bool flagModeSensor;		//!< flag used to track if we are in CCD mode.
 	bool flagModeTelrad;		//!< flag used to track if we are in Telrad mode.

diff --git a/plugins/Oculars/src/gui/OcularsGuiPanel.cpp b/plugins/Oculars/src/gui/OcularsGuiPanel.cpp
@@ -413,6 +413,7 @@ OcularsGuiPanel::~OcularsGuiPanel()
 
 	delete buttonCrosshairs; buttonCrosshairs = Q_NULLPTR;
 	delete buttonCcd; buttonCcd = Q_NULLPTR;
+	delete buttonFinder;	buttonFinder= Q_NULLPTR;
 	delete buttonTelrad;	buttonTelrad = Q_NULLPTR;
 	delete buttonConfiguration; buttonConfiguration = Q_NULLPTR;
 	delete fieldOcularFl; fieldOcularFl = Q_NULLPTR;