From 74457f66cdfe76298f1a40a5f57d4fa029fa0056 Mon Sep 17 00:00:00 2001 From: Nav Date: Sun, 2 May 2021 15:54:32 +0100 Subject: [PATCH] Tidying --- .../CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp b/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp index 7ddd9d5f..b2484750 100644 --- a/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp +++ b/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp @@ -35,6 +35,7 @@ using namespace Bloom::DebugToolDrivers::Protocols::CmsisDap::Edbg::Avr; using namespace Bloom::Exceptions; + using Bloom::Targets::TargetState; using Bloom::Targets::TargetRegister; using Bloom::Targets::TargetRegisters; @@ -327,7 +328,7 @@ std::unique_ptr EdbgAvr8Interface::getAvrEvent() { default: { /* * TODO: This isn't very nice as we're performing an unnecessary copy. Maybe requestAvrEvents should - * return a unique_ptr instead? + * return a unique_ptr instead? */ return std::make_unique(event.value()); } @@ -347,6 +348,7 @@ void EdbgAvr8Interface::activatePhysical(bool applyExternalReset) { // Try again with external reset applied Logger::debug("Failed to activate physical interface on AVR8 target - retrying with external reset applied."); return this->activatePhysical(true); + } else { throw Exception("Activate physical interface command failed"); }