Changed useless pass by ref argument in gps::getVelocity

GpsTracker/Core.cpp

@@ -13,7 +13,7 @@ namespace core {
 		PositionEntryMetadata metadata;
 		if (positions::acquire(metadata)) {
 			positions::appendLast(metadata);
-			updateSleepTime();
+			setSleepTime();
 		}
 
 		if (positions::needsToSend()) {
@@ -21,12 +21,8 @@ namespace core {
 		}
 	}
 
-	void updateSleepTime() {
-		VERBOSE("updateSleepTime");
-
-		uint8_t velocity;
-		gps::getVelocity(velocity);
-		setSleepTime(velocity);
+	void setSleepTime() {
+		setSleepTime(gps::getVelocity());
 	}
 
 	void setSleepTime(uint8_t velocity) {

GpsTracker/Core.h

@@ -15,5 +15,5 @@ namespace core {
 
 	void main();
 	void setSleepTime(uint8_t velocity);
-	void updateSleepTime();
+	void setSleepTime();
 }

GpsTracker/Gps.cpp

@@ -44,10 +44,13 @@ namespace gps {
 		return currentStatus;
 	}
 
-	void getVelocity(uint8_t &velocity) {
+	uint8_t getVelocity() {
+		uint8_t velocity;
 		hardware::sim808::device.getGpsField(lastPosition, SIM808_GPS_FIELD::SPEED, &velocity);
 
 		VERBOSE_FORMAT("getVelocity", "%d", velocity);
+
+		return velocity;
 	}
 
 	void getTime(tmElements_t &time) {

GpsTracker/Gps.h

@@ -17,6 +17,6 @@ namespace gps {
 
 	SIM808_GPS_STATUS acquireCurrentPosition(uint16_t timeout);
 
-	void getVelocity(uint8_t &velocity);
+	uint8_t getVelocity();
 	void getTime(tmElements_t &time);
 }