4 files changed, 4 insertions, 5 deletions
diff --git a/Swiften/Elements/Last.h b/Swiften/Elements/Last.h
index ba302e3..fe0323a 100644
--- a/Swiften/Elements/Last.h
+++ b/Swiften/Elements/Last.h
@@ -11,8 +11,7 @@
 namespace Swift {
 	class Last : public Payload {
 		public:
-			Last() {};
-			Last(int seconds) : seconds_(seconds) {};
+			Last(int seconds = 0) : seconds_(seconds) {};
 
 			int getSeconds() const {return seconds_;}
 			void setSeconds(int seconds) {seconds_ = seconds;}
diff --git a/Swiften/Elements/RosterItemExchangePayload.cpp b/Swiften/Elements/RosterItemExchangePayload.cpp
index a425551..abd5296 100644
--- a/Swiften/Elements/RosterItemExchangePayload.cpp
+++ b/Swiften/Elements/RosterItemExchangePayload.cpp
@@ -9,7 +9,7 @@
 
 namespace Swift {
 
-RosterItemExchangePayload::Item::Item() {
+RosterItemExchangePayload::Item::Item(Action action) : action(action) {
 }
 
 RosterItemExchangePayload::RosterItemExchangePayload() {
diff --git a/Swiften/Elements/RosterItemExchangePayload.h b/Swiften/Elements/RosterItemExchangePayload.h
index cbe9ffa..c39c084 100644
--- a/Swiften/Elements/RosterItemExchangePayload.h
+++ b/Swiften/Elements/RosterItemExchangePayload.h
@@ -23,7 +23,7 @@ namespace Swift {
 				public:
 					enum Action { Add, Modify, Delete };
 
-					Item();
+					Item(Action action = Add);
 
 					Action getAction() const {
 						return action;
diff --git a/Swiften/Elements/VCard.cpp b/Swiften/Elements/VCard.cpp
index 4f7700e..8bf3eb9 100644
--- a/Swiften/Elements/VCard.cpp
+++ b/Swiften/Elements/VCard.cpp
@@ -16,7 +16,7 @@ VCard::EMailAddress VCard::getPreferredEMailAddress() const {
 			return address;
 		}
 	}
-	if (emailAddresses_.size() > 0) {
+	if (!emailAddresses_.empty()) {
 		return emailAddresses_[0];
 	}
 	return EMailAddress();