Remove dead code and fix a mistake

Framework/API/inc/MantidAPI/Algorithm.h

@@ -338,7 +338,6 @@ protected:
 
   void setExecutionState(const ExecutionState state); ///< Sets the current execution state
   void setResultState(const ResultState state); ///< Sets the result execution state
-  void setExecuted(bool state);
 
   void store();
 

Framework/API/src/Algorithm.cpp

@@ -143,23 +143,6 @@ bool Algorithm::isExecuted() const {
   return ((executionState() == ExecutionState::FINISHED) && (resultState() == ResultState::SUCCESS)) ;
 }
 
-//---------------------------------------------------------------------------------------------
-/** Set the executed flag to the specified state
-// Public in Gaudi - don't know why and will leave here unless we find a reason
-otherwise
-//     Also don't know reason for different return type and argument.
-@param state :: New executed state
-*/
-void Algorithm::setExecuted(bool state) {
-  if (state) {
-    setExecutionState(ExecutionState::FINISHED);
-    setResultState(ResultState::SUCCESS);
-  } else {
-    setExecutionState(ExecutionState::FINISHED);
-    setResultState(ResultState::FAILED);
-  }
-}
-
 //---------------------------------------------------------------------------------------------
 /** To query whether algorithm is a child.
  *  @returns true - the algorithm is a child algorithm.  False - this is a full
@@ -765,7 +748,7 @@ bool Algorithm::executeInternal() {
   // Only gets to here if algorithm ended normally
   if (algIsExecuted) {
     setExecutionState(ExecutionState::FINISHED);
-    setResultState(ResultState::FAILED);
+    setResultState(ResultState::SUCCESS);
   }
   notificationCenter().postNotification(
       new FinishedNotification(this, algIsExecuted));