Review fixes.

generator/cities_boundaries_builder.cpp

@@ -14,11 +14,10 @@
 
 #include "platform/local_country_file.hpp"
 
-#include "coding/compressed_bit_vector.hpp"
-
 #include "base/assert.hpp"
 #include "base/checked_cast.hpp"
 
+#include <cstdint>
 #include <map>
 #include <vector>
 

generator/osm_source.cpp

@@ -1,3 +1,5 @@
+#include "generator/osm_source.hpp"
+
 #include "generator/cities_boundaries_builder.hpp"
 #include "generator/coastlines_generator.hpp"
 #include "generator/feature_generator.hpp"
@@ -5,7 +7,6 @@
 #include "generator/intermediate_elements.hpp"
 #include "generator/osm_element.hpp"
 #include "generator/osm_o5m_source.hpp"
-#include "generator/osm_source.hpp"
 #include "generator/osm_translator.hpp"
 #include "generator/osm_xml_source.hpp"
 #include "generator/polygonizer.hpp"

generator/osm_translator.hpp

@@ -380,9 +380,6 @@ class OsmToFeatureTranslator
     if (!ft.IsGeometryClosed())
       return;
 
-    // Key point here is that IsDrawableLike and RemoveNoDrawableTypes
-    // work a bit different for GEOM_AREA.
-
     if (IsCityBoundary(params))
     {
       auto fb = ft;
@@ -390,6 +387,8 @@ class OsmToFeatureTranslator
       m_emitter.EmitCityBoundary(fb, params);
     }
 
+    // Key point here is that IsDrawableLike and RemoveNoDrawableTypes
+    // work a bit different for GEOM_AREA.
     if (IsDrawableLike(params.m_Types, GEOM_AREA))
     {
       // Make the area feature if it has unique area styles.

generator/utils.hpp

@@ -9,12 +9,14 @@
 
 #include "base/logging.hpp"
 
+#include <string>
+
 namespace generator
 {
 void LoadIndex(Index & index);
 
 template <typename ToDo>
-bool ForEachOsmId2FeatureId(string const & path, ToDo && toDo)
+bool ForEachOsmId2FeatureId(std::string const & path, ToDo && toDo)
 {
   gen::OsmID2FeatureID mapping;
   try

indexer/cities_boundaries_serdes.hpp

@@ -18,6 +18,7 @@
 #include "geometry/point2d.hpp"
 
 #include "base/assert.hpp"
+#include "base/checked_cast.hpp"
 #include "base/logging.hpp"
 #include "base/macros.hpp"
 #include "base/visitor.hpp"
@@ -147,8 +148,10 @@ public:
 
     void EncodeDelta(m2::PointU const & curr, m2::PointU const & next)
     {
-      auto const dx = static_cast<int32_t>(next.x) - static_cast<int32_t>(curr.x);
-      auto const dy = static_cast<int32_t>(next.y) - static_cast<int32_t>(curr.y);
+      auto const dx = base::asserted_cast<int32_t>(next.x) -
+                      base::asserted_cast<int32_t>(curr.x);
+      auto const dy = base::asserted_cast<int32_t>(next.y) -
+                      base::asserted_cast<int32_t>(curr.y);
       WriteVarInt(m_sink, dx);
       WriteVarInt(m_sink, dy);
     }
@@ -308,7 +311,7 @@ public:
     {
       auto const dx = ReadVarUint<uint32_t>(m_source);
       auto const dy = ReadVarUint<uint32_t>(m_source);
-      return m2::PointU(static_cast<uint32_t>(dx), static_cast<uint32_t>(dy));
+      return m2::PointU(dx, dy);
     }
 
     Source & m_source;

indexer/indexer_tests/cities_boundaries_serdes_tests.cpp

@@ -26,7 +26,7 @@ using Boundaries = vector<Boundary>;
 static_assert(CitiesBoundariesSerDes::kLatestVersion == 0, "");
 static_assert(CitiesBoundariesSerDes::HeaderV0::kDefaultCoordBits == 19, "");
 
-// Precision of mercator coords encoded with 19 bits.
+// Absolute precision of mercator coords encoded with 19 bits.
 double const kEps = 1e-3;
 
 void TestEqual(vector<PointD> const & lhs, vector<PointD> const & rhs)