Remove std::ostringstream from TileKey::ToHereTile (#1351)

andrey-kashcheev · web-flow · commit 286d025541d2 · 2022-07-22T10:24:48.000+02:00
Not the most efficient way of converting number to string. Replace
with dedicated std::to_string.
`ToHereTile` is being used a lot in the logs so any performance gains
are welcomed.

Relates-To: OAM-1686
Signed-off-by: Andrey Kashcheev &lt;ext-andrey.kashcheev@here.com&gt;
diff --git a/olp-cpp-sdk-core/src/geo/tiling/TileKey.cpp b/olp-cpp-sdk-core/src/geo/tiling/TileKey.cpp
@@ -76,9 +76,7 @@ TileKey TileKey::FromQuadKey(const std::string& quad_key) {
 }
 
 std::string TileKey::ToHereTile() const {
-  std::ostringstream os;
-  os << std::dec << ToQuadKey64();
-  return os.str();
+  return std::to_string(ToQuadKey64());
 }
 
 TileKey TileKey::FromHereTile(const std::string& key) {

Original file line number	Diff line number	Diff line change
`@@ -76,9 +76,7 @@ TileKey TileKey::FromQuadKey(const std::string& quad_key) {`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`std::string TileKey::ToHereTile() const {`
`79`		`- std::ostringstream os;`
`80`		`- os << std::dec << ToQuadKey64();`
`81`		`- return os.str();`
	`79`	`+ return std::to_string(ToQuadKey64());`
`82`	`80`	`}`
`83`	`81`
`84`	`82`	`TileKey TileKey::FromHereTile(const std::string& key) {`