Use Key API in indexes

Summary:
- Only in TypeIndex, not in Index (since we might want to store something other than identifiers as values)
- We might want to do the same in the `SynchronizerStore` for localId ↔ remoteId indexes

Depends on D13735

Some quick benchmarks (against develop and D13735): {F6022279}

Reviewers: cmollekopf

Reviewed By: cmollekopf

Tags: #sink

Differential Revision: https://phabricator.kde.org/D13902

1 files changed, 12 insertions, 0 deletions

diff --git a/common/index.cpp b/common/index.cpp
index 86a2dd5..238a745 100644
--- a/common/index.cpp
+++ b/common/index.cpp
@@ -2,6 +2,8 @@
 
 #include "log.h"
 
+using Sink::Storage::Identifier;
+
 Index::Index(const QString &storageRoot, const QString &dbName, const QString &indexName, Sink::Storage::DataStore::AccessMode mode)
     : mTransaction(Sink::Storage::DataStore(storageRoot, dbName, mode).createTransaction(mode)),
       mDb(mTransaction.openDatabase(indexName.toLatin1(), std::function<void(const Sink::Storage::DataStore::Error &)>(), true)),
@@ -32,6 +34,11 @@ Index::Index(const QByteArray &name, Sink::Storage::DataStore::Transaction &tran
 {
 }
 
+void Index::add(const Identifier &key, const QByteArray &value)
+{
+    add(key.toInternalByteArray(), value);
+}
+
 void Index::add(const QByteArray &key, const QByteArray &value)
 {
     Q_ASSERT(!key.isEmpty());
@@ -40,6 +47,11 @@ void Index::add(const QByteArray &key, const QByteArray &value)
     });
 }
 
+void Index::remove(const Identifier &key, const QByteArray &value)
+{
+    remove(key.toInternalByteArray(), value);
+}
+
 void Index::remove(const QByteArray &key, const QByteArray &value)
 {
     mDb.remove(key, value, [&] (const Sink::Storage::DataStore::Error &error) {