7 files changed, 28 insertions, 18 deletions
diff --git a/SwifTools/Dock/Dock.h b/SwifTools/Dock/Dock.h
index 6120445..362aabc 100644
--- a/SwifTools/Dock/Dock.h
+++ b/SwifTools/Dock/Dock.h
@@ -1,3 +1,3 @@
 /*
- * Copyright (c) 2010 Isode Limited.
+ * Copyright (c) 2010-2018 Isode Limited.
  * All rights reserved.
@@ -8,2 +8,4 @@
 
+#include <cstddef>
+
 namespace Swift {
@@ -15,3 +17,3 @@ namespace Swift {
 
-            virtual void setNumberOfPendingMessages(int i) = 0;
+            virtual void setNumberOfPendingMessages(size_t i) = 0;
     };
diff --git a/SwifTools/Dock/MacOSXDock.h b/SwifTools/Dock/MacOSXDock.h
index 6b33506..ef85a88 100644
--- a/SwifTools/Dock/MacOSXDock.h
+++ b/SwifTools/Dock/MacOSXDock.h
@@ -1,3 +1,3 @@
 /*
- * Copyright (c) 2010 Isode Limited.
+ * Copyright (c) 2010-2018 Isode Limited.
  * All rights reserved.
@@ -18,3 +18,3 @@ namespace Swift {
 
-            virtual void setNumberOfPendingMessages(int i);
+            virtual void setNumberOfPendingMessages(size_t i);
     };
diff --git a/SwifTools/Dock/MacOSXDock.mm b/SwifTools/Dock/MacOSXDock.mm
index 5c7207c..3935ac0 100644
--- a/SwifTools/Dock/MacOSXDock.mm
+++ b/SwifTools/Dock/MacOSXDock.mm
@@ -1,3 +1,3 @@
 /*
- * Copyright (c) 2015-2016 Isode Limited.
+ * Copyright (c) 2015-2018 Isode Limited.
  * All rights reserved.
@@ -20,3 +20,3 @@ MacOSXDock::MacOSXDock(CocoaApplication*) {
 
-void MacOSXDock::setNumberOfPendingMessages(int i) {
+void MacOSXDock::setNumberOfPendingMessages(size_t i) {
     std::string label(i > 0 ? boost::lexical_cast<std::string>(i) : "");
diff --git a/SwifTools/Dock/NullDock.h b/SwifTools/Dock/NullDock.h
index 9f3c554..137ba03 100644
--- a/SwifTools/Dock/NullDock.h
+++ b/SwifTools/Dock/NullDock.h
@@ -1,3 +1,3 @@
 /*
- * Copyright (c) 2010 Isode Limited.
+ * Copyright (c) 2010-2018 Isode Limited.
  * All rights reserved.
@@ -8,2 +8,4 @@
 
+#include <cstddef>
+
 #include <SwifTools/Dock/Dock.h>
@@ -15,3 +17,3 @@ namespace Swift {
 
-            virtual void setNumberOfPendingMessages(int) {
+            virtual void setNumberOfPendingMessages(size_t) {
             }
diff --git a/SwifTools/Dock/WindowsDock.h b/SwifTools/Dock/WindowsDock.h
index fc10a48..f9a9dae 100644
--- a/SwifTools/Dock/WindowsDock.h
+++ b/SwifTools/Dock/WindowsDock.h
@@ -1,3 +1,3 @@
 /*
- * Copyright (c) 2010-2016 Isode Limited.
+ * Copyright (c) 2010-2018 Isode Limited.
  * All rights reserved.
@@ -20,3 +20,3 @@ namespace Swift {
 
-            virtual void setNumberOfPendingMessages(int i) {
+            virtual void setNumberOfPendingMessages(size_t i) {
                 if (notifier->isAvailable()) {
diff --git a/SwifTools/Idle/MacOSXIdleQuerier.cpp b/SwifTools/Idle/MacOSXIdleQuerier.cpp
index 6d6780b..89fa050 100644
--- a/SwifTools/Idle/MacOSXIdleQuerier.cpp
+++ b/SwifTools/Idle/MacOSXIdleQuerier.cpp
@@ -1,3 +1,3 @@
 /*
- * Copyright (c) 2010 Isode Limited.
+ * Copyright (c) 2010-2018 Isode Limited.
  * All rights reserved.
@@ -12,2 +12,4 @@
 #include <iostream>
+#include <limits>
+
 #include <boost/numeric/conversion/cast.hpp>
@@ -31,3 +33,8 @@ int MacOSXIdleQuerier::getIdleTimeSeconds() {
     CFRelease(property);
-    return boost::numeric_cast<int>(idle / 1000000000);
+    try {
+        return boost::numeric_cast<int>(idle / 1000000000);
+    }
+    catch (const boost::numeric::bad_numeric_cast&) {
+        return std::numeric_limits<int>::max();
+    }
 }
diff --git a/SwifTools/SpellParser.cpp b/SwifTools/SpellParser.cpp
index 5bafa6e..e449b45 100644
--- a/SwifTools/SpellParser.cpp
+++ b/SwifTools/SpellParser.cpp
@@ -7,3 +7,3 @@
 /*
- * Copyright (c) 2016 Isode Limited.
+ * Copyright (c) 2016-2018 Isode Limited.
  * All rights reserved.
@@ -17,3 +17,2 @@
 #include <boost/bind.hpp>
-#include <boost/numeric/conversion/cast.hpp>
 #include <boost/ref.hpp>
@@ -49,10 +48,10 @@ struct counter
             case ID_WWW:
-                position += boost::numeric_cast<size_t>(t.value().size());
+                position += static_cast<std::size_t>(t.value().size());
                 break;
             case ID_HTTP:
-                position += boost::numeric_cast<size_t>(t.value().size());
+                position += static_cast<std::size_t>(t.value().size());
                 break;
             case ID_WORD:       // matched a word
-                wordPositions.push_back(boost::tuples::make_tuple(position, position + boost::numeric_cast<size_t>(t.value().size())));
-                position += boost::numeric_cast<size_t>(t.value().size());
+                wordPositions.push_back(boost::tuples::make_tuple(position, position + static_cast<std::size_t>(t.value().size())));
+                position += static_cast<std::size_t>(t.value().size());
                 break;