simpler way for limp manager to handle engine stop (#3194)

* simpler

* why would we lie about spinning?

firmware/controllers/algo/engine.cpp

@@ -669,8 +669,6 @@ void doScheduleStopEngine(DECLARE_ENGINE_PARAMETER_SIGNATURE) {
 	efiPrintf("Starting doScheduleStopEngine");
 	engine->stopEngineRequestTimeNt = getTimeNowNt();
 	engine->ignitionOnTimeNt = 0;
-	// let's close injectors or else if these happen to be open right now
-	enginePins.stopPins();
 	// todo: initiate stepper motor parking
 	// make sure we have stored all the info
 #if EFI_PROD_CODE

firmware/controllers/engine_cycle/rpm_calculator.cpp

@@ -98,10 +98,6 @@ bool RpmCalculator::isRunning() const {
  * @return true if engine is spinning (cranking or running)
  */
 bool RpmCalculator::checkIfSpinning(efitick_t nowNt) const {
-	if (ENGINE(needToStopEngine(nowNt))) {
-		return false;
-	}
-
 	/**
 	 * note that the result of this subtraction could be negative, that would happen if
 	 * we have a trigger event between the time we've invoked 'getTimeNow' and here

firmware/controllers/limp_manager.cpp

@@ -59,6 +59,10 @@ void LimpManager::updateState(int rpm, efitick_t nowNt) {
 		m_hadOilPressureAfterStart = false;
 	}
 
+	if (engine->needToStopEngine(nowNt)) {
+		allowFuel.clear();
+	}
+
 	m_transientAllowInjection = allowFuel;
 	m_transientAllowIgnition = allowSpark;
 }