diff options
-rw-r--r-- | protocols/VKontakte/src/misc.cpp | 48 | ||||
-rw-r--r-- | protocols/VKontakte/src/vk_chats.cpp | 9 | ||||
-rw-r--r-- | protocols/VKontakte/src/vk_feed.cpp | 31 | ||||
-rw-r--r-- | protocols/VKontakte/src/vk_messages.cpp | 21 | ||||
-rw-r--r-- | protocols/VKontakte/src/vk_pollserver.cpp | 4 | ||||
-rw-r--r-- | protocols/VKontakte/src/vk_search.cpp | 6 | ||||
-rw-r--r-- | protocols/VKontakte/src/vk_thread.cpp | 16 |
7 files changed, 58 insertions, 77 deletions
diff --git a/protocols/VKontakte/src/misc.cpp b/protocols/VKontakte/src/misc.cpp index 95510c591d..453f886766 100644 --- a/protocols/VKontakte/src/misc.cpp +++ b/protocols/VKontakte/src/misc.cpp @@ -1069,9 +1069,7 @@ CMStringW CVkProto::GetAttachmentDescr(const JSONNode &jnAttachments, BBCSupport res += SetBBCString(TranslateT("Attachments:"), iBBC, vkbbcB);
res.AppendChar('\n');
- for (auto it = jnAttachments.begin(); it != jnAttachments.end(); ++it) {
- const JSONNode &jnAttach = (*it);
-
+ for (auto &jnAttach : jnAttachments) {
res.AppendChar('\t');
CMStringW wszType(jnAttach["type"].as_mstring());
if (wszType == L"photo") {
@@ -1137,24 +1135,24 @@ CMStringW CVkProto::GetAttachmentDescr(const JSONNode &jnAttachments, BBCSupport wszText.IsEmpty() ? L" " : wszText.c_str());
const JSONNode &jnCopyHystory = jnWall["copy_history"];
- for (auto aCHit = jnCopyHystory.begin(); aCHit != jnCopyHystory.end(); ++aCHit) {
- const JSONNode &jnCopyHystoryItem = (*aCHit);
-
- CMStringW wszCHText(jnCopyHystoryItem["text"].as_mstring());
- int iCHid = jnCopyHystoryItem["id"].as_int();
- int iCHfromID = jnCopyHystoryItem["from_id"].as_int();
- CMStringW wszCHUrl(FORMAT, L"https://vk.com/wall%d_%d", iCHfromID, iCHid);
- wszCHText.Replace(L"\n", L"\n\t\t");
- res.AppendFormat(L"\n\t\t%s: %s",
- SetBBCString(TranslateT("Wall post"), iBBC, vkbbcUrl, wszCHUrl).c_str(),
- wszCHText.IsEmpty() ? L" " : wszCHText.c_str());
-
- const JSONNode &jnSubAttachments = jnCopyHystoryItem["attachments"];
- if (jnSubAttachments) {
- debugLogA("CVkProto::GetAttachmentDescr SubAttachments");
- CMStringW wszAttachmentDescr = GetAttachmentDescr(jnSubAttachments, iBBC);
- wszAttachmentDescr.Replace(L"\n", L"\n\t\t");
- res += L"\n\t\t" + wszAttachmentDescr;
+ if (jnCopyHystory) {
+ for (auto &jnCopyHystoryItem : jnCopyHystory) {
+ CMStringW wszCHText(jnCopyHystoryItem["text"].as_mstring());
+ int iCHid = jnCopyHystoryItem["id"].as_int();
+ int iCHfromID = jnCopyHystoryItem["from_id"].as_int();
+ CMStringW wszCHUrl(FORMAT, L"https://vk.com/wall%d_%d", iCHfromID, iCHid);
+ wszCHText.Replace(L"\n", L"\n\t\t");
+ res.AppendFormat(L"\n\t\t%s: %s",
+ SetBBCString(TranslateT("Wall post"), iBBC, vkbbcUrl, wszCHUrl).c_str(),
+ wszCHText.IsEmpty() ? L" " : wszCHText.c_str());
+
+ const JSONNode &jnSubAttachments = jnCopyHystoryItem["attachments"];
+ if (jnSubAttachments) {
+ debugLogA("CVkProto::GetAttachmentDescr SubAttachments");
+ CMStringW wszAttachmentDescr = GetAttachmentDescr(jnSubAttachments, iBBC);
+ wszAttachmentDescr.Replace(L"\n", L"\n\t\t");
+ res += L"\n\t\t" + wszAttachmentDescr;
+ }
}
}
@@ -1285,9 +1283,7 @@ CMStringW CVkProto::GetFwdMessages(const JSONNode &jnMessages, const JSONNode &j OBJLIST<CVkUserInfo> vkUsers(2, NumericKeySortT);
- for (auto it = jnFUsers.begin(); it != jnFUsers.end(); ++it) {
- const JSONNode &jnUser = (*it);
-
+ for (auto &jnUser : jnFUsers) {
int iUserId = jnUser["id"].as_int();
CMStringW wszNick(FORMAT, L"%s %s", jnUser["first_name"].as_mstring().c_str(), jnUser["last_name"].as_mstring().c_str());
CMStringW wszLink(FORMAT, L"https://vk.com/id%d", iUserId);
@@ -1297,9 +1293,7 @@ CMStringW CVkProto::GetFwdMessages(const JSONNode &jnMessages, const JSONNode &j }
- for (auto it = jnMessages.begin(); it != jnMessages.end(); ++it) {
- const JSONNode &jnMsg = (*it);
-
+ for (auto &jnMsg : jnMessages) {
UINT uid = jnMsg["user_id"].as_int();
CVkUserInfo *vkUser = vkUsers.find((CVkUserInfo *)&uid);
CMStringW wszNick, wszUrl;
diff --git a/protocols/VKontakte/src/vk_chats.cpp b/protocols/VKontakte/src/vk_chats.cpp index 9594e80f11..fec559ad87 100644 --- a/protocols/VKontakte/src/vk_chats.cpp +++ b/protocols/VKontakte/src/vk_chats.cpp @@ -136,8 +136,7 @@ void CVkProto::OnReceiveChatInfo(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pRe for (int i = 0; i < cc->m_users.getCount(); i++)
cc->m_users[i].m_bDel = true;
- for (auto it = jnUsers.begin(); it != jnUsers.end(); ++it) {
- const JSONNode &jnUser = (*it);
+ for (auto &jnUser : jnUsers) {
if (!jnUser)
break;
@@ -195,8 +194,7 @@ void CVkProto::OnReceiveChatInfo(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pRe }
const JSONNode &jnMsgsUsers = jnResponse["msgs_users"];
- for (auto it = jnMsgsUsers.begin(); it != jnMsgsUsers.end(); ++it) {
- const JSONNode &jnUser = (*it);
+ for (auto &jnUser : jnMsgsUsers) {
LONG uid = jnUser["id"].as_int();
CVkChatUser *cu = cc->m_users.find((CVkChatUser*)&uid);
if (cu)
@@ -215,8 +213,7 @@ void CVkProto::OnReceiveChatInfo(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pRe if (jnMsgs) {
const JSONNode &jnItems = jnMsgs["items"];
if (jnItems) {
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- const JSONNode &jnMsg = (*it);
+ for (auto &jnMsg : jnItems) {
if (!jnMsg)
break;
diff --git a/protocols/VKontakte/src/vk_feed.cpp b/protocols/VKontakte/src/vk_feed.cpp index 6920e73f28..67502aee15 100644 --- a/protocols/VKontakte/src/vk_feed.cpp +++ b/protocols/VKontakte/src/vk_feed.cpp @@ -122,8 +122,7 @@ void CVkProto::CreateVkUserInfoList(OBJLIST<CVkUserInfo> &vkUsers, const JSONNod const JSONNode &jnProfiles = jnResponse["profiles"];
if (jnProfiles)
- for (auto it = jnProfiles.begin(); it != jnProfiles.end(); ++it) {
- const JSONNode &jnProfile = (*it);
+ for (auto &jnProfile : jnProfiles) {
if (!jnProfile["id"])
continue;
LONG UserId = jnProfile["id"].as_int();
@@ -142,8 +141,7 @@ void CVkProto::CreateVkUserInfoList(OBJLIST<CVkUserInfo> &vkUsers, const JSONNod const JSONNode &jnGroups = jnResponse["groups"];
if (jnGroups)
- for (auto it = jnGroups.begin(); it != jnGroups.end(); ++it) {
- const JSONNode &jnProfile = (*it);
+ for (auto &jnProfile : jnGroups) {
if (!jnProfile["id"])
continue;
LONG UserId = -jnProfile["id"].as_int();
@@ -189,8 +187,8 @@ CVKNewsItem* CVkProto::GetVkNewsItem(const JSONNode &jnItem, OBJLIST<CVkUserInfo if (jnPhotoItems) {
wszText = TranslateT("User was tagged in these photos:");
wszPopupText = wszText + TranslateT("(photos)");
- for (auto it = jnPhotoItems.begin(); it != jnPhotoItems.end(); ++it)
- wszText += L"\n" + GetVkPhotoItem((*it), m_vkOptions.BBCForNews());
+ for (auto &it : jnPhotoItems)
+ wszText += L"\n" + GetVkPhotoItem(it, m_vkOptions.BBCForNews());
}
}
}
@@ -202,8 +200,7 @@ CVKNewsItem* CVkProto::GetVkNewsItem(const JSONNode &jnItem, OBJLIST<CVkUserInfo const JSONNode &jnPhotoItems = jnPhotos["items"];
if (jnPhotoItems) {
wszPopupText += TranslateT("(photos)");
- for (auto it = jnPhotoItems.begin(); it != jnPhotoItems.end(); ++it) {
- const JSONNode &jnPhotoItem = (*it);
+ for (auto &jnPhotoItem : jnPhotoItems) {
wszText += GetVkPhotoItem(jnPhotoItem, m_vkOptions.BBCForNews()) + L"\n";
if (i == 0 && vkNewsItem->wszType == L"wall_photo") {
if (jnPhotoItem["post_id"]) {
@@ -301,8 +298,7 @@ CMStringW CVkProto::GetVkFeedback(const JSONNode &jnFeedback, VKObjType vkFeedba const JSONNode &jnUsers = jnFeedback["items"];
CMStringW wszUsers;
- for (auto it = jnUsers.begin(); it != jnUsers.end(); ++it) {
- const JSONNode &jnUserItem = (*it);
+ for (auto &jnUserItem : jnUsers) {
if (!jnUserItem["from_id"])
continue;
iUserId = jnUserItem["from_id"].as_int();
@@ -513,8 +509,7 @@ void CVkProto::OnFriendAccepted(const JSONNode & jnFeedback) {
const JSONNode &jnUsers = jnFeedback["items"];
- for (auto it = jnUsers.begin(); it != jnUsers.end(); ++it) {
- const JSONNode &jnUserItem = (*it);
+ for (auto &jnUserItem : jnUsers) {
if (!jnUserItem["from_id"])
continue;
@@ -668,8 +663,8 @@ void CVkProto::OnReceiveUnreadNews(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *p OBJLIST<CVKNewsItem> vkNews(5, sttCompareVKNewsItems);
if (jnItems)
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- CVKNewsItem *vkNewsItem = GetVkNewsItem((*it), vkUsers);
+ for (auto &it : jnItems) {
+ CVKNewsItem *vkNewsItem = GetVkNewsItem(it, vkUsers);
if (!vkNewsItem)
continue;
CVKNewsItem *vkNewsFoundItem = vkNews.find(vkNewsItem);
@@ -779,8 +774,8 @@ void CVkProto::OnReceiveUnreadNotifications(NETLIBHTTPREQUEST *reply, AsyncHttpR const JSONNode &jnItems = jnNotifications["items"];
if (jnItems)
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- CVKNewsItem *vkNotificationItem = GetVkNotificationsItem((*it), vkUsers);
+ for (auto &it : jnItems) {
+ CVKNewsItem *vkNotificationItem = GetVkNotificationsItem(it, vkUsers);
if (!vkNotificationItem)
continue;
if (vkNotification.find(vkNotificationItem) == nullptr)
@@ -794,8 +789,8 @@ void CVkProto::OnReceiveUnreadNotifications(NETLIBHTTPREQUEST *reply, AsyncHttpR const JSONNode &jnItems = jnGroupInvates["items"];
if (jnItems)
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- CVKNewsItem *vkNotificationItem = GetVkGroupInvates((*it), vkUsers);
+ for (auto &it : jnItems) {
+ CVKNewsItem *vkNotificationItem = GetVkGroupInvates(it, vkUsers);
if (!vkNotificationItem)
continue;
if (vkNotification.find(vkNotificationItem) == nullptr)
diff --git a/protocols/VKontakte/src/vk_messages.cpp b/protocols/VKontakte/src/vk_messages.cpp index 9a7ebf0f95..cd37e5c30c 100644 --- a/protocols/VKontakte/src/vk_messages.cpp +++ b/protocols/VKontakte/src/vk_messages.cpp @@ -241,8 +241,7 @@ void CVkProto::OnReceiveMessages(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pRe debugLogA("CVkProto::OnReceiveMessages numMessages = %d", numMessages);
- for (auto it = jnMsgs.begin(); it != jnMsgs.end(); ++it) {
- const JSONNode &jnMsg = (*it);
+ for (auto &jnMsg : jnMsgs) {
if (!jnMsg) {
debugLogA("CVkProto::OnReceiveMessages pMsg == nullptr");
break;
@@ -378,9 +377,9 @@ void CVkProto::OnReceiveDlgs(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pReq) LIST<void> lufUsers(20, PtrKeySortT);
const JSONNode &jnUsers = jnResponse["users"];
if (jnUsers)
- for (auto it = jnUsers.begin(); it != jnUsers.end(); ++it) {
- int iUserId = (*it)["user_id"].as_int();
- int iStatus = (*it)["friend_status"].as_int();
+ for (auto &it : jnUsers) {
+ int iUserId = it["user_id"].as_int();
+ int iStatus = it["friend_status"].as_int();
// iStatus == 3 - user is friend
// uid < 0 - user is group
@@ -392,8 +391,8 @@ void CVkProto::OnReceiveDlgs(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pReq) const JSONNode &jnGroups = jnResponse["groups"];
if (jnGroups)
- for (auto it = jnGroups.begin(); it != jnGroups.end(); ++it) {
- int iUserId = 1000000000 + (*it).as_int();
+ for (auto &it : jnGroups) {
+ int iUserId = 1000000000 + it.as_int();
if (lufUsers.indexOf((HANDLE)iUserId) != -1)
continue;
@@ -403,11 +402,11 @@ void CVkProto::OnReceiveDlgs(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pReq) CMStringA szGroupIds;
- for (auto it = jnDlgs.begin(); it != jnDlgs.end(); ++it) {
- if (!(*it))
+ for (auto &it : jnDlgs) {
+ if (!it)
break;
- int numUnread = (*it)["unread"].as_int();
- const JSONNode &jnDlg = (*it)["message"];
+ int numUnread = it["unread"].as_int();
+ const JSONNode &jnDlg = it["message"];
if (!jnDlg)
break;
diff --git a/protocols/VKontakte/src/vk_pollserver.cpp b/protocols/VKontakte/src/vk_pollserver.cpp index 77fceab9a3..49c9b017eb 100644 --- a/protocols/VKontakte/src/vk_pollserver.cpp +++ b/protocols/VKontakte/src/vk_pollserver.cpp @@ -81,8 +81,8 @@ void CVkProto::PollUpdates(const JSONNode &jnUpdates) bool bNonEdited = true;
MCONTACT hContact;
- for (auto it = jnUpdates.begin(); it != jnUpdates.end(); ++it) {
- const JSONNode &jnChild = (*it).as_array();
+ for (auto &it : jnUpdates) {
+ const JSONNode &jnChild = it.as_array();
switch (jnChild[json_index_t(0)].as_int()) {
case VKPOLL_MSG_DELFLAGS:
if (jnChild.size() < 4)
diff --git a/protocols/VKontakte/src/vk_search.cpp b/protocols/VKontakte/src/vk_search.cpp index 085e382e1d..3796fbb530 100644 --- a/protocols/VKontakte/src/vk_search.cpp +++ b/protocols/VKontakte/src/vk_search.cpp @@ -112,8 +112,7 @@ void CVkProto::OnSearch(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pReq) }
const JSONNode &jnItems = !jnResponse["items"] ? jnResponse : jnResponse["items"];
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- const JSONNode &jnRecord = (*it);
+ for (auto &jnRecord : jnItems) {
if (!jnRecord)
break;
@@ -170,8 +169,7 @@ void CVkProto::OnSearchByMail(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pReq) return;
}
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- const JSONNode &jnRecord = (*it);
+ for (auto &jnRecord : jnItems) {
if (!jnRecord)
break;
diff --git a/protocols/VKontakte/src/vk_thread.cpp b/protocols/VKontakte/src/vk_thread.cpp index a9016dde16..d49242f3ea 100644 --- a/protocols/VKontakte/src/vk_thread.cpp +++ b/protocols/VKontakte/src/vk_thread.cpp @@ -606,8 +606,8 @@ void CVkProto::OnReceiveUserInfo(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pRe if (!isChatRoom(hContact) && !IsGroupUser(hContact))
arContacts.insert((HANDLE)hContact);
- for (auto it = jnUsers.begin(); it != jnUsers.end(); ++it) {
- hContact = SetContactInfo((*it));
+ for (auto &it : jnUsers) {
+ hContact = SetContactInfo(it);
if (hContact)
arContacts.remove((HANDLE)hContact);
}
@@ -642,8 +642,8 @@ void CVkProto::OnReceiveUserInfo(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pRe return;
debugLogA("CVkProto::OnReceiveUserInfo AuthRequests");
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- LONG userid = (*it).as_int();
+ for (auto it : jnItems) {
+ LONG userid = it.as_int();
if (userid == 0)
break;
hContact = FindUser(userid, true);
@@ -668,9 +668,7 @@ void CVkProto::OnReceiveGroupInfo(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pR if (!jnResponse)
return;
- for (auto it = jnResponse.begin(); it != jnResponse.end(); ++it) {
- const JSONNode &jnItem = (*it);
-
+ for (auto &jnItem : jnResponse) {
int iGroupId = (-1)*jnItem["id"].as_int();
bool bIsMember = jnItem["is_member"].as_bool();
@@ -769,8 +767,8 @@ void CVkProto::OnReceiveFriends(NETLIBHTTPREQUEST *reply, AsyncHttpRequest *pReq const JSONNode &jnItems = jnResponse["items"];
if (jnItems)
- for (auto it = jnItems.begin(); it != jnItems.end(); ++it) {
- MCONTACT hContact = SetContactInfo((*it), true);
+ for (auto &it : jnItems) {
+ MCONTACT hContact = SetContactInfo(it, true);
if (hContact == 0 || hContact == INVALID_CONTACT_ID)
continue;
|