Tidying

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

@@ -330,14 +330,14 @@ namespace Targets::Microchip::Avr::Avr8Bit
         this->avr8DebugInterface->clearAllBreakpoints();
     }
 
-    void Avr8::writeRegisters(TargetRegisters registers) {
-        this->avr8DebugInterface->writeRegisters(registers);
-    }
-
     TargetRegisters Avr8::readRegisters(const Targets::TargetRegisterDescriptorIds& descriptorIds) {
         return this->avr8DebugInterface->readRegisters(descriptorIds);
     }
 
+    void Avr8::writeRegisters(const TargetRegisters& registers) {
+        this->avr8DebugInterface->writeRegisters(registers);
+    }
+
     TargetMemoryBuffer Avr8::readMemory(
         TargetMemoryType memoryType,
         std::uint32_t startAddress,

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

@@ -66,7 +66,7 @@ namespace Targets::Microchip::Avr::Avr8Bit
         void removeHardwareBreakpoint(TargetMemoryAddress address) override;
         void clearAllBreakpoints() override;
 
-        void writeRegisters(TargetRegisters registers) override;
+        void writeRegisters(const TargetRegisters& registers) override;
         TargetRegisters readRegisters(const Targets::TargetRegisterDescriptorIds& descriptorIds) override;
 
         TargetMemoryBuffer readMemory(

src/Targets/RiscV/RiscV.hpp

@@ -62,8 +62,8 @@ namespace Targets::RiscV
         void removeHardwareBreakpoint(TargetMemoryAddress address) override;
         void clearAllBreakpoints() override;
 
-        void writeRegisters(TargetRegisters registers) override;
         TargetRegisters readRegisters(const Targets::TargetRegisterDescriptorIds& descriptorIds) override;
+        void writeRegisters(const TargetRegisters& registers) override;
 
         TargetMemoryBuffer readMemory(
             TargetMemoryType memoryType,

src/Targets/Target.hpp

@@ -145,13 +145,6 @@ namespace Targets
          */
         virtual void clearAllBreakpoints() = 0;
 
-        /**
-         * Should update the value of the given registers.
-         *
-         * @param registers
-         */
-        virtual void writeRegisters(TargetRegisters registers) = 0;
-
         /**
          * Should read register values of the registers described by the given descriptors.
          *
@@ -161,6 +154,13 @@ namespace Targets
          */
         virtual TargetRegisters readRegisters(const Targets::TargetRegisterDescriptorIds& descriptorIds) = 0;
 
+        /**
+         * Should update the value of the given registers.
+         *
+         * @param registers
+         */
+        virtual void writeRegisters(const TargetRegisters& registers) = 0;
+
         /**
          * Should read memory from the target.
          *

src/Targets/TargetRegister.hpp

@@ -103,9 +103,13 @@ namespace Targets
     struct TargetRegister
     {
         TargetRegisterDescriptorId descriptorId;
+
+        /**
+         * Register values should be in MSB form
+         */
         TargetMemoryBuffer value;
 
-        TargetRegister(TargetRegisterDescriptorId descriptorId, std::vector<unsigned char> value)
+        TargetRegister(TargetRegisterDescriptorId descriptorId, TargetMemoryBuffer value)
             : descriptorId(descriptorId)
             , value(std::move(value))
         {};