better error handling
This commit is contained in:
parent
6118be22a2
commit
5e4ac52f56
|
@ -111,7 +111,8 @@ void FuelSchedule::clear() {
|
||||||
bool FuelSchedule::addFuelEventsForCylinder(int i DECLARE_ENGINE_PARAMETER_S) {
|
bool FuelSchedule::addFuelEventsForCylinder(int i DECLARE_ENGINE_PARAMETER_S) {
|
||||||
efiAssert(engine!=NULL, "engine is NULL", false);
|
efiAssert(engine!=NULL, "engine is NULL", false);
|
||||||
|
|
||||||
if (cisnan(engine->rpmCalculator.oneDegreeUs)) {
|
float oneDegreeUs = ENGINE(rpmCalculator.oneDegreeUs); // local copy
|
||||||
|
if (cisnan(oneDegreeUs)) {
|
||||||
// in order to have fuel schedule we need to have current RPM
|
// in order to have fuel schedule we need to have current RPM
|
||||||
// wonder if this line slows engine startup?
|
// wonder if this line slows engine startup?
|
||||||
return false;
|
return false;
|
||||||
|
@ -124,8 +125,9 @@ bool FuelSchedule::addFuelEventsForCylinder(int i DECLARE_ENGINE_PARAMETER_S) {
|
||||||
* todo: since this method is not invoked within trigger event handler and
|
* todo: since this method is not invoked within trigger event handler and
|
||||||
* engineState.injectionOffset is calculated from the same utility timer should we more that logic here?
|
* engineState.injectionOffset is calculated from the same utility timer should we more that logic here?
|
||||||
*/
|
*/
|
||||||
angle_t injectionDuration = MS2US(ENGINE(fuelMs)) / ENGINE(rpmCalculator.oneDegreeUs);
|
angle_t injectionDuration = MS2US(ENGINE(fuelMs)) / oneDegreeUs;
|
||||||
angle_t baseAngle = ENGINE(engineState.injectionOffset) - injectionDuration;
|
const angle_t baseAngle = ENGINE(engineState.injectionOffset) - injectionDuration;
|
||||||
|
efiAssert(!cisnan(baseAngle), "NaN baseAngle", false);
|
||||||
|
|
||||||
int index;
|
int index;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue