Target register tidying

src/DebugServers/GdbRsp/ResponsePackets/TargetStopped.hpp

@@ -15,7 +15,6 @@ namespace Bloom::DebugServers::Gdb::ResponsePackets
     {
     public:
         Signal signal;
-        std::optional<Targets::TargetRegisterMap> registerMap;
         std::optional<StopReason> stopReason;
 
         explicit TargetStopped(Signal signal): signal(signal) {}
@@ -32,13 +31,6 @@ namespace Bloom::DebugServers::Gdb::ResponsePackets
                 }
             }
 
-            if (this->registerMap.has_value()) {
-                for (const auto& [registerId, registerValue] : this->registerMap.value()) {
-                    output += Packet::dataToHex({static_cast<unsigned char>(registerId)});
-                    output += ":" + Packet::dataToHex(registerValue.value) + ";";
-                }
-            }
-
             return std::vector<unsigned char>(output.begin(), output.end());
         }
     };

src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp

@@ -1197,7 +1197,12 @@ TargetRegisters EdbgAvr8Interface::readGeneralPurposeRegisters(std::set<std::siz
             continue;
         }
 
-        output.push_back(TargetRegister(registerIndex, {registers[registerIndex]}));
+        output.push_back(
+            TargetRegister(
+                {registerIndex, TargetRegisterType::GENERAL_PURPOSE_REGISTER},
+                {registers[registerIndex]}
+            )
+        );
     }
 
     return output;

src/Targets/TargetRegister.hpp

@@ -12,17 +12,17 @@ namespace Bloom::Targets
         PROGRAM_COUNTER,
         STACK_POINTER,
         STATUS_REGISTER,
+        OTHER,
     };
 
     struct TargetRegisterDescriptor
     {
-        using IdType = size_t;
+        std::optional<size_t> id;
-        std::optional<IdType> id;
+        TargetRegisterType type = TargetRegisterType::OTHER;
-        TargetRegisterType type = TargetRegisterType::GENERAL_PURPOSE_REGISTER;
 
         TargetRegisterDescriptor() = default;
         explicit TargetRegisterDescriptor(TargetRegisterType type): type(type) {};
-        TargetRegisterDescriptor(IdType id, TargetRegisterType type): id(id), type(type) {};
+        TargetRegisterDescriptor(size_t id, TargetRegisterType type): id(id), type(type) {};
 
         bool operator==(const TargetRegisterDescriptor& other) const {
             return this->id == other.id && this->type == other.type;
@@ -31,19 +31,12 @@ namespace Bloom::Targets
 
     struct TargetRegister
     {
-        std::optional<size_t> id;
-        std::vector<unsigned char> value;
         TargetRegisterDescriptor descriptor;
-
+        std::vector<unsigned char> value;
-        explicit TargetRegister(std::vector<unsigned char> value): value(std::move(value)) {};
 
         TargetRegister(TargetRegisterDescriptor descriptor, std::vector<unsigned char> value): value(std::move(value)),
         descriptor(descriptor) {};
 
-        TargetRegister(size_t id, std::vector<unsigned char> value): id(id), value(std::move(value)) {
-            this->descriptor.id = id;
-        };
-
         [[nodiscard]] size_t size() const {
             return this->value.size();
         }