Skip to content

Commit d5de82e

Browse files
committed
ews: optimize process<mGetDelegateRequest>
References: GXL-600
1 parent 12d289c commit d5de82e

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

exch/ews/requests.cpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -220,15 +220,14 @@ void process(mGetDelegateRequest&& request, XMLElement* response, const EWSConte
220220

221221
std::unordered_set<std::string> requested;
222222
if (request.UserIds) {
223-
for (const auto &uid : *request.UserIds) {
223+
for (auto &&uid : *request.UserIds)
224224
if (uid.PrimarySmtpAddress)
225-
requested.insert(*uid.PrimarySmtpAddress);
226-
}
225+
requested.insert(std::move(*uid.PrimarySmtpAddress));
227226
}
228227

229228
std::unordered_set<std::string> found;
230229
for (const auto &deleg : delegate_list) {
231-
if (requested.empty() || requested.count(deleg)) {
230+
if (requested.empty() || requested.contains(deleg)) {
232231
auto &msg = data.ResponseMessages.emplace_back();
233232
msg.success();
234233
msg.DelegateUser.UserId.PrimarySmtpAddress.emplace(deleg);
@@ -238,7 +237,7 @@ void process(mGetDelegateRequest&& request, XMLElement* response, const EWSConte
238237

239238
if (!requested.empty()) {
240239
for (const auto &req : requested) {
241-
if (!found.count(req)) {
240+
if (!found.contains(req)) {
242241
auto &msg = data.ResponseMessages.emplace_back();
243242
msg.error("ErrorDelegateNotFound", "Delegate not found");
244243
msg.DelegateUser.UserId.PrimarySmtpAddress.emplace(req);

0 commit comments

Comments
 (0)