summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTobias Markmann <tm@ayena.de>2016-06-10 12:38:40 (GMT)
committerTobias Markmann <tm@ayena.de>2016-06-10 13:11:04 (GMT)
commitf038599ba697acc17a328ef4ade5514fede8887e (patch)
tree1a7cb59f2490bbfd60fff2c92e147d93ea6d1166 /Swiften/Parser/PayloadParsers
parent33ffcd9bf5bc9806d0b404aae2aaf6d67a2945db (diff)
downloadswift-f038599ba697acc17a328ef4ade5514fede8887e.zip
swift-f038599ba697acc17a328ef4ade5514fede8887e.tar.bz2
Fix remaining compiler warnings on OS X with clang
This changes HippoMocks include to -isystem include so it will not cause compiler warnings. This changes UserLocation floating point values to double and remove remaining numerical casts. Change C++11 compiler test to not cause warnings. Test-Information: Builds with allow_warnings=0 on OS X 10.11.5 with Xcode clang. Change-Id: I26c60265a86c9be656089ade63fe9ad63851580f
Diffstat (limited to 'Swiften/Parser/PayloadParsers')
-rw-r--r--Swiften/Parser/PayloadParsers/UnitTest/UserLocationParserTest.cpp24
-rw-r--r--Swiften/Parser/PayloadParsers/UserLocationParser.cpp16
2 files changed, 20 insertions, 20 deletions
diff --git a/Swiften/Parser/PayloadParsers/UnitTest/UserLocationParserTest.cpp b/Swiften/Parser/PayloadParsers/UnitTest/UserLocationParserTest.cpp
index 6d16377..98a147f 100644
--- a/Swiften/Parser/PayloadParsers/UnitTest/UserLocationParserTest.cpp
+++ b/Swiften/Parser/PayloadParsers/UnitTest/UserLocationParserTest.cpp
@@ -46,10 +46,10 @@ class UserLocationParserTest : public CppUnit::TestFixture
UserLocation* payload = dynamic_cast<UserLocation*>(parser.getPayload().get());
CPPUNIT_ASSERT(payload);
CPPUNIT_ASSERT_EQUAL(std::string("Barbaric"), payload->getArea().get());
- CPPUNIT_ASSERT_EQUAL(5.75F, payload->getAltitude().get());
+ CPPUNIT_ASSERT_EQUAL(5.75, payload->getAltitude().get());
CPPUNIT_ASSERT_EQUAL(std::string("Near"), payload->getLocality().get());
- CPPUNIT_ASSERT_EQUAL(1.670F, payload->getLatitude().get());
- CPPUNIT_ASSERT_EQUAL(0.95F, payload->getAccuracy().get());
+ CPPUNIT_ASSERT_EQUAL(1.670, payload->getLatitude().get());
+ CPPUNIT_ASSERT_EQUAL(0.95, payload->getAccuracy().get());
CPPUNIT_ASSERT_EQUAL(std::string("Nice"), payload->getDescription().get());
CPPUNIT_ASSERT_EQUAL(std::string("+91"), payload->getCountryCode().get());
CPPUNIT_ASSERT_EQUAL(std::string("2015-06-11T20:55:50Z"), dateTimeToString(payload->getTimestamp().get()));
@@ -59,14 +59,14 @@ class UserLocationParserTest : public CppUnit::TestFixture
CPPUNIT_ASSERT_EQUAL(std::string("USA"), payload->getCountry().get());
CPPUNIT_ASSERT_EQUAL(std::string("NewSode"), payload->getRegion().get());
CPPUNIT_ASSERT_EQUAL(std::string("URIs"), payload->getURI().get());
- CPPUNIT_ASSERT_EQUAL(6.7578F, payload->getLongitude().get());
- CPPUNIT_ASSERT_EQUAL(5.66F, payload->getError().get());
+ CPPUNIT_ASSERT_EQUAL(6.7578, payload->getLongitude().get());
+ CPPUNIT_ASSERT_EQUAL(5.66, payload->getError().get());
CPPUNIT_ASSERT_EQUAL(std::string("67"), payload->getPostalCode().get());
- CPPUNIT_ASSERT_EQUAL(12.89F, payload->getBearing().get());
+ CPPUNIT_ASSERT_EQUAL(12.89, payload->getBearing().get());
CPPUNIT_ASSERT_EQUAL(std::string("Hello"), payload->getText().get());
CPPUNIT_ASSERT_EQUAL(std::string("Datee"), payload->getDatum().get());
CPPUNIT_ASSERT_EQUAL(std::string("Highway"), payload->getStreet().get());
- CPPUNIT_ASSERT_EQUAL(56.77F, payload->getSpeed().get());
+ CPPUNIT_ASSERT_EQUAL(56.77, payload->getSpeed().get());
}
void testParse_with_Some_variables() {
@@ -83,10 +83,10 @@ class UserLocationParserTest : public CppUnit::TestFixture
UserLocation* payload = dynamic_cast<UserLocation*>(parser.getPayload().get());
CPPUNIT_ASSERT(payload);
CPPUNIT_ASSERT_EQUAL(std::string("Barbaric"), payload->getArea().get());
- CPPUNIT_ASSERT_EQUAL(5.75F, payload->getAltitude().get());
+ CPPUNIT_ASSERT_EQUAL(5.75, payload->getAltitude().get());
CPPUNIT_ASSERT_EQUAL(std::string("Near"), payload->getLocality().get());
CPPUNIT_ASSERT(!payload->getLatitude());
- CPPUNIT_ASSERT_EQUAL(0.95F, payload->getAccuracy().get());
+ CPPUNIT_ASSERT_EQUAL(0.95, payload->getAccuracy().get());
CPPUNIT_ASSERT_EQUAL(std::string("Nice"), payload->getDescription().get());
CPPUNIT_ASSERT_EQUAL(std::string("+91"), payload->getCountryCode().get());
CPPUNIT_ASSERT_EQUAL(std::string("2015-06-11T20:55:50Z"), dateTimeToString(payload->getTimestamp().get()));
@@ -96,10 +96,10 @@ class UserLocationParserTest : public CppUnit::TestFixture
CPPUNIT_ASSERT(!payload->getCountry());
CPPUNIT_ASSERT_EQUAL(std::string("NewSode"), payload->getRegion().get());
CPPUNIT_ASSERT_EQUAL(std::string("URIs"), payload->getURI().get());
- CPPUNIT_ASSERT_EQUAL(6.7578F, payload->getLongitude().get());
- CPPUNIT_ASSERT_EQUAL(5.66F, payload->getError().get());
+ CPPUNIT_ASSERT_EQUAL(6.7578, payload->getLongitude().get());
+ CPPUNIT_ASSERT_EQUAL(5.66, payload->getError().get());
CPPUNIT_ASSERT_EQUAL(std::string("67"), payload->getPostalCode().get());
- CPPUNIT_ASSERT_EQUAL(12.89F, payload->getBearing().get());
+ CPPUNIT_ASSERT_EQUAL(12.89, payload->getBearing().get());
CPPUNIT_ASSERT_EQUAL(std::string("Hello"), payload->getText().get());
CPPUNIT_ASSERT(!payload->getDatum());
CPPUNIT_ASSERT(!payload->getStreet());
diff --git a/Swiften/Parser/PayloadParsers/UserLocationParser.cpp b/Swiften/Parser/PayloadParsers/UserLocationParser.cpp
index 77a0fa1..09d0e9a 100644
--- a/Swiften/Parser/PayloadParsers/UserLocationParser.cpp
+++ b/Swiften/Parser/PayloadParsers/UserLocationParser.cpp
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2013 Isode Limited.
+ * Copyright (c) 2013-2016 Isode Limited.
* All rights reserved.
* See the COPYING file for more information.
*/
@@ -30,16 +30,16 @@ void UserLocationParser::handleEndElement(const std::string& element, const std:
if (level == 1) {
try {
if (element == "accuracy") {
- getPayloadInternal()->setAccuracy(boost::lexical_cast<float>(currentText));
+ getPayloadInternal()->setAccuracy(boost::lexical_cast<double>(currentText));
}
else if (element == "alt") {
- getPayloadInternal()->setAltitude(boost::lexical_cast<float>(currentText));
+ getPayloadInternal()->setAltitude(boost::lexical_cast<double>(currentText));
}
else if (element == "area") {
getPayloadInternal()->setArea(currentText);
}
else if (element == "bearing") {
- getPayloadInternal()->setBearing(boost::lexical_cast<float>(currentText));
+ getPayloadInternal()->setBearing(boost::lexical_cast<double>(currentText));
}
else if (element == "building") {
getPayloadInternal()->setBuilding(currentText);
@@ -57,19 +57,19 @@ void UserLocationParser::handleEndElement(const std::string& element, const std:
getPayloadInternal()->setDescription(currentText);
}
else if (element == "error") {
- getPayloadInternal()->setError(boost::lexical_cast<float>(currentText));
+ getPayloadInternal()->setError(boost::lexical_cast<double>(currentText));
}
else if (element == "floor") {
getPayloadInternal()->setFloor(currentText);
}
else if (element == "lat") {
- getPayloadInternal()->setLatitude(boost::lexical_cast<float>(currentText));
+ getPayloadInternal()->setLatitude(boost::lexical_cast<double>(currentText));
}
else if (element == "locality") {
getPayloadInternal()->setLocality(currentText);
}
else if (element == "lon") {
- getPayloadInternal()->setLongitude(boost::lexical_cast<float>(currentText));
+ getPayloadInternal()->setLongitude(boost::lexical_cast<double>(currentText));
}
else if (element == "postalcode") {
getPayloadInternal()->setPostalCode(currentText);
@@ -81,7 +81,7 @@ void UserLocationParser::handleEndElement(const std::string& element, const std:
getPayloadInternal()->setRoom(currentText);
}
else if (element == "speed") {
- getPayloadInternal()->setSpeed(boost::lexical_cast<float>(currentText));
+ getPayloadInternal()->setSpeed(boost::lexical_cast<double>(currentText));
}
else if (element == "street") {
getPayloadInternal()->setStreet(currentText);