cppfixes fixes updates

refs #15550

MantidQt/CustomInterfaces/src/EnggDiffraction/EnggDiffractionPresenter.cpp

@@ -612,7 +612,6 @@ void EnggDiffractionPresenter::runFittingAlgs(
         FocusedFitPeaksTableName +
         " workspace could not be found. "
         "Please check the log messages for more details.");
-    return;
   };
 
   ITableWorkspace_sptr table =
@@ -742,7 +741,6 @@ void EnggDiffractionPresenter::runCropWorkspaceAlg(std::string workspaceName) {
     cropWS->initialize();
     cropWS->setProperty("InputWorkspace", workspaceName);
     cropWS->setProperty("OutputWorkspace", workspaceName);
-    std::string outputWorkspace = "engggui_fitting_single_peaks";
     cropWS->setProperty("StartWorkspaceIndex", 1);
     cropWS->setProperty("EndWorkspaceIndex", 1);
     cropWS->execute();

MantidQt/CustomInterfaces/src/EnggDiffraction/EnggDiffractionViewQtGUI.cpp

@@ -1218,15 +1218,14 @@ std::string EnggDiffractionViewQtGUI::fittingPeaksData() const {
       m_uiTabFitting.lineEdit_fitting_peaks->text().toStdString();
   size_t strLength = exptPeaks.length() - 1;
 
-  char ch = ',';
   if (!exptPeaks.empty()) {
 
-    if (exptPeaks.at(size_t(0)) == ch) {
+    if (exptPeaks.at(size_t(0)) == ',') {
       exptPeaks.erase(size_t(0), 1);
       strLength -= size_t(1);
     }
 
-    if (exptPeaks.at(strLength) == ch) {
+    if (exptPeaks.at(strLength) == ',') {
       exptPeaks.erase(strLength, 1);
     }
   }

docs/source/interfaces/Engineering_Diffraction.rst

@@ -300,7 +300,7 @@ Output
 
 Once the Fit button has been clicked, wait until the Fitting process has
 completed and upon completion you should be able to view on the Fitting
-tab the focused workspace plotted in the background in black, whereas the
+tab plots the focused workspace in the background in black, whereas the
 expected peaks plotted in various colours over lapping the focused
 workspace peaks.