verbose trigger sync should reuse engineSnifferRpmThreshold fix #4259

firmware/CHANGELOG.md

@@ -24,6 +24,7 @@ Release template (copy/paste this for new release):
 # Changelog
 
 ### Added
+ - verbose trigger sync should reuse engineSnifferRpmThreshold #4259
 
 ### Fixed
  - Improved logic used to disambiguate trigger sync using cam/VVT information. Engine now runs in wasted spark until cam sync is achieved, at which point it switches to fully sequential. #4099

firmware/controllers/trigger/trigger_decoder.cpp

@@ -559,7 +559,9 @@ expected<TriggerDecodeResult> TriggerDecoderBase::decodeTriggerEvent(
 #endif /* EFI_UNIT_TEST */
 
 #if EFI_PROD_CODE || EFI_SIMULATOR
-			if (triggerConfiguration.VerboseTriggerSynchDetails || (someSortOfTriggerError() && !silentTriggerError)) {
+			bool verbose = engine->isEngineSnifferEnabled && triggerConfiguration.VerboseTriggerSynchDetails;
+
+			if (verbose || (someSortOfTriggerError() && !silentTriggerError)) {
 
 				int rpm = Sensor::getOrZero(SensorType::Rpm);
 				floatms_t engineCycleDuration = getEngineCycleDuration(rpm);