Skip to content

Commit f9cc493

Browse files
committed
DPL: get rid of MessageSet::pairMap
Everything calculated on the fly
1 parent 757173b commit f9cc493

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

Framework/Core/include/Framework/MessageSet.h

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -50,22 +50,21 @@ struct MessageSet {
5050
// payload index within the O2 message
5151
size_t payloadIndex = 0;
5252
};
53-
std::vector<PairMapping> pairMap;
5453

5554
MessageSet()
56-
: messages(), messageMap(), pairMap()
55+
: messages(), messageMap()
5756
{
5857
}
5958

6059
template <typename F>
6160
MessageSet(F getter, size_t size)
62-
: messages(), messageMap(), pairMap()
61+
: messages(), messageMap()
6362
{
6463
add(std::forward<F>(getter), size);
6564
}
6665

6766
MessageSet(MessageSet&& other)
68-
: messages(std::move(other.messages)), messageMap(std::move(other.messageMap)), pairMap(std::move(other.pairMap))
67+
: messages(std::move(other.messages)), messageMap(std::move(other.messageMap))
6968
{
7069
other.clear();
7170
}
@@ -77,7 +76,6 @@ struct MessageSet {
7776
}
7877
messages = std::move(other.messages);
7978
messageMap = std::move(other.messageMap);
80-
pairMap = std::move(other.pairMap);
8179
other.clear();
8280
return *this;
8381
}
@@ -99,7 +97,6 @@ struct MessageSet {
9997
{
10098
messages.clear();
10199
messageMap.clear();
102-
pairMap.clear();
103100
}
104101

105102
// this is more or less legacy
@@ -116,7 +113,6 @@ struct MessageSet {
116113
// add content of the part ref
117114
void add(PartRef&& ref)
118115
{
119-
pairMap.emplace_back(messageMap.size(), 0);
120116
messageMap.emplace_back(messages.size(), 1);
121117
messages.emplace_back(std::move(ref.header));
122118
messages.emplace_back(std::move(ref.payload));
@@ -126,12 +122,8 @@ struct MessageSet {
126122
template <typename F>
127123
void add(F getter, size_t size)
128124
{
129-
auto partid = messageMap.size();
130125
messageMap.emplace_back(messages.size(), size - 1);
131126
for (size_t i = 0; i < size; ++i) {
132-
if (i > 0) {
133-
pairMap.emplace_back(partid, i - 1);
134-
}
135127
messages.emplace_back(std::move(getter(i)));
136128
}
137129
}

0 commit comments

Comments
 (0)