Add an option to print errors as warnings (#546)
This commit is contained in:
parent
bf181f88d8
commit
6bc6697e3c
|
@ -102,6 +102,18 @@ int unitTestWarningCounter = 0;
|
||||||
|
|
||||||
#endif /* EFI_SIMULATOR || EFI_PROD_CODE */
|
#endif /* EFI_SIMULATOR || EFI_PROD_CODE */
|
||||||
|
|
||||||
|
void printWarning(const char *fmt, va_list ap) {
|
||||||
|
resetLogging(&logger); // todo: is 'reset' really needed here?
|
||||||
|
appendMsgPrefix(&logger);
|
||||||
|
|
||||||
|
append(&logger, WARNING_PREFIX);
|
||||||
|
|
||||||
|
printToStream(&warningStream, fmt, ap);
|
||||||
|
|
||||||
|
append(&logger, warningBuffer);
|
||||||
|
append(&logger, DELIMETER);
|
||||||
|
scheduleLogging(&logger);
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* OBD_PCM_Processor_Fault is the general error code for now
|
* OBD_PCM_Processor_Fault is the general error code for now
|
||||||
|
@ -128,19 +140,10 @@ bool warning(obd_code_e code, const char *fmt, ...) {
|
||||||
return true; // we just had another warning, let's not spam
|
return true; // we just had another warning, let's not spam
|
||||||
engine->engineState.timeOfPreviousWarning = now;
|
engine->engineState.timeOfPreviousWarning = now;
|
||||||
|
|
||||||
resetLogging(&logger); // todo: is 'reset' really needed here?
|
|
||||||
appendMsgPrefix(&logger);
|
|
||||||
|
|
||||||
append(&logger, WARNING_PREFIX);
|
|
||||||
|
|
||||||
va_list ap;
|
va_list ap;
|
||||||
va_start(ap, fmt);
|
va_start(ap, fmt);
|
||||||
printToStream(&warningStream, fmt, ap);
|
printWarning(fmt, ap);
|
||||||
va_end(ap);
|
va_end(ap);
|
||||||
|
|
||||||
append(&logger, warningBuffer);
|
|
||||||
append(&logger, DELIMETER);
|
|
||||||
scheduleLogging(&logger);
|
|
||||||
#else
|
#else
|
||||||
unitTestWarningCounter++;
|
unitTestWarningCounter++;
|
||||||
printf("unit_test_warning: ");
|
printf("unit_test_warning: ");
|
||||||
|
@ -197,6 +200,12 @@ void firmwareError(obd_code_e code, const char *fmt, ...) {
|
||||||
if (hasFirmwareErrorFlag)
|
if (hasFirmwareErrorFlag)
|
||||||
return;
|
return;
|
||||||
addWarningCode(code);
|
addWarningCode(code);
|
||||||
|
#ifdef EFI_PRINT_ERRORS_AS_WARNINGS
|
||||||
|
va_list ap;
|
||||||
|
va_start(ap, fmt);
|
||||||
|
printWarning(fmt, ap);
|
||||||
|
va_end(ap);
|
||||||
|
#endif
|
||||||
ON_FATAL_ERROR()
|
ON_FATAL_ERROR()
|
||||||
;
|
;
|
||||||
hasFirmwareErrorFlag = true;
|
hasFirmwareErrorFlag = true;
|
||||||
|
|
Loading…
Reference in New Issue