aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVincent Richard <[email protected]>2016-03-23 19:05:59 +0000
committerVincent Richard <[email protected]>2016-03-23 19:05:59 +0000
commit4b62ae41746e82d5a1638dac5ad0c19ae8c5cd39 (patch)
treee40ba4c8394e7fc5ced7190f2feb5eb89ed548b9
parentFixed #135: getPeerAddress() is incorrect. Error handling. (diff)
downloadvmime-4b62ae41746e82d5a1638dac5ad0c19ae8c5cd39.tar.gz
vmime-4b62ae41746e82d5a1638dac5ad0c19ae8c5cd39.zip
Issue #126: fixed warnings about sign mismatch.
-rw-r--r--examples/example4.cpp4
-rw-r--r--examples/example5.cpp2
-rw-r--r--examples/example6.cpp20
-rw-r--r--examples/example7.cpp4
-rw-r--r--src/vmime/net/events.cpp8
-rw-r--r--src/vmime/net/events.hpp12
-rw-r--r--src/vmime/net/folder.hpp8
-rw-r--r--src/vmime/net/folderStatus.hpp4
-rw-r--r--src/vmime/net/imap/IMAPFolder.cpp36
-rw-r--r--src/vmime/net/imap/IMAPFolder.hpp8
-rw-r--r--src/vmime/net/imap/IMAPFolderStatus.cpp30
-rw-r--r--src/vmime/net/imap/IMAPFolderStatus.hpp12
-rw-r--r--src/vmime/net/imap/IMAPMessage.cpp12
-rw-r--r--src/vmime/net/imap/IMAPMessage.hpp10
-rw-r--r--src/vmime/net/imap/IMAPMessagePart.cpp8
-rw-r--r--src/vmime/net/imap/IMAPMessagePart.hpp10
-rw-r--r--src/vmime/net/imap/IMAPMessageStructure.cpp2
-rw-r--r--src/vmime/net/imap/IMAPUtils.cpp8
-rw-r--r--src/vmime/net/imap/IMAPUtils.hpp2
-rw-r--r--src/vmime/net/maildir/maildirFolder.cpp52
-rw-r--r--src/vmime/net/maildir/maildirFolder.hpp20
-rw-r--r--src/vmime/net/maildir/maildirFolderStatus.cpp8
-rw-r--r--src/vmime/net/maildir/maildirFolderStatus.hpp12
-rw-r--r--src/vmime/net/maildir/maildirMessage.cpp4
-rw-r--r--src/vmime/net/maildir/maildirMessage.hpp6
-rw-r--r--src/vmime/net/maildir/maildirMessagePart.cpp4
-rw-r--r--src/vmime/net/maildir/maildirMessagePart.hpp6
-rw-r--r--src/vmime/net/maildir/maildirMessageStructure.cpp2
-rw-r--r--src/vmime/net/maildir/maildirUtils.cpp6
-rw-r--r--src/vmime/net/maildir/maildirUtils.hpp2
-rw-r--r--src/vmime/net/message.hpp4
-rw-r--r--src/vmime/net/messageSet.cpp26
-rw-r--r--src/vmime/net/messageSet.hpp16
-rw-r--r--src/vmime/net/pop3/POP3Folder.cpp39
-rw-r--r--src/vmime/net/pop3/POP3Folder.hpp12
-rw-r--r--src/vmime/net/pop3/POP3FolderStatus.cpp8
-rw-r--r--src/vmime/net/pop3/POP3FolderStatus.hpp12
-rw-r--r--src/vmime/net/pop3/POP3Message.cpp4
-rw-r--r--src/vmime/net/pop3/POP3Message.hpp6
-rw-r--r--src/vmime/net/pop3/POP3Utils.cpp14
-rw-r--r--src/vmime/net/pop3/POP3Utils.hpp4
-rw-r--r--tests/net/maildir/maildirStoreTest.cpp2
-rw-r--r--tests/net/messageSetTest.cpp2
-rw-r--r--tests/net/pop3/POP3UtilsTest.cpp2
44 files changed, 235 insertions, 238 deletions
diff --git a/examples/example4.cpp b/examples/example4.cpp
index 1d077575..1e0d6657 100644
--- a/examples/example4.cpp
+++ b/examples/example4.cpp
@@ -59,7 +59,7 @@ int main()
vmime::messageParser mp("<...MIME message content...>");
// Enumerate text parts
- for (int i = 0 ; i < mp.getTextPartCount() ; ++i)
+ for (size_t i = 0 ; i < mp.getTextPartCount() ; ++i)
{
const vmime::textPart& part = *mp.getTextPartAt(i);
@@ -75,7 +75,7 @@ int main()
// Corresponding plain text is in "hp.getPlainText()"
// Enumerate embedded objects (eg. images)
- for (int j = 0 ; j < hp.getObjectCount() ; ++j)
+ for (size_t j = 0 ; j < hp.getObjectCount() ; ++j)
{
const vmime::htmlTextPart::embeddedObject& obj = *hp.getObjectAt(j);
diff --git a/examples/example5.cpp b/examples/example5.cpp
index 849ffa75..d33c64f7 100644
--- a/examples/example5.cpp
+++ b/examples/example5.cpp
@@ -59,7 +59,7 @@ int main()
vmime::messageParser mp("<...MIME message content...>");
// Enumerate attachments
- for (int i = 0 ; i < mp.getAttachmentCount() ; ++i)
+ for (size_t i = 0 ; i < mp.getAttachmentCount() ; ++i)
{
const vmime::attachment& att = *mp.getAttachmentAt(i);
diff --git a/examples/example6.cpp b/examples/example6.cpp
index 428cb6a4..09c61014 100644
--- a/examples/example6.cpp
+++ b/examples/example6.cpp
@@ -53,9 +53,9 @@ static const std::string findAvailableProtocols(const vmime::net::service::Type
vmime::net::serviceFactory::getInstance();
std::ostringstream res;
- int count = 0;
+ size_t count = 0;
- for (int i = 0 ; i < sf->getServiceCount() ; ++i)
+ for (size_t i = 0 ; i < sf->getServiceCount() ; ++i)
{
const vmime::net::serviceFactory::registeredService& serv = *sf->getServiceAt(i);
@@ -136,7 +136,7 @@ static std::ostream& operator<<(std::ostream& os, const vmime::exception& e)
*/
static void printStructure(vmime::shared_ptr <const vmime::net::messageStructure> s, const int level = 0)
{
- for (int i = 0 ; i < s->getPartCount() ; ++i)
+ for (size_t i = 0 ; i < s->getPartCount() ; ++i)
{
vmime::shared_ptr <const vmime::net::messagePart> part = s->getPartAt(i);
@@ -437,14 +437,14 @@ static void connectStore()
f->open(vmime::net::folder::MODE_READ_WRITE);
- int count = f->getMessageCount();
+ vmime::size_t count = f->getMessageCount();
std::cout << std::endl;
std::cout << count << " message(s) in your inbox" << std::endl;
for (bool cont = true ; cont ; )
{
- typedef std::map <int, vmime::shared_ptr <vmime::net::message> > MessageList;
+ typedef std::map <vmime::size_t, vmime::shared_ptr <vmime::net::message> > MessageList;
MessageList msgList;
try
@@ -481,7 +481,7 @@ static void connectStore()
std::istringstream iss(line);
- int num = 0;
+ vmime::size_t num = 0;
iss >> num;
if (num < 1 || num > f->getMessageCount())
@@ -572,7 +572,7 @@ static void connectStore()
{
vmime::shared_ptr <vmime::message> parsedMsg = msg->getParsedMessage();
- std::vector <vmime::shared_ptr <const vmime::attachment> > attchs =
+ std::vector <vmime::shared_ptr <const vmime::attachment> > attchs =
vmime::attachmentHelper::findAttachmentsInMessage(parsedMsg);
if (attchs.size() > 0)
@@ -634,7 +634,7 @@ static void connectStore()
// Status
case 7:
{
- int count, unseen;
+ vmime::size_t count, unseen;
f->status(count, unseen);
std::cout << "Status: count=" << count << ", unseen=" << unseen << std::endl;
@@ -725,7 +725,7 @@ static void connectStore()
}
else
{
- const int number =
+ const vmime::size_t number =
dynamic_cast <const vmime::net::numberMessageRange&>(range).getFirst();
std::cout << "Message has successfully been added, "
@@ -760,7 +760,7 @@ static void connectStore()
}
else
{
- const int number =
+ const vmime::size_t number =
dynamic_cast <const vmime::net::numberMessageRange&>(range).getFirst();
std::cout << "Message has successfully been copied, "
diff --git a/examples/example7.cpp b/examples/example7.cpp
index e69f65f3..ae508697 100644
--- a/examples/example7.cpp
+++ b/examples/example7.cpp
@@ -47,7 +47,7 @@ int main()
std::cout << "Available encoders:" << std::endl;
- for (int i = 0 ; i < ef->getEncoderCount() ; ++i)
+ for (size_t i = 0 ; i < ef->getEncoderCount() ; ++i)
{
vmime::shared_ptr <const vmime::utility::encoder::encoderFactory::registeredEncoder>
enc = ef->getEncoderAt(i);
@@ -71,7 +71,7 @@ int main()
std::cout << "Available messaging services:" << std::endl;
- for (int i = 0 ; i < sf->getServiceCount() ; ++i)
+ for (size_t i = 0 ; i < sf->getServiceCount() ; ++i)
{
const vmime::net::serviceFactory::registeredService& serv = *sf->getServiceAt(i);
diff --git a/src/vmime/net/events.cpp b/src/vmime/net/events.cpp
index a19e1738..45f7f98e 100644
--- a/src/vmime/net/events.cpp
+++ b/src/vmime/net/events.cpp
@@ -60,7 +60,7 @@ const char* messageCountEvent::EVENT_CLASS = "messageCountEvent";
messageCountEvent::messageCountEvent
- (shared_ptr <folder> folder, const Types type, const std::vector <int>& nums)
+ (shared_ptr <folder> folder, const Types type, const std::vector <size_t>& nums)
: m_folder(folder), m_type(type)
{
m_nums.resize(nums.size());
@@ -70,7 +70,7 @@ messageCountEvent::messageCountEvent
shared_ptr <folder> messageCountEvent::getFolder() const { return (m_folder); }
messageCountEvent::Types messageCountEvent::getType() const { return (m_type); }
-const std::vector <int>& messageCountEvent::getNumbers() const { return (m_nums); }
+const std::vector <size_t>& messageCountEvent::getNumbers() const { return (m_nums); }
void messageCountEvent::dispatch(messageCountListener* listener)
@@ -96,7 +96,7 @@ const char* messageChangedEvent::EVENT_CLASS = "messageChangedEvent";
messageChangedEvent::messageChangedEvent
- (shared_ptr <folder> folder, const Types type, const std::vector <int>& nums)
+ (shared_ptr <folder> folder, const Types type, const std::vector <size_t>& nums)
: m_folder(folder), m_type(type)
{
m_nums.resize(nums.size());
@@ -106,7 +106,7 @@ messageChangedEvent::messageChangedEvent
shared_ptr <folder> messageChangedEvent::getFolder() const { return (m_folder); }
messageChangedEvent::Types messageChangedEvent::getType() const { return (m_type); }
-const std::vector <int>& messageChangedEvent::getNumbers() const { return (m_nums); }
+const std::vector <size_t>& messageChangedEvent::getNumbers() const { return (m_nums); }
void messageChangedEvent::dispatch(messageChangedListener* listener)
diff --git a/src/vmime/net/events.hpp b/src/vmime/net/events.hpp
index a3e952d4..0c6a16d3 100644
--- a/src/vmime/net/events.hpp
+++ b/src/vmime/net/events.hpp
@@ -75,7 +75,7 @@ public:
};
- messageCountEvent(shared_ptr <folder> folder, const Types type, const std::vector <int>& nums);
+ messageCountEvent(shared_ptr <folder> folder, const Types type, const std::vector <size_t>& nums);
/** Return the folder in which messages have been added/removed.
*
@@ -93,7 +93,7 @@ public:
*
* @return a list of message numbers
*/
- const std::vector <int>& getNumbers() const;
+ const std::vector <size_t>& getNumbers() const;
/** Dispatch the event to the specified listener.
*
@@ -108,7 +108,7 @@ private:
shared_ptr <folder> m_folder;
const Types m_type;
- std::vector <int> m_nums;
+ std::vector <size_t> m_nums;
};
@@ -144,7 +144,7 @@ public:
};
- messageChangedEvent(shared_ptr <folder> folder, const Types type, const std::vector <int>& nums);
+ messageChangedEvent(shared_ptr <folder> folder, const Types type, const std::vector <size_t>& nums);
/** Return the folder in which messages have changed.
*
@@ -162,7 +162,7 @@ public:
*
* @return a list of message numbers
*/
- const std::vector <int>& getNumbers() const;
+ const std::vector <size_t>& getNumbers() const;
/** Dispatch the event to the specified listener.
*
@@ -177,7 +177,7 @@ private:
shared_ptr <folder> m_folder;
const Types m_type;
- std::vector <int> m_nums;
+ std::vector <size_t> m_nums;
};
diff --git a/src/vmime/net/folder.hpp b/src/vmime/net/folder.hpp
index 0054b1e6..fc21d6b3 100644
--- a/src/vmime/net/folder.hpp
+++ b/src/vmime/net/folder.hpp
@@ -185,7 +185,7 @@ public:
* @return a new object referencing the specified message
* @throw exceptions::net_exception if an error occurs
*/
- virtual shared_ptr <message> getMessage(const int num) = 0;
+ virtual shared_ptr <message> getMessage(const size_t num) = 0;
/** Get new references to messages in this folder, given either their
* sequence numbers or UIDs.
@@ -220,7 +220,7 @@ public:
*
* @return number of messages in the folder
*/
- virtual int getMessageCount() = 0;
+ virtual size_t getMessageCount() = 0;
/** Get a new reference to a sub-folder in this folder.
*
@@ -318,7 +318,7 @@ public:
* @param unseen will receive the number of unseen messages in the folder
* @throw exceptions::net_exception if an error occurs
*/
- virtual void status(int& count, int& unseen) = 0;
+ virtual void status(size_t& count, size_t& unseen) = 0;
/** Request folder status without opening it.
*
@@ -394,7 +394,7 @@ public:
* @param uid the uid of the first message
* @throw exceptions::net_exception if an error occurs
*/
- virtual std::vector <int> getMessageNumbersStartingOnUID(const message::uid& uid) = 0;
+ virtual std::vector <size_t> getMessageNumbersStartingOnUID(const message::uid& uid) = 0;
// Event listeners
void addMessageChangedListener(events::messageChangedListener* l);
diff --git a/src/vmime/net/folderStatus.hpp b/src/vmime/net/folderStatus.hpp
index b94db052..e80b5a9f 100644
--- a/src/vmime/net/folderStatus.hpp
+++ b/src/vmime/net/folderStatus.hpp
@@ -49,13 +49,13 @@ public:
*
* @return number of messages
*/
- virtual unsigned int getMessageCount() const = 0;
+ virtual size_t getMessageCount() const = 0;
/** Returns the number of unseen messages in the folder.
*
* @return number of unseen messages
*/
- virtual unsigned int getUnseenCount() const = 0;
+ virtual size_t getUnseenCount() const = 0;
/** Clones this object.
*
diff --git a/src/vmime/net/imap/IMAPFolder.cpp b/src/vmime/net/imap/IMAPFolder.cpp
index 5f8c8b71..2ed64936 100644
--- a/src/vmime/net/imap/IMAPFolder.cpp
+++ b/src/vmime/net/imap/IMAPFolder.cpp
@@ -520,7 +520,7 @@ bool IMAPFolder::isOpen() const
}
-shared_ptr <message> IMAPFolder::getMessage(const int num)
+shared_ptr <message> IMAPFolder::getMessage(const size_t num)
{
if (!isOpen())
throw exceptions::illegal_state("Folder not open");
@@ -544,11 +544,11 @@ std::vector <shared_ptr <message> > IMAPFolder::getMessages(const messageSet& ms
if (msgs.isNumberSet())
{
- const std::vector <int> numbers = IMAPUtils::messageSetToNumberList(msgs);
+ const std::vector <size_t> numbers = IMAPUtils::messageSetToNumberList(msgs);
shared_ptr <IMAPFolder> thisFolder = dynamicCast <IMAPFolder>(shared_from_this());
- for (std::vector <int>::const_iterator it = numbers.begin() ; it != numbers.end() ; ++it)
+ for (std::vector <size_t>::const_iterator it = numbers.begin() ; it != numbers.end() ; ++it)
messages.push_back(make_shared <IMAPMessage>(thisFolder, *it));
}
else if (msgs.isUIDSet())
@@ -594,7 +594,7 @@ std::vector <shared_ptr <message> > IMAPFolder::getMessages(const messageSet& ms
continue;
// Get Process fetch response for this message
- const int msgNum = static_cast <int>(messageData->number());
+ const size_t msgNum = messageData->number();
message::uid msgUID;
// Find UID in message attributes
@@ -622,7 +622,7 @@ std::vector <shared_ptr <message> > IMAPFolder::getMessages(const messageSet& ms
}
-int IMAPFolder::getMessageCount()
+size_t IMAPFolder::getMessageCount()
{
if (!isOpen())
throw exceptions::illegal_state("Folder not open");
@@ -760,10 +760,10 @@ void IMAPFolder::fetchMessages(std::vector <shared_ptr <message> >& msg, const f
return;
// Build message numbers list
- std::vector <int> list;
+ std::vector <size_t> list;
list.reserve(msg.size());
- std::map <int, shared_ptr <IMAPMessage> > numberToMsg;
+ std::map <size_t, shared_ptr <IMAPMessage> > numberToMsg;
for (std::vector <shared_ptr <message> >::iterator it = msg.begin() ; it != msg.end() ; ++it)
{
@@ -813,9 +813,9 @@ void IMAPFolder::fetchMessages(std::vector <shared_ptr <message> >& msg, const f
continue;
// Process fetch response for this message
- const int num = static_cast <int>(messageData->number());
+ const size_t num = messageData->number();
- std::map <int, shared_ptr <IMAPMessage> >::iterator msg = numberToMsg.find(num);
+ std::map <size_t, shared_ptr <IMAPMessage> >::iterator msg = numberToMsg.find(num);
if (msg != numberToMsg.end())
{
@@ -903,7 +903,7 @@ std::vector <shared_ptr <message> > IMAPFolder::getAndFetchMessages
continue;
// Get message number
- const int msgNum = static_cast <int>(messageData->number());
+ const size_t msgNum = messageData->number();
// Get message UID
const std::vector <IMAPParser::msg_att_item*> atts = messageData->msg_att()->items();
@@ -1289,7 +1289,7 @@ messageSet IMAPFolder::copyMessages(const folder::path& dest, const messageSet&
}
-void IMAPFolder::status(int& count, int& unseen)
+void IMAPFolder::status(size_t& count, size_t& unseen)
{
count = 0;
unseen = 0;
@@ -1384,7 +1384,7 @@ void IMAPFolder::noop()
}
-std::vector <int> IMAPFolder::getMessageNumbersStartingOnUID(const message::uid& uid)
+std::vector <size_t> IMAPFolder::getMessageNumbersStartingOnUID(const message::uid& uid)
{
// Send the request
std::ostringstream uidSearchKey;
@@ -1407,7 +1407,7 @@ std::vector <int> IMAPFolder::getMessageNumbersStartingOnUID(const message::uid&
}
const std::vector <IMAPParser::continue_req_or_response_data*>& respDataList = resp->continue_req_or_response_data();
- std::vector <int> seqNumbers;
+ std::vector <size_t> seqNumbers;
for (std::vector <IMAPParser::continue_req_or_response_data*>::const_iterator
it = respDataList.begin() ; it != respDataList.end() ; ++it)
@@ -1492,7 +1492,7 @@ void IMAPFolder::processStatusUpdate(const IMAPParser::response* resp)
else if ((*it)->response_data() && (*it)->response_data()->message_data())
{
const IMAPParser::message_data* msgData = (*it)->response_data()->message_data();
- const int msgNumber = msgData->number();
+ const size_t msgNumber = msgData->number();
if ((*it)->response_data()->message_data()->type() == IMAPParser::message_data::FETCH)
{
@@ -1507,7 +1507,7 @@ void IMAPFolder::processStatusUpdate(const IMAPParser::response* resp)
events.push_back(make_shared <events::messageChangedEvent>
(dynamicCast <folder>(shared_from_this()),
events::messageChangedEvent::TYPE_FLAGS,
- std::vector <int>(1, msgNumber)));
+ std::vector <size_t>(1, msgNumber)));
}
else if ((*it)->response_data()->message_data()->type() == IMAPParser::message_data::EXPUNGE)
{
@@ -1524,7 +1524,7 @@ void IMAPFolder::processStatusUpdate(const IMAPParser::response* resp)
events.push_back(make_shared <events::messageCountEvent>
(dynamicCast <folder>(shared_from_this()),
events::messageCountEvent::TYPE_REMOVED,
- std::vector <int>(1, msgNumber)));
+ std::vector <size_t>(1, msgNumber)));
expungedMessageCount++;
}
@@ -1534,9 +1534,9 @@ void IMAPFolder::processStatusUpdate(const IMAPParser::response* resp)
// New messages arrived
if (m_status->getMessageCount() > oldStatus->getMessageCount() - expungedMessageCount)
{
- std::vector <int> newMessageNumbers;
+ std::vector <size_t> newMessageNumbers;
- for (int msgNumber = oldStatus->getMessageCount() - expungedMessageCount ;
+ for (size_t msgNumber = oldStatus->getMessageCount() - expungedMessageCount ;
msgNumber <= m_status->getMessageCount() ; ++msgNumber)
{
newMessageNumbers.push_back(msgNumber);
diff --git a/src/vmime/net/imap/IMAPFolder.hpp b/src/vmime/net/imap/IMAPFolder.hpp
index 2604cf75..0841c7b7 100644
--- a/src/vmime/net/imap/IMAPFolder.hpp
+++ b/src/vmime/net/imap/IMAPFolder.hpp
@@ -87,12 +87,12 @@ public:
bool isOpen() const;
- shared_ptr <message> getMessage(const int num);
+ shared_ptr <message> getMessage(const size_t num);
std::vector <shared_ptr <message> > getMessages(const messageSet& msgs);
- std::vector <int> getMessageNumbersStartingOnUID(const message::uid& uid);
+ std::vector <size_t> getMessageNumbersStartingOnUID(const message::uid& uid);
- int getMessageCount();
+ size_t getMessageCount();
shared_ptr <folder> getFolder(const folder::path::component& name);
std::vector <shared_ptr <folder> > getFolders(const bool recursive = false);
@@ -118,7 +118,7 @@ public:
messageSet copyMessages(const folder::path& dest, const messageSet& msgs);
- void status(int& count, int& unseen);
+ void status(size_t& count, size_t& unseen);
shared_ptr <folderStatus> getStatus();
void noop();
diff --git a/src/vmime/net/imap/IMAPFolderStatus.cpp b/src/vmime/net/imap/IMAPFolderStatus.cpp
index c78a40f3..73e1dfed 100644
--- a/src/vmime/net/imap/IMAPFolderStatus.cpp
+++ b/src/vmime/net/imap/IMAPFolderStatus.cpp
@@ -58,19 +58,19 @@ IMAPFolderStatus::IMAPFolderStatus(const IMAPFolderStatus& other)
}
-unsigned int IMAPFolderStatus::getMessageCount() const
+size_t IMAPFolderStatus::getMessageCount() const
{
return m_count;
}
-unsigned int IMAPFolderStatus::getUnseenCount() const
+size_t IMAPFolderStatus::getUnseenCount() const
{
return m_unseen;
}
-unsigned int IMAPFolderStatus::getRecentCount() const
+size_t IMAPFolderStatus::getRecentCount() const
{
return m_recent;
}
@@ -115,8 +115,8 @@ bool IMAPFolderStatus::updateFromResponse(const IMAPParser::mailbox_data* resp)
{
case IMAPParser::status_att_val::MESSAGES:
{
- const unsigned int count =
- static_cast <unsigned int>((*jt)->value_as_number()->value());
+ const size_t count =
+ static_cast <size_t>((*jt)->value_as_number()->value());
if (m_count != count)
{
@@ -128,8 +128,8 @@ bool IMAPFolderStatus::updateFromResponse(const IMAPParser::mailbox_data* resp)
}
case IMAPParser::status_att_val::UNSEEN:
{
- const unsigned int unseen =
- static_cast <unsigned int>((*jt)->value_as_number()->value());
+ const size_t unseen =
+ static_cast <size_t>((*jt)->value_as_number()->value());
if (m_unseen != unseen)
{
@@ -141,8 +141,8 @@ bool IMAPFolderStatus::updateFromResponse(const IMAPParser::mailbox_data* resp)
}
case IMAPParser::status_att_val::RECENT:
{
- const unsigned int recent =
- static_cast <unsigned int>((*jt)->value_as_number()->value());
+ const size_t recent =
+ static_cast <size_t>((*jt)->value_as_number()->value());
if (m_recent != recent)
{
@@ -197,8 +197,8 @@ bool IMAPFolderStatus::updateFromResponse(const IMAPParser::mailbox_data* resp)
}
else if (resp->type() == IMAPParser::mailbox_data::EXISTS)
{
- const unsigned int count =
- static_cast <unsigned int>(resp->number()->value());
+ const size_t count =
+ static_cast <size_t>(resp->number()->value());
if (m_count != count)
{
@@ -208,8 +208,8 @@ bool IMAPFolderStatus::updateFromResponse(const IMAPParser::mailbox_data* resp)
}
else if (resp->type() == IMAPParser::mailbox_data::RECENT)
{
- const unsigned int recent =
- static_cast <unsigned int>(resp->number()->value());
+ const size_t recent =
+ static_cast <size_t>(resp->number()->value());
if (m_recent != recent)
{
@@ -256,8 +256,8 @@ bool IMAPFolderStatus::updateFromResponse(const IMAPParser::resp_text_code* resp
}
case IMAPParser::resp_text_code::UNSEEN:
{
- const unsigned int unseen =
- static_cast <unsigned int>(resp->nz_number()->value());
+ const size_t unseen =
+ static_cast <size_t>(resp->nz_number()->value());
if (m_unseen != unseen)
{
diff --git a/src/vmime/net/imap/IMAPFolderStatus.hpp b/src/vmime/net/imap/IMAPFolderStatus.hpp
index 03ca5937..63143548 100644
--- a/src/vmime/net/imap/IMAPFolderStatus.hpp
+++ b/src/vmime/net/imap/IMAPFolderStatus.hpp
@@ -52,8 +52,8 @@ public:
IMAPFolderStatus(const IMAPFolderStatus& other);
// Inherited from folderStatus
- unsigned int getMessageCount() const;
- unsigned int getUnseenCount() const;
+ size_t getMessageCount() const;
+ size_t getUnseenCount() const;
shared_ptr <folderStatus> clone() const;
@@ -61,7 +61,7 @@ public:
*
* @return number of messages flagged Recent
*/
- unsigned int getRecentCount() const;
+ size_t getRecentCount() const;
/** Returns the UID validity of the folder for the current session.
* If the server is capable of persisting UIDs accross sessions,
@@ -105,9 +105,9 @@ public:
private:
- unsigned int m_count;
- unsigned int m_unseen;
- unsigned int m_recent;
+ size_t m_count;
+ size_t m_unseen;
+ size_t m_recent;
vmime_uint32 m_uidValidity;
vmime_uint32 m_uidNext;
vmime_uint64 m_highestModSeq;
diff --git a/src/vmime/net/imap/IMAPMessage.cpp b/src/vmime/net/imap/IMAPMessage.cpp
index c0141337..f32434b4 100644
--- a/src/vmime/net/imap/IMAPMessage.cpp
+++ b/src/vmime/net/imap/IMAPMessage.cpp
@@ -97,7 +97,7 @@ private:
//
-IMAPMessage::IMAPMessage(shared_ptr <IMAPFolder> folder, const int num)
+IMAPMessage::IMAPMessage(shared_ptr <IMAPFolder> folder, const size_t num)
: m_folder(folder), m_num(num), m_size(-1U), m_flags(FLAG_UNDEFINED),
m_expunged(false), m_modseq(0), m_structure(null)
{
@@ -105,7 +105,7 @@ IMAPMessage::IMAPMessage(shared_ptr <IMAPFolder> folder, const int num)
}
-IMAPMessage::IMAPMessage(shared_ptr <IMAPFolder> folder, const int num, const uid& uid)
+IMAPMessage::IMAPMessage(shared_ptr <IMAPFolder> folder, const size_t num, const uid& uid)
: m_folder(folder), m_num(num), m_size(-1), m_flags(FLAG_UNDEFINED),
m_expunged(false), m_uid(uid), m_modseq(0), m_structure(null)
{
@@ -135,7 +135,7 @@ void IMAPMessage::onFolderClosed()
}
-int IMAPMessage::getNumber() const
+size_t IMAPMessage::getNumber() const
{
return (m_num);
}
@@ -286,7 +286,7 @@ void IMAPMessage::extractImpl
if (p != NULL)
{
shared_ptr <const IMAPMessagePart> currentPart = dynamicCast <const IMAPMessagePart>(p);
- std::vector <int> numbers;
+ std::vector <size_t> numbers;
numbers.push_back(currentPart->getNumber());
currentPart = currentPart->getParent();
@@ -299,7 +299,7 @@ void IMAPMessage::extractImpl
numbers.erase(numbers.end() - 1);
- for (std::vector <int>::reverse_iterator it = numbers.rbegin() ; it != numbers.rend() ; ++it)
+ for (std::vector <size_t>::reverse_iterator it = numbers.rbegin() ; it != numbers.rend() ; ++it)
{
if (it != numbers.rbegin()) section << ".";
section << (*it + 1);
@@ -634,7 +634,7 @@ shared_ptr <vmime::message> IMAPMessage::getParsedMessage()
}
-void IMAPMessage::renumber(const int number)
+void IMAPMessage::renumber(const size_t number)
{
m_num = number;
}
diff --git a/src/vmime/net/imap/IMAPMessage.hpp b/src/vmime/net/imap/IMAPMessage.hpp
index 92903d69..2dea1725 100644
--- a/src/vmime/net/imap/IMAPMessage.hpp
+++ b/src/vmime/net/imap/IMAPMessage.hpp
@@ -59,12 +59,12 @@ private:
public:
- IMAPMessage(shared_ptr <IMAPFolder> folder, const int num);
- IMAPMessage(shared_ptr <IMAPFolder> folder, const int num, const uid& uid);
+ IMAPMessage(shared_ptr <IMAPFolder> folder, const size_t num);
+ IMAPMessage(shared_ptr <IMAPFolder> folder, const size_t num, const uid& uid);
~IMAPMessage();
- int getNumber() const;
+ size_t getNumber() const;
const uid getUID() const;
@@ -115,7 +115,7 @@ private:
*
* @param number new sequence number
*/
- void renumber(const int number);
+ void renumber(const size_t number);
/** Marks the message as expunged.
*/
@@ -169,7 +169,7 @@ private:
weak_ptr <IMAPFolder> m_folder;
- int m_num;
+ size_t m_num;
size_t m_size;
int m_flags;
bool m_expunged;
diff --git a/src/vmime/net/imap/IMAPMessagePart.cpp b/src/vmime/net/imap/IMAPMessagePart.cpp
index eed885dc..1539f2ca 100644
--- a/src/vmime/net/imap/IMAPMessagePart.cpp
+++ b/src/vmime/net/imap/IMAPMessagePart.cpp
@@ -36,7 +36,7 @@ namespace net {
namespace imap {
-IMAPMessagePart::IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const int number, const IMAPParser::body_type_mpart* mpart)
+IMAPMessagePart::IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const size_t number, const IMAPParser::body_type_mpart* mpart)
: m_parent(parent), m_header(null), m_number(number), m_size(0)
{
m_mediaType = vmime::mediaType
@@ -44,7 +44,7 @@ IMAPMessagePart::IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const int
}
-IMAPMessagePart::IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const int number, const IMAPParser::body_type_1part* part)
+IMAPMessagePart::IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const size_t number, const IMAPParser::body_type_1part* part)
: m_parent(parent), m_header(null), m_number(number), m_size(0)
{
if (part->body_type_text())
@@ -110,7 +110,7 @@ size_t IMAPMessagePart::getSize() const
}
-int IMAPMessagePart::getNumber() const
+size_t IMAPMessagePart::getNumber() const
{
return m_number;
}
@@ -127,7 +127,7 @@ shared_ptr <const header> IMAPMessagePart::getHeader() const
// static
shared_ptr <IMAPMessagePart> IMAPMessagePart::create
- (shared_ptr <IMAPMessagePart> parent, const int number, const IMAPParser::body* body)
+ (shared_ptr <IMAPMessagePart> parent, const size_t number, const IMAPParser::body* body)
{
if (body->body_type_mpart())
{
diff --git a/src/vmime/net/imap/IMAPMessagePart.hpp b/src/vmime/net/imap/IMAPMessagePart.hpp
index af8581d7..3b00887b 100644
--- a/src/vmime/net/imap/IMAPMessagePart.hpp
+++ b/src/vmime/net/imap/IMAPMessagePart.hpp
@@ -48,8 +48,8 @@ class VMIME_EXPORT IMAPMessagePart : public messagePart
{
public:
- IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const int number, const IMAPParser::body_type_mpart* mpart);
- IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const int number, const IMAPParser::body_type_1part* part);
+ IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const size_t number, const IMAPParser::body_type_mpart* mpart);
+ IMAPMessagePart(shared_ptr <IMAPMessagePart> parent, const size_t number, const IMAPParser::body_type_1part* part);
shared_ptr <const messageStructure> getStructure() const;
shared_ptr <messageStructure> getStructure();
@@ -58,13 +58,13 @@ public:
const mediaType& getType() const;
size_t getSize() const;
- int getNumber() const;
+ size_t getNumber() const;
shared_ptr <const header> getHeader() const;
static shared_ptr <IMAPMessagePart> create
- (shared_ptr <IMAPMessagePart> parent, const int number, const IMAPParser::body* body);
+ (shared_ptr <IMAPMessagePart> parent, const size_t number, const IMAPParser::body* body);
header& getOrCreateHeader();
@@ -75,7 +75,7 @@ private:
weak_ptr <IMAPMessagePart> m_parent;
shared_ptr <header> m_header;
- int m_number;
+ size_t m_number;
size_t m_size;
mediaType m_mediaType;
};
diff --git a/src/vmime/net/imap/IMAPMessageStructure.cpp b/src/vmime/net/imap/IMAPMessageStructure.cpp
index 8dc333e9..689c495d 100644
--- a/src/vmime/net/imap/IMAPMessageStructure.cpp
+++ b/src/vmime/net/imap/IMAPMessageStructure.cpp
@@ -49,7 +49,7 @@ IMAPMessageStructure::IMAPMessageStructure(const IMAPParser::body* body)
IMAPMessageStructure::IMAPMessageStructure(shared_ptr <IMAPMessagePart> parent, const std::vector <IMAPParser::body*>& list)
{
- int number = 0;
+ size_t number = 0;
for (std::vector <IMAPParser::body*>::const_iterator
it = list.begin() ; it != list.end() ; ++it, ++number)
diff --git a/src/vmime/net/imap/IMAPUtils.cpp b/src/vmime/net/imap/IMAPUtils.cpp
index 578e1379..d92ee240 100644
--- a/src/vmime/net/imap/IMAPUtils.cpp
+++ b/src/vmime/net/imap/IMAPUtils.cpp
@@ -740,7 +740,7 @@ public:
void enumerateNumberMessageRange(const vmime::net::numberMessageRange& range)
{
- for (int i = range.getFirst(), last = range.getLast() ; i <= last ; ++i)
+ for (size_t i = range.getFirst(), last = range.getLast() ; i <= last ; ++i)
m_list.push_back(i);
}
@@ -749,14 +749,14 @@ public:
// Not used
}
- const std::vector <int>& list() const
+ const std::vector <size_t>& list() const
{
return m_list;
}
public:
- std::vector <int> m_list;
+ std::vector <size_t> m_list;
};
@@ -772,7 +772,7 @@ const string IMAPUtils::messageSetToSequenceSet(const messageSet& msgs)
// static
-const std::vector <int> IMAPUtils::messageSetToNumberList(const messageSet& msgs)
+const std::vector <size_t> IMAPUtils::messageSetToNumberList(const messageSet& msgs)
{
IMAPMessageSetEnumerator en;
msgs.enumerate(en);
diff --git a/src/vmime/net/imap/IMAPUtils.hpp b/src/vmime/net/imap/IMAPUtils.hpp
index 61b45e48..729bc0f0 100644
--- a/src/vmime/net/imap/IMAPUtils.hpp
+++ b/src/vmime/net/imap/IMAPUtils.hpp
@@ -119,7 +119,7 @@ public:
* @param msgs message set
* @return list of message numbers
*/
- static const std::vector <int> messageSetToNumberList(const messageSet& msgs);
+ static const std::vector <size_t> messageSetToNumberList(const messageSet& msgs);
/** Constructs a message set from a parser 'uid_set' structure.
*
diff --git a/src/vmime/net/maildir/maildirFolder.cpp b/src/vmime/net/maildir/maildirFolder.cpp
index 87200473..81e64cee 100644
--- a/src/vmime/net/maildir/maildirFolder.cpp
+++ b/src/vmime/net/maildir/maildirFolder.cpp
@@ -312,7 +312,7 @@ void maildirFolder::scanFolder()
}
// Update/delete existing messages (found in previous scan)
- for (unsigned int i = 0 ; i < m_messageInfos.size() ; ++i)
+ for (size_t i = 0 ; i < m_messageInfos.size() ; ++i)
{
messageInfos& msgInfos = m_messageInfos[i];
@@ -388,7 +388,7 @@ void maildirFolder::scanFolder()
}
// Update message count
- int unreadMessageCount = 0;
+ size_t unreadMessageCount = 0;
for (std::vector <messageInfos>::const_iterator
it = m_messageInfos.begin() ; it != m_messageInfos.end() ; ++it)
@@ -398,7 +398,7 @@ void maildirFolder::scanFolder()
}
m_unreadMessageCount = unreadMessageCount;
- m_messageCount = m_messageInfos.size();
+ m_messageCount = static_cast <size_t>(m_messageInfos.size());
}
catch (exceptions::filesystem_exception&)
{
@@ -407,7 +407,7 @@ void maildirFolder::scanFolder()
}
-shared_ptr <message> maildirFolder::getMessage(const int num)
+shared_ptr <message> maildirFolder::getMessage(const size_t num)
{
if (!isOpen())
throw exceptions::illegal_state("Folder not open");
@@ -427,12 +427,12 @@ std::vector <shared_ptr <message> > maildirFolder::getMessages(const messageSet&
if (msgs.isNumberSet())
{
- const std::vector <int> numbers = maildirUtils::messageSetToNumberList(msgs);
+ const std::vector <size_t> numbers = maildirUtils::messageSetToNumberList(msgs);
std::vector <shared_ptr <message> > messages;
shared_ptr <maildirFolder> thisFolder = dynamicCast <maildirFolder>(shared_from_this());
- for (std::vector <int>::const_iterator it = numbers.begin() ; it != numbers.end() ; ++it)
+ for (std::vector <size_t>::const_iterator it = numbers.begin() ; it != numbers.end() ; ++it)
{
if (*it < 1|| *it > m_messageCount)
throw exceptions::message_not_found();
@@ -449,7 +449,7 @@ std::vector <shared_ptr <message> > maildirFolder::getMessages(const messageSet&
}
-int maildirFolder::getMessageCount()
+size_t maildirFolder::getMessageCount()
{
return (m_messageCount);
}
@@ -595,7 +595,7 @@ void maildirFolder::setMessageFlags
if (msgs.isNumberSet())
{
- const std::vector <int> nums = maildirUtils::messageSetToNumberList(msgs);
+ const std::vector <size_t> nums = maildirUtils::messageSetToNumberList(msgs);
// Change message flags
shared_ptr <utility::fileSystemFactory> fsf = platform::getHandler()->getFileSystemFactory();
@@ -603,10 +603,10 @@ void maildirFolder::setMessageFlags
utility::file::path curDirPath = store->getFormat()->
folderPathToFileSystemPath(m_path, maildirFormat::CUR_DIRECTORY);
- for (std::vector <int>::const_iterator it =
+ for (std::vector <size_t>::const_iterator it =
nums.begin() ; it != nums.end() ; ++it)
{
- const int num = *it - 1;
+ const size_t num = *it - 1;
try
{
@@ -785,7 +785,7 @@ messageSet maildirFolder::addMessage
m_unreadMessageCount++;
// Notification
- std::vector <int> nums;
+ std::vector <size_t> nums;
nums.push_back(m_messageCount);
shared_ptr <events::messageCountEvent> event =
@@ -950,14 +950,14 @@ messageSet maildirFolder::copyMessages(const folder::path& dest, const messageSe
}
// Copy messages
- const std::vector <int> nums = maildirUtils::messageSetToNumberList(msgs);
+ const std::vector <size_t> nums = maildirUtils::messageSetToNumberList(msgs);
try
{
- for (std::vector <int>::const_iterator it =
+ for (std::vector <size_t>::const_iterator it =
nums.begin() ; it != nums.end() ; ++it)
{
- const int num = *it;
+ const size_t num = *it;
const messageInfos& msg = m_messageInfos[num - 1];
const int flags = maildirUtils::extractFlags(msg.path);
@@ -995,7 +995,7 @@ void maildirFolder::notifyMessagesCopied(const folder::path& dest)
{
// We only need to update the first folder we found as calling
// status() will notify all the folders with the same path.
- int count, unseen;
+ size_t count, unseen;
(*it)->status(count, unseen);
return;
@@ -1004,7 +1004,7 @@ void maildirFolder::notifyMessagesCopied(const folder::path& dest)
}
-void maildirFolder::status(int& count, int& unseen)
+void maildirFolder::status(size_t& count, size_t& unseen)
{
count = 0;
unseen = 0;
@@ -1020,7 +1020,7 @@ shared_ptr <folderStatus> maildirFolder::getStatus()
{
shared_ptr <maildirStore> store = m_store.lock();
- const int oldCount = m_messageCount;
+ const size_t oldCount = m_messageCount;
scanFolder();
@@ -1032,10 +1032,10 @@ shared_ptr <folderStatus> maildirFolder::getStatus()
// Notify message count changed (new messages)
if (m_messageCount > oldCount)
{
- std::vector <int> nums;
+ std::vector <size_t> nums;
nums.reserve(m_messageCount - oldCount);
- for (int i = oldCount + 1, j = 0 ; i <= m_messageCount ; ++i, ++j)
+ for (size_t i = oldCount + 1, j = 0 ; i <= m_messageCount ; ++i, ++j)
nums[j] = i;
shared_ptr <events::messageCountEvent> event =
@@ -1087,10 +1087,10 @@ void maildirFolder::expunge()
utility::file::path curDirPath = store->getFormat()->
folderPathToFileSystemPath(m_path, maildirFormat::CUR_DIRECTORY);
- std::vector <int> nums;
- int unreadCount = 0;
+ std::vector <size_t> nums;
+ size_t unreadCount = 0;
- for (int num = 1 ; num <= m_messageCount ; ++num)
+ for (size_t num = 1 ; num <= m_messageCount ; ++num)
{
messageInfos& infos = m_messageInfos[num - 1];
@@ -1125,11 +1125,11 @@ void maildirFolder::expunge()
if (!nums.empty())
{
- for (std::vector <int>::size_type i = nums.size() ; i != 0 ; --i)
+ for (std::vector <size_t>::size_type i = nums.size() ; i != 0 ; --i)
m_messageInfos.erase(m_messageInfos.begin() + (i - 1));
}
- m_messageCount -= nums.size();
+ m_messageCount -= static_cast <size_t>(nums.size());
m_unreadMessageCount -= unreadCount;
// Notify message expunged
@@ -1255,7 +1255,7 @@ int maildirFolder::getFetchCapabilities() const
}
-const utility::file::path maildirFolder::getMessageFSPath(const int number) const
+const utility::file::path maildirFolder::getMessageFSPath(const size_t number) const
{
utility::file::path curDirPath = m_store.lock()->getFormat()->
folderPathToFileSystemPath(m_path, maildirFormat::CUR_DIRECTORY);
@@ -1264,7 +1264,7 @@ const utility::file::path maildirFolder::getMessageFSPath(const int number) cons
}
-std::vector <int> maildirFolder::getMessageNumbersStartingOnUID(const message::uid& /* uid */)
+std::vector <size_t> maildirFolder::getMessageNumbersStartingOnUID(const message::uid& /* uid */)
{
throw exceptions::operation_not_supported();
}
diff --git a/src/vmime/net/maildir/maildirFolder.hpp b/src/vmime/net/maildir/maildirFolder.hpp
index 9dfb7a5e..a5f74186 100644
--- a/src/vmime/net/maildir/maildirFolder.hpp
+++ b/src/vmime/net/maildir/maildirFolder.hpp
@@ -86,10 +86,10 @@ public:
bool isOpen() const;
- shared_ptr <message> getMessage(const int num);
+ shared_ptr <message> getMessage(const size_t num);
std::vector <shared_ptr <message> > getMessages(const messageSet& msgs);
- int getMessageCount();
+ size_t getMessageCount();
shared_ptr <folder> getFolder(const folder::path::component& name);
std::vector <shared_ptr <folder> > getFolders(const bool recursive = false);
@@ -105,7 +105,7 @@ public:
messageSet copyMessages(const folder::path& dest, const messageSet& msgs);
- void status(int& count, int& unseen);
+ void status(size_t& count, size_t& unseen);
shared_ptr <folderStatus> getStatus();
void expunge();
@@ -124,7 +124,7 @@ public:
int getFetchCapabilities() const;
- std::vector <int> getMessageNumbersStartingOnUID(const message::uid& uid);
+ std::vector <size_t> getMessageNumbersStartingOnUID(const message::uid& uid);
private:
@@ -135,16 +135,16 @@ private:
void registerMessage(maildirMessage* msg);
void unregisterMessage(maildirMessage* msg);
- const utility::file::path getMessageFSPath(const int number) const;
+ const utility::file::path getMessageFSPath(const size_t number) const;
void onStoreDisconnected();
void onClose();
- void deleteMessagesImpl(const std::vector <int>& nums);
- void setMessageFlagsImpl(const std::vector <int>& nums, const int flags, const int mode);
+ void deleteMessagesImpl(const std::vector <size_t>& nums);
+ void setMessageFlagsImpl(const std::vector <size_t>& nums, const int flags, const int mode);
- void copyMessagesImpl(const folder::path& dest, const std::vector <int>& nums);
+ void copyMessagesImpl(const folder::path& dest, const std::vector <size_t>& nums);
void copyMessageImpl(const utility::file::path& tmpDirPath, const utility::file::path& curDirPath, const utility::file::path::component& filename, utility::inputStream& is, const size_t size, utility::progressListener* progress);
void notifyMessagesCopied(const folder::path& dest);
@@ -158,8 +158,8 @@ private:
int m_mode;
bool m_open;
- int m_unreadMessageCount;
- int m_messageCount;
+ size_t m_unreadMessageCount;
+ size_t m_messageCount;
// Store information about scanned messages
struct messageInfos
diff --git a/src/vmime/net/maildir/maildirFolderStatus.cpp b/src/vmime/net/maildir/maildirFolderStatus.cpp
index 9ee84dba..9817c1ca 100644
--- a/src/vmime/net/maildir/maildirFolderStatus.cpp
+++ b/src/vmime/net/maildir/maildirFolderStatus.cpp
@@ -50,25 +50,25 @@ maildirFolderStatus::maildirFolderStatus(const maildirFolderStatus& other)
}
-unsigned int maildirFolderStatus::getMessageCount() const
+size_t maildirFolderStatus::getMessageCount() const
{
return m_count;
}
-unsigned int maildirFolderStatus::getUnseenCount() const
+size_t maildirFolderStatus::getUnseenCount() const
{
return m_unseen;
}
-void maildirFolderStatus::setMessageCount(const unsigned int count)
+void maildirFolderStatus::setMessageCount(const size_t count)
{
m_count = count;
}
-void maildirFolderStatus::setUnseenCount(const unsigned int unseen)
+void maildirFolderStatus::setUnseenCount(const size_t unseen)
{
m_unseen = unseen;
}
diff --git a/src/vmime/net/maildir/maildirFolderStatus.hpp b/src/vmime/net/maildir/maildirFolderStatus.hpp
index 155fb20f..0921f01e 100644
--- a/src/vmime/net/maildir/maildirFolderStatus.hpp
+++ b/src/vmime/net/maildir/maildirFolderStatus.hpp
@@ -50,19 +50,19 @@ public:
maildirFolderStatus(const maildirFolderStatus& other);
// Inherited from folderStatus
- unsigned int getMessageCount() const;
- unsigned int getUnseenCount() const;
+ size_t getMessageCount() const;
+ size_t getUnseenCount() const;
shared_ptr <folderStatus> clone() const;
- void setMessageCount(const unsigned int count);
- void setUnseenCount(const unsigned int unseen);
+ void setMessageCount(const size_t count);
+ void setUnseenCount(const size_t unseen);
private:
- unsigned int m_count;
- unsigned int m_unseen;
+ size_t m_count;
+ size_t m_unseen;
};
diff --git a/src/vmime/net/maildir/maildirMessage.cpp b/src/vmime/net/maildir/maildirMessage.cpp
index 541c58b6..faf00f6a 100644
--- a/src/vmime/net/maildir/maildirMessage.cpp
+++ b/src/vmime/net/maildir/maildirMessage.cpp
@@ -48,7 +48,7 @@ namespace net {
namespace maildir {
-maildirMessage::maildirMessage(shared_ptr <maildirFolder> folder, const int num)
+maildirMessage::maildirMessage(shared_ptr <maildirFolder> folder, const size_t num)
: m_folder(folder), m_num(num), m_size(-1), m_flags(FLAG_UNDEFINED),
m_expunged(false), m_structure(null)
{
@@ -78,7 +78,7 @@ void maildirMessage::onFolderClosed()
}
-int maildirMessage::getNumber() const
+size_t maildirMessage::getNumber() const
{
return (m_num);
}
diff --git a/src/vmime/net/maildir/maildirMessage.hpp b/src/vmime/net/maildir/maildirMessage.hpp
index 7480d49c..5cdabfc9 100644
--- a/src/vmime/net/maildir/maildirMessage.hpp
+++ b/src/vmime/net/maildir/maildirMessage.hpp
@@ -54,12 +54,12 @@ class VMIME_EXPORT maildirMessage : public message
public:
- maildirMessage(shared_ptr <maildirFolder> folder, const int num);
+ maildirMessage(shared_ptr <maildirFolder> folder, const size_t num);
~maildirMessage();
- int getNumber() const;
+ size_t getNumber() const;
const uid getUID() const;
@@ -95,7 +95,7 @@ private:
weak_ptr <maildirFolder> m_folder;
- int m_num;
+ size_t m_num;
size_t m_size;
int m_flags;
bool m_expunged;
diff --git a/src/vmime/net/maildir/maildirMessagePart.cpp b/src/vmime/net/maildir/maildirMessagePart.cpp
index 6ae085c9..0b813749 100644
--- a/src/vmime/net/maildir/maildirMessagePart.cpp
+++ b/src/vmime/net/maildir/maildirMessagePart.cpp
@@ -36,7 +36,7 @@ namespace net {
namespace maildir {
-maildirMessagePart::maildirMessagePart(shared_ptr <maildirMessagePart> parent, const int number, const bodyPart& part)
+maildirMessagePart::maildirMessagePart(shared_ptr <maildirMessagePart> parent, const size_t number, const bodyPart& part)
: m_parent(parent), m_header(null), m_number(number)
{
m_headerParsedOffset = part.getHeader()->getParsedOffset();
@@ -99,7 +99,7 @@ size_t maildirMessagePart::getSize() const
}
-int maildirMessagePart::getNumber() const
+size_t maildirMessagePart::getNumber() const
{
return m_number;
}
diff --git a/src/vmime/net/maildir/maildirMessagePart.hpp b/src/vmime/net/maildir/maildirMessagePart.hpp
index 3a4be0f3..db1aeb75 100644
--- a/src/vmime/net/maildir/maildirMessagePart.hpp
+++ b/src/vmime/net/maildir/maildirMessagePart.hpp
@@ -46,7 +46,7 @@ class maildirMessagePart : public messagePart
{
public:
- maildirMessagePart(shared_ptr <maildirMessagePart> parent, const int number, const bodyPart& part);
+ maildirMessagePart(shared_ptr <maildirMessagePart> parent, const size_t number, const bodyPart& part);
~maildirMessagePart();
@@ -57,7 +57,7 @@ public:
const mediaType& getType() const;
size_t getSize() const;
- int getNumber() const;
+ size_t getNumber() const;
shared_ptr <const header> getHeader() const;
@@ -77,7 +77,7 @@ private:
weak_ptr <maildirMessagePart> m_parent;
shared_ptr <header> m_header;
- int m_number;
+ size_t m_number;
size_t m_size;
mediaType m_mediaType;
diff --git a/src/vmime/net/maildir/maildirMessageStructure.cpp b/src/vmime/net/maildir/maildirMessageStructure.cpp
index f3b7cf59..54eba981 100644
--- a/src/vmime/net/maildir/maildirMessageStructure.cpp
+++ b/src/vmime/net/maildir/maildirMessageStructure.cpp
@@ -55,7 +55,7 @@ maildirMessageStructure::maildirMessageStructure(shared_ptr <maildirMessagePart>
maildirMessageStructure::maildirMessageStructure(shared_ptr <maildirMessagePart> parent, const std::vector <shared_ptr <const vmime::bodyPart> >& list)
{
- for (unsigned int i = 0 ; i < list.size() ; ++i)
+ for (size_t i = 0 ; i < list.size() ; ++i)
{
shared_ptr <maildirMessagePart> mpart = make_shared <maildirMessagePart>(parent, i, *list[i]);
mpart->initStructure(*list[i]);
diff --git a/src/vmime/net/maildir/maildirUtils.cpp b/src/vmime/net/maildir/maildirUtils.cpp
index 77aac715..8c40f076 100644
--- a/src/vmime/net/maildir/maildirUtils.cpp
+++ b/src/vmime/net/maildir/maildirUtils.cpp
@@ -220,7 +220,7 @@ public:
void enumerateNumberMessageRange(const vmime::net::numberMessageRange& range)
{
- for (int i = range.getFirst(), last = range.getLast() ; i <= last ; ++i)
+ for (size_t i = range.getFirst(), last = range.getLast() ; i <= last ; ++i)
list.push_back(i);
}
@@ -231,12 +231,12 @@ public:
public:
- std::vector <int> list;
+ std::vector <size_t> list;
};
// static
-const std::vector <int> maildirUtils::messageSetToNumberList(const messageSet& msgs)
+const std::vector <size_t> maildirUtils::messageSetToNumberList(const messageSet& msgs)
{
maildirMessageSetEnumerator en;
msgs.enumerate(en);
diff --git a/src/vmime/net/maildir/maildirUtils.hpp b/src/vmime/net/maildir/maildirUtils.hpp
index 82deefbb..152db45b 100644
--- a/src/vmime/net/maildir/maildirUtils.hpp
+++ b/src/vmime/net/maildir/maildirUtils.hpp
@@ -137,7 +137,7 @@ public:
* @param msgs message set
* @return list of message numbers
*/
- static const std::vector <int> messageSetToNumberList(const messageSet& msgs);
+ static const std::vector <size_t> messageSetToNumberList(const messageSet& msgs);
};
diff --git a/src/vmime/net/message.hpp b/src/vmime/net/message.hpp
index 2a8eb30d..bc1b62b4 100644
--- a/src/vmime/net/message.hpp
+++ b/src/vmime/net/message.hpp
@@ -97,7 +97,7 @@ public:
*
* @return part number
*/
- virtual int getNumber() const = 0;
+ virtual size_t getNumber() const = 0;
/** Return the sub-part at the specified position (zero is the
* first part).
@@ -231,7 +231,7 @@ public:
*
* @return sequence number of the message
*/
- virtual int getNumber() const = 0;
+ virtual size_t getNumber() const = 0;
/** Return the unique identifier (UID) of this message in its
* folder (must fetch before).
diff --git a/src/vmime/net/messageSet.cpp b/src/vmime/net/messageSet.cpp
index 8a923a3f..516dce12 100644
--- a/src/vmime/net/messageSet.cpp
+++ b/src/vmime/net/messageSet.cpp
@@ -52,7 +52,7 @@ messageRange::~messageRange()
// numberMessageRange
-numberMessageRange::numberMessageRange(const int number)
+numberMessageRange::numberMessageRange(const size_t number)
: m_first(number), m_last(number)
{
if (number < 1)
@@ -60,12 +60,12 @@ numberMessageRange::numberMessageRange(const int number)
}
-numberMessageRange::numberMessageRange(const int first, const int last)
+numberMessageRange::numberMessageRange(const size_t first, const size_t last)
: m_first(first), m_last(last)
{
if (first < 1)
throw std::invalid_argument("first");
- else if (last != -1 && last < first)
+ else if (last != -1U && last < first)
throw std::invalid_argument("last");
}
@@ -76,13 +76,13 @@ numberMessageRange::numberMessageRange(const numberMessageRange& other)
}
-int numberMessageRange::getFirst() const
+size_t numberMessageRange::getFirst() const
{
return m_first;
}
-int numberMessageRange::getLast() const
+size_t numberMessageRange::getLast() const
{
return m_last;
}
@@ -177,7 +177,7 @@ messageSet messageSet::empty()
// static
-messageSet messageSet::byNumber(const int number)
+messageSet messageSet::byNumber(const size_t number)
{
messageSet set;
set.m_ranges.push_back(new numberMessageRange(number));
@@ -187,7 +187,7 @@ messageSet messageSet::byNumber(const int number)
// static
-messageSet messageSet::byNumber(const int first, const int last)
+messageSet messageSet::byNumber(const size_t first, const size_t last)
{
messageSet set;
set.m_ranges.push_back(new numberMessageRange(first, last));
@@ -197,10 +197,10 @@ messageSet messageSet::byNumber(const int first, const int last)
// static
-messageSet messageSet::byNumber(const std::vector <int>& numbers)
+messageSet messageSet::byNumber(const std::vector <size_t>& numbers)
{
// Sort a copy of the list
- std::vector <int> sortedNumbers;
+ std::vector <size_t> sortedNumbers;
sortedNumbers.resize(numbers.size());
@@ -208,18 +208,18 @@ messageSet messageSet::byNumber(const std::vector <int>& numbers)
std::sort(sortedNumbers.begin(), sortedNumbers.end());
// Build the set by detecting ranges of continuous numbers
- int previous = -1, rangeStart = -1;
+ size_t previous = -1U, rangeStart = -1U;
messageSet set;
- for (std::vector <int>::const_iterator it = sortedNumbers.begin() ;
+ for (std::vector <size_t>::const_iterator it = sortedNumbers.begin() ;
it != sortedNumbers.end() ; ++it)
{
- const int current = *it;
+ const size_t current = *it;
if (current == previous)
continue; // skip duplicates
- if (previous == -1)
+ if (previous == -1U)
{
previous = current;
rangeStart = current;
diff --git a/src/vmime/net/messageSet.hpp b/src/vmime/net/messageSet.hpp
index dbe1cce3..1be49761 100644
--- a/src/vmime/net/messageSet.hpp
+++ b/src/vmime/net/messageSet.hpp
@@ -95,7 +95,7 @@ public:
*
* @param number message number (numbering starts at 1, not 0)
*/
- numberMessageRange(const int number);
+ numberMessageRange(const size_t number);
/** Constructs a message range for multiple messages.
*
@@ -104,7 +104,7 @@ public:
* @param last number of the last message in the range, or use the
* special value -1 to designate the last message in the folder
*/
- numberMessageRange(const int first, const int last);
+ numberMessageRange(const size_t first, const size_t last);
/** Constructs a message range by copying from another range.
*
@@ -116,14 +116,14 @@ public:
*
* @return number of the first message
*/
- int getFirst() const;
+ size_t getFirst() const;
/** Returns the number of the last message in the range, or -1
* to designate the last message in the folder
*
* @return number of the last message
*/
- int getLast() const;
+ size_t getLast() const;
void enumerate(messageSetEnumerator& en) const;
@@ -131,7 +131,7 @@ public:
private:
- int m_first, m_last;
+ size_t m_first, m_last;
};
@@ -232,7 +232,7 @@ public:
* @param number message number (numbering starts at 1, not 0)
* @return new message set
*/
- static messageSet byNumber(const int number);
+ static messageSet byNumber(const size_t number);
/** Constructs a new message set and initializes it with a range
* of messages represented by their sequence number.
@@ -243,7 +243,7 @@ public:
* special value -1 to designate the last message in the folder
* @return new message set
*/
- static messageSet byNumber(const int first, const int last);
+ static messageSet byNumber(const size_t first, const size_t last);
/** Constructs a new message set and initializes it with a possibly
* unsorted list of messages represented by their sequence number.
@@ -258,7 +258,7 @@ public:
* @param numbers a vector containing numbers of the messages
* @return new message set
*/
- static messageSet byNumber(const std::vector <int>& numbers);
+ static messageSet byNumber(const std::vector <size_t>& numbers);
/** Constructs a new message set and initializes it with a single
* message represented by its UID.
diff --git a/src/vmime/net/pop3/POP3Folder.cpp b/src/vmime/net/pop3/POP3Folder.cpp
index 17c6bf9e..6621fd37 100644
--- a/src/vmime/net/pop3/POP3Folder.cpp
+++ b/src/vmime/net/pop3/POP3Folder.cpp
@@ -219,7 +219,7 @@ bool POP3Folder::isOpen() const
}
-shared_ptr <message> POP3Folder::getMessage(const int num)
+shared_ptr <message> POP3Folder::getMessage(const size_t num)
{
shared_ptr <POP3Store> store = m_store.lock();
@@ -245,12 +245,12 @@ std::vector <shared_ptr <message> > POP3Folder::getMessages(const messageSet& ms
if (msgs.isNumberSet())
{
- const std::vector <int> numbers = POP3Utils::messageSetToNumberList(msgs);
+ const std::vector <size_t> numbers = POP3Utils::messageSetToNumberList(msgs);
std::vector <shared_ptr <message> > messages;
shared_ptr <POP3Folder> thisFolder(dynamicCast <POP3Folder>(shared_from_this()));
- for (std::vector <int>::const_iterator it = numbers.begin() ; it != numbers.end() ; ++it)
+ for (std::vector <size_t>::const_iterator it = numbers.begin() ; it != numbers.end() ; ++it)
{
if (*it < 1|| *it > m_messageCount)
throw exceptions::message_not_found();
@@ -267,7 +267,7 @@ std::vector <shared_ptr <message> > POP3Folder::getMessages(const messageSet& ms
}
-int POP3Folder::getMessageCount()
+size_t POP3Folder::getMessageCount()
{
shared_ptr <POP3Store> store = m_store.lock();
@@ -357,7 +357,7 @@ void POP3Folder::fetchMessages(std::vector <shared_ptr <message> >& msg, const f
// S: 1 47548
// S: 2 12653
// S: .
- std::map <int, string> result;
+ std::map <size_t, string> result;
POP3Utils::parseMultiListOrUidlResponse(response, result);
for (std::vector <shared_ptr <message> >::iterator it = msg.begin() ;
@@ -365,7 +365,7 @@ void POP3Folder::fetchMessages(std::vector <shared_ptr <message> >& msg, const f
{
shared_ptr <POP3Message> m = dynamicCast <POP3Message>(*it);
- std::map <int, string>::const_iterator x = result.find(m->m_num);
+ std::map <size_t, string>::const_iterator x = result.find(m->m_num);
if (x != result.end())
{
@@ -397,7 +397,7 @@ void POP3Folder::fetchMessages(std::vector <shared_ptr <message> >& msg, const f
// S: 1 whqtswO00WBw418f9t5JxYwZ
// S: 2 QhdPYR:00WBw1Ph7x7
// S: .
- std::map <int, string> result;
+ std::map <size_t, string> result;
POP3Utils::parseMultiListOrUidlResponse(response, result);
for (std::vector <shared_ptr <message> >::iterator it = msg.begin() ;
@@ -405,7 +405,7 @@ void POP3Folder::fetchMessages(std::vector <shared_ptr <message> >& msg, const f
{
shared_ptr <POP3Message> m = dynamicCast <POP3Message>(*it);
- std::map <int, string>::const_iterator x = result.find(m->m_num);
+ std::map <size_t, string>::const_iterator x = result.find(m->m_num);
if (x != result.end())
m->m_uid = (*x).second;
@@ -558,7 +558,7 @@ void POP3Folder::deleteMessages(const messageSet& msgs)
{
shared_ptr <POP3Store> store = m_store.lock();
- const std::vector <int> nums = POP3Utils::messageSetToNumberList(msgs);
+ const std::vector <size_t> nums = POP3Utils::messageSetToNumberList(msgs);
if (nums.empty())
throw exceptions::invalid_argument();
@@ -568,7 +568,7 @@ void POP3Folder::deleteMessages(const messageSet& msgs)
else if (!isOpen())
throw exceptions::illegal_state("Folder not open");
- for (std::vector <int>::const_iterator
+ for (std::vector <size_t>::const_iterator
it = nums.begin() ; it != nums.end() ; ++it)
{
POP3Command::DELE(*it)->send(store->getConnection());
@@ -581,7 +581,7 @@ void POP3Folder::deleteMessages(const messageSet& msgs)
}
// Sort message list
- std::vector <int> list;
+ std::vector <size_t> list;
list.resize(nums.size());
std::copy(nums.begin(), nums.end(), list.begin());
@@ -589,7 +589,7 @@ void POP3Folder::deleteMessages(const messageSet& msgs)
std::sort(list.begin(), list.end());
// Update local flags
- for (std::map <POP3Message*, int>::iterator it =
+ for (std::map <POP3Message*, size_t>::iterator it =
m_messages.begin() ; it != m_messages.end() ; ++it)
{
POP3Message* msg = (*it).first;
@@ -644,7 +644,7 @@ messageSet POP3Folder::copyMessages
}
-void POP3Folder::status(int& count, int& unseen)
+void POP3Folder::status(size_t& count, size_t& unseen)
{
count = 0;
unseen = 0;
@@ -672,14 +672,11 @@ shared_ptr <folderStatus> POP3Folder::getStatus()
throw exceptions::command_error("STAT", response->getFirstLine());
- int count = 0;
+ size_t count = 0;
std::istringstream iss(response->getText());
iss >> count;
- if (count < 0)
- throw exceptions::invalid_response("STAT", response->getText());
-
shared_ptr <POP3FolderStatus> status = make_shared <POP3FolderStatus>();
status->setMessageCount(count);
@@ -688,16 +685,16 @@ shared_ptr <folderStatus> POP3Folder::getStatus()
// Update local message count
if (m_messageCount != count)
{
- const int oldCount = m_messageCount;
+ const size_t oldCount = m_messageCount;
m_messageCount = count;
if (count > oldCount)
{
- std::vector <int> nums;
+ std::vector <size_t> nums;
nums.resize(count - oldCount);
- for (int i = oldCount + 1, j = 0 ; i <= count ; ++i, ++j)
+ for (size_t i = oldCount + 1, j = 0 ; i <= count ; ++i, ++j)
nums[j] = i;
// Notify message count changed
@@ -738,7 +735,7 @@ void POP3Folder::expunge()
}
-std::vector <int> POP3Folder::getMessageNumbersStartingOnUID(const message::uid& /* uid */)
+std::vector <size_t> POP3Folder::getMessageNumbersStartingOnUID(const message::uid& /* uid */)
{
throw exceptions::operation_not_supported();
}
diff --git a/src/vmime/net/pop3/POP3Folder.hpp b/src/vmime/net/pop3/POP3Folder.hpp
index a549ec96..9b073cc8 100644
--- a/src/vmime/net/pop3/POP3Folder.hpp
+++ b/src/vmime/net/pop3/POP3Folder.hpp
@@ -83,10 +83,10 @@ public:
bool isOpen() const;
- shared_ptr <message> getMessage(const int num);
+ shared_ptr <message> getMessage(const size_t num);
std::vector <shared_ptr <message> > getMessages(const messageSet& msgs);
- int getMessageCount();
+ size_t getMessageCount();
shared_ptr <folder> getFolder(const folder::path::component& name);
std::vector <shared_ptr <folder> > getFolders(const bool recursive = false);
@@ -102,7 +102,7 @@ public:
messageSet copyMessages(const folder::path& dest, const messageSet& msgs);
- void status(int& count, int& unseen);
+ void status(size_t& count, size_t& unseen);
shared_ptr <folderStatus> getStatus();
void expunge();
@@ -121,7 +121,7 @@ public:
int getFetchCapabilities() const;
- std::vector <int> getMessageNumbersStartingOnUID(const message::uid& uid);
+ std::vector <size_t> getMessageNumbersStartingOnUID(const message::uid& uid);
private:
@@ -141,9 +141,9 @@ private:
int m_mode;
bool m_open;
- int m_messageCount;
+ size_t m_messageCount;
- typedef std::map <POP3Message*, int> MessageMap;
+ typedef std::map <POP3Message*, size_t> MessageMap;
MessageMap m_messages;
};
diff --git a/src/vmime/net/pop3/POP3FolderStatus.cpp b/src/vmime/net/pop3/POP3FolderStatus.cpp
index 944379ac..15e4cc92 100644
--- a/src/vmime/net/pop3/POP3FolderStatus.cpp
+++ b/src/vmime/net/pop3/POP3FolderStatus.cpp
@@ -50,25 +50,25 @@ POP3FolderStatus::POP3FolderStatus(const POP3FolderStatus& other)
}
-unsigned int POP3FolderStatus::getMessageCount() const
+size_t POP3FolderStatus::getMessageCount() const
{
return m_count;
}
-unsigned int POP3FolderStatus::getUnseenCount() const
+size_t POP3FolderStatus::getUnseenCount() const
{
return m_unseen;
}
-void POP3FolderStatus::setMessageCount(const unsigned int count)
+void POP3FolderStatus::setMessageCount(const size_t count)
{
m_count = count;
}
-void POP3FolderStatus::setUnseenCount(const unsigned int unseen)
+void POP3FolderStatus::setUnseenCount(const size_t unseen)
{
m_unseen = unseen;
}
diff --git a/src/vmime/net/pop3/POP3FolderStatus.hpp b/src/vmime/net/pop3/POP3FolderStatus.hpp
index 70ba48b6..74c07a51 100644
--- a/src/vmime/net/pop3/POP3FolderStatus.hpp
+++ b/src/vmime/net/pop3/POP3FolderStatus.hpp
@@ -50,19 +50,19 @@ public:
POP3FolderStatus(const POP3FolderStatus& other);
// Inherited from folderStatus
- unsigned int getMessageCount() const;
- unsigned int getUnseenCount() const;
+ size_t getMessageCount() const;
+ size_t getUnseenCount() const;
shared_ptr <folderStatus> clone() const;
- void setMessageCount(const unsigned int count);
- void setUnseenCount(const unsigned int unseen);
+ void setMessageCount(const size_t count);
+ void setUnseenCount(const size_t unseen);
private:
- unsigned int m_count;
- unsigned int m_unseen;
+ size_t m_count;
+ size_t m_unseen;
};
diff --git a/src/vmime/net/pop3/POP3Message.cpp b/src/vmime/net/pop3/POP3Message.cpp
index 07684dce..67ad6d16 100644
--- a/src/vmime/net/pop3/POP3Message.cpp
+++ b/src/vmime/net/pop3/POP3Message.cpp
@@ -44,7 +44,7 @@ namespace net {
namespace pop3 {
-POP3Message::POP3Message(shared_ptr <POP3Folder> folder, const int num)
+POP3Message::POP3Message(shared_ptr <POP3Folder> folder, const size_t num)
: m_folder(folder), m_num(num), m_size(-1), m_deleted(false)
{
folder->registerMessage(this);
@@ -73,7 +73,7 @@ void POP3Message::onFolderClosed()
}
-int POP3Message::getNumber() const
+size_t POP3Message::getNumber() const
{
return (m_num);
}
diff --git a/src/vmime/net/pop3/POP3Message.hpp b/src/vmime/net/pop3/POP3Message.hpp
index 87e71ba7..e4e3b079 100644
--- a/src/vmime/net/pop3/POP3Message.hpp
+++ b/src/vmime/net/pop3/POP3Message.hpp
@@ -56,12 +56,12 @@ private:
public:
- POP3Message(shared_ptr <POP3Folder> folder, const int num);
+ POP3Message(shared_ptr <POP3Folder> folder, const size_t num);
~POP3Message();
- int getNumber() const;
+ size_t getNumber() const;
const uid getUID() const;
@@ -101,7 +101,7 @@ private:
void onFolderClosed();
weak_ptr <POP3Folder> m_folder;
- int m_num;
+ size_t m_num;
uid m_uid;
size_t m_size;
diff --git a/src/vmime/net/pop3/POP3Utils.cpp b/src/vmime/net/pop3/POP3Utils.cpp
index 7ba65fff..6627cca6 100644
--- a/src/vmime/net/pop3/POP3Utils.cpp
+++ b/src/vmime/net/pop3/POP3Utils.cpp
@@ -39,9 +39,9 @@ namespace pop3 {
// static
-void POP3Utils::parseMultiListOrUidlResponse(shared_ptr <POP3Response> response, std::map <int, string>& result)
+void POP3Utils::parseMultiListOrUidlResponse(shared_ptr <POP3Response> response, std::map <size_t, string>& result)
{
- std::map <int, string> ids;
+ std::map <size_t, string> ids;
for (size_t i = 0, n = response->getLineCount() ; i < n ; ++i)
{
@@ -53,7 +53,7 @@ void POP3Utils::parseMultiListOrUidlResponse(shared_ptr <POP3Response> response,
if (it != line.end())
{
- int number = 0;
+ size_t number = 0;
while (it != line.end() && (*it >= '0' && *it <= '9'))
{
@@ -66,7 +66,7 @@ void POP3Utils::parseMultiListOrUidlResponse(shared_ptr <POP3Response> response,
if (it != line.end())
{
- result.insert(std::map <int, string>::value_type(number, string(it, line.end())));
+ result.insert(std::map <size_t, string>::value_type(number, string(it, line.end())));
}
}
}
@@ -80,7 +80,7 @@ public:
void enumerateNumberMessageRange(const vmime::net::numberMessageRange& range)
{
- for (int i = range.getFirst(), last = range.getLast() ; i <= last ; ++i)
+ for (size_t i = range.getFirst(), last = range.getLast() ; i <= last ; ++i)
list.push_back(i);
}
@@ -91,12 +91,12 @@ public:
public:
- std::vector <int> list;
+ std::vector <size_t> list;
};
// static
-const std::vector <int> POP3Utils::messageSetToNumberList(const messageSet& msgs)
+const std::vector <size_t> POP3Utils::messageSetToNumberList(const messageSet& msgs)
{
POP3MessageSetEnumerator en;
msgs.enumerate(en);
diff --git a/src/vmime/net/pop3/POP3Utils.hpp b/src/vmime/net/pop3/POP3Utils.hpp
index c7459efe..995c9115 100644
--- a/src/vmime/net/pop3/POP3Utils.hpp
+++ b/src/vmime/net/pop3/POP3Utils.hpp
@@ -64,14 +64,14 @@ public:
* number to its corresponding data (either UID or size)
*/
static void parseMultiListOrUidlResponse
- (shared_ptr <POP3Response> response, std::map <int, string>& result);
+ (shared_ptr <POP3Response> response, std::map <size_t, string>& result);
/** Returns a list of message numbers given a message set.
*
* @param msgs message set
* @return list of message numbers
*/
- static const std::vector <int> messageSetToNumberList(const messageSet& msgs);
+ static const std::vector <size_t> messageSetToNumberList(const messageSet& msgs);
};
diff --git a/tests/net/maildir/maildirStoreTest.cpp b/tests/net/maildir/maildirStoreTest.cpp
index 9911c484..df827b21 100644
--- a/tests/net/maildir/maildirStoreTest.cpp
+++ b/tests/net/maildir/maildirStoreTest.cpp
@@ -291,7 +291,7 @@ public:
vmime::shared_ptr <vmime::net::folder> folder = store->getFolder
(fpath() / "Folder" / "SubFolder" / "SubSubFolder2");
- int count, unseen;
+ vmime::index_t count, unseen;
folder->status(count, unseen);
VASSERT_EQ("Message count", 1, count);
diff --git a/tests/net/messageSetTest.cpp b/tests/net/messageSetTest.cpp
index 3aef7ab0..70ba051c 100644
--- a/tests/net/messageSetTest.cpp
+++ b/tests/net/messageSetTest.cpp
@@ -116,7 +116,7 @@ VMIME_TEST_SUITE_BEGIN(messageSetTest)
void testNumberSet_Multiple()
{
- std::vector <int> numbers;
+ std::vector <vmime::index_t> numbers;
numbers.push_back(1); // test grouping 1:3
numbers.push_back(89); // test sorting
numbers.push_back(2);
diff --git a/tests/net/pop3/POP3UtilsTest.cpp b/tests/net/pop3/POP3UtilsTest.cpp
index 55a99806..cc2cb7d3 100644
--- a/tests/net/pop3/POP3UtilsTest.cpp
+++ b/tests/net/pop3/POP3UtilsTest.cpp
@@ -58,7 +58,7 @@ VMIME_TEST_SUITE_BEGIN(POP3UtilsTest)
vmime::shared_ptr <POP3Response> resp =
POP3Response::readMultilineResponse(conn);
- std::map <int, vmime::string> result;
+ std::map <vmime::index_t, vmime::string> result;
POP3Utils::parseMultiListOrUidlResponse(resp, result);
VASSERT_EQ("Count", 5, result.size());