summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/tangara/database/records.cpp10
-rw-r--r--src/util/include/debug.hpp4
2 files changed, 8 insertions, 6 deletions
diff --git a/src/tangara/database/records.cpp b/src/tangara/database/records.cpp
index 88ddbd91..3e76ecad 100644
--- a/src/tangara/database/records.cpp
+++ b/src/tangara/database/records.cpp
@@ -19,6 +19,7 @@
#include "cppbor.h"
#include "cppbor_parse.h"
+#include "debug.hpp"
#include "esp_log.h"
#include "database/index.hpp"
@@ -229,16 +230,16 @@ auto ParseIndexKey(const leveldb::Slice& slice) -> std::optional<IndexKey> {
std::istringstream in(key_data.substr(header_length + 1));
std::stringbuf buffer{};
+ // FIXME: what if the item contains a '\0'? Probably we make a big mess.
in.get(buffer, kFieldSeparator);
if (buffer.str().size() > 0) {
result.item = buffer.str();
}
- buffer = {};
- in.get(buffer);
- std::string id_str = buffer.str();
+ std::string id_str =
+ key_data.substr(header_length + 1 + buffer.str().size() + 1);
if (id_str.size() > 1) {
- result.track = BytesToTrackId(id_str.substr(1));
+ result.track = BytesToTrackId(id_str);
}
return result;
@@ -252,6 +253,7 @@ auto BytesToTrackId(std::span<const char> bytes) -> std::optional<TrackId> {
auto [res, unused, err] = cppbor::parse(
reinterpret_cast<const uint8_t*>(bytes.data()), bytes.size());
if (!res || res->type() != cppbor::UINT) {
+ ESP_LOGE(kTag, "Track ID parsing failed!!");
return {};
}
return res->asUint()->unsignedValue();
diff --git a/src/util/include/debug.hpp b/src/util/include/debug.hpp
index 27fb2999..06e3833e 100644
--- a/src/util/include/debug.hpp
+++ b/src/util/include/debug.hpp
@@ -31,8 +31,8 @@ inline std::string format_hex_string(std::span<const std::byte> data) {
oss << " ";
}
int byte_val = (int)byte;
- oss << "[0x" << std::uppercase << std::setfill('0') << std::setw(2)
- << std::hex << byte_val << ']';
+ oss << std::uppercase << std::setfill('0') << std::setw(2) << std::hex
+ << byte_val << ' ';
if (byte_val >= 32 && byte_val < 127) {
ascii_values << (char)byte;
} else {