Tidying

src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/CommandFrames/AVR8Generic/GetParameter.hpp

@@ -16,7 +16,7 @@ namespace Bloom::DebugToolDrivers::Protocols::CmsisDap::Edbg::Avr::CommandFrames
              * 2. Version (0x00)
              * 3. Param context (Avr8Parameter::context)
              * 4. Param ID (Avr8Parameter::id)
-             * 5. Param value length (this->size)
+             * 5. Param value length (size)
              */
             this->payload = {
                 0x02,

src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/CommandFrames/AVR8Generic/ReadMemory.cpp

@@ -12,13 +12,13 @@ namespace Bloom::DebugToolDrivers::Protocols::CmsisDap::Edbg::Avr::CommandFrames
         const std::set<std::uint32_t>& excludedAddresses
     ) {
         /*
-         * The read memory command consists of 11/11 + (this->bytes / 8) bytes:
+         * The read memory command consists of 11 + (bytes / 8) bytes:
          * 1. Command ID (0x21 for the general read memory command, 0x22 for reading with a mask)
          * 2. Version (0x00)
          * 3. Memory type (Avr8MemoryType)
          * 4. Start address (4 bytes)
          * 5. Number of bytes to read (4 bytes)
-         * 6. Mask to apply (this->bytes / 8) - only required if we're using the masked read command (command ID 0x22).
+         * 6. Mask to apply (bytes / 8) - only required if we're using the masked read command (command ID 0x22).
          */
         this->payload = std::vector<unsigned char>(11, 0x00);
         this->payload[0] = excludedAddresses.empty() ? 0x21 : 0x22;

src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/CommandFrames/AVR8Generic/SetParameter.hpp

@@ -9,12 +9,12 @@ namespace Bloom::DebugToolDrivers::Protocols::CmsisDap::Edbg::Avr::CommandFrames
     public:
         SetParameter(const Avr8EdbgParameter& parameter, const std::vector<unsigned char>& value) {
             /*
-             * The set param command consists of this->value.size() + 5 bytes. The first five bytes consist of:
+             * The set param command consists of value.size() + 5 bytes. The first five bytes consist of:
              * 1. Command ID (0x01)
              * 2. Version (0x00)
              * 3. Param context (Avr8Parameter::context)
              * 4. Param ID (Avr8Parameter::id)
-             * 5. Param value length (this->value.size()) - this is only one byte in size, so its value should
+             * 5. Param value length (value.size()) - this is only one byte in size, so its value should
              *    never exceed 255.
              */
             this->payload = std::vector<unsigned char>(5, 0x00);

src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/CommandFrames/HouseKeeping/GetParameter.hpp

@@ -17,7 +17,7 @@ namespace Bloom::DebugToolDrivers::Protocols::CmsisDap::Edbg::Avr::CommandFrames
              * 2. Version (0x00)
              * 3. Param context (Parameter::context)
              * 4. Param ID (Parameter::id)
-             * 5. Param value length (this->size)
+             * 5. Param value length (size)
              */
             this->payload = {
                 0x02,

src/Targets/Microchip/AVR/AVR8/Avr8.hpp

@@ -55,7 +55,7 @@ namespace Bloom::Targets::Microchip::Avr::Avr8Bit
 
         void setDebugTool(DebugTool* debugTool) override {
             this->avr8DebugInterface = debugTool->getAvr8DebugInterface();
-        };
+        }
 
         /**
          * Instances to this target class can be promoted. See Avr8::promote() method for more.