From 200f038f2ccab6cd6de3a26fd209e70c48938c24 Mon Sep 17 00:00:00 2001
From: Dan Garry <djgwiki@gmail.com>
Date: Sun, 19 May 2019 14:32:08 +0200
Subject: [PATCH] Remove unnecessary use of String.valueOf() from GPSExtractor
 (#2958)

valueOf() was being used in cases where it wasn't necessary because the method is called implicitly; this patch removes these usages to make the code clearer and easier to read.
---
 app/src/main/java/fr/free/nrw/commons/upload/GPSExtractor.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/upload/GPSExtractor.java b/app/src/main/java/fr/free/nrw/commons/upload/GPSExtractor.java
index a485d53b8..e45e7ec47 100644
--- a/app/src/main/java/fr/free/nrw/commons/upload/GPSExtractor.java
+++ b/app/src/main/java/fr/free/nrw/commons/upload/GPSExtractor.java
@@ -122,7 +122,7 @@ class GPSExtractor {
             decLongitude = 0 - convertToDegree(longitude);
         }
 
-        String decimalCoords = String.valueOf(decLatitude) + "|" + String.valueOf(decLongitude);
+        String decimalCoords = decLatitude + "|" + decLongitude;
         Timber.d("Latitude and Longitude are %s", decimalCoords);
         return decimalCoords;
     }