Review fixes.

generator/feature_processing_layers.cpp

@@ -262,14 +262,13 @@ void ComplexFeaturesMixer::Process(std::function<void(feature::FeatureBuilder &)
   if (!next)
     return;
 
-  // For all objects in the hierarchy, there must be one areal object and one linear.
-  // Exceptions are point features and parts of buildings.
+  // For rendering purposes all hierarchy objects with closed geometry except building parts must
+  // be stored as one areal object and one linear object.
   if (fb.IsPoint() || !fb.IsGeometryClosed())
     return;
 
   auto const id = MakeCompositeId(fb);
-  auto const it = m_hierarchyNodesSet.find(id);
-  if (it == std::end(m_hierarchyNodesSet))
+  if (m_hierarchyNodesSet.count(id) == 0)
     return;
 
   static auto const & buildingPartChecker = ftypes::IsBuildingPartChecker::Instance();
@@ -280,14 +279,14 @@ void ComplexFeaturesMixer::Process(std::function<void(feature::FeatureBuilder &)
   auto const canBeLine = RepresentationLayer::CanBeLine(fb.GetParams());
   if (!canBeArea)
   {
-    LOG(LINFO, ("Add a areal complex feature for", fb.GetMostGenericOsmId()));
+    LOG(LINFO, ("Adding an areal complex feature for", fb.GetMostGenericOsmId()));
     auto complexFb = MakeComplexAreaFrom(fb);
     next(complexFb);
   }
 
   if (!canBeLine)
   {
-    LOG(LINFO, ("Add a linear complex feature for", fb.GetMostGenericOsmId()));
+    LOG(LINFO, ("Adding a linear complex feature for", fb.GetMostGenericOsmId()));
     auto complexFb = MakeComplexLineFrom(fb);
     next(complexFb);
   }
@@ -301,7 +300,7 @@ feature::FeatureBuilder ComplexFeaturesMixer::MakeComplexLineFrom(feature::Featu
   auto lineFb = MakeLine(fb);
   auto & params = lineFb.GetParams();
   params.ClearName();
-  params.GetMetadata() = {};
+  params.ClearMetadata();
   params.SetType(m_complexEntryType);
   return lineFb;
 }
@@ -315,7 +314,7 @@ feature::FeatureBuilder ComplexFeaturesMixer::MakeComplexAreaFrom(feature::Featu
   areaFb.SetArea();
   auto & params = areaFb.GetParams();
   params.ClearName();
-  params.GetMetadata() = {};
+  params.ClearMetadata();
   params.SetType(m_complexEntryType);
   return areaFb;
 }

generator/feature_processing_layers.hpp

@@ -11,9 +11,11 @@
 #include "generator/promo_catalog_cities.hpp"
 #include "generator/world_map_generator.hpp"
 
+#include <functional>
 #include <memory>
 #include <sstream>
 #include <string>
+#include <unordered_set>
 
 class CoastlineFeaturesGenerator;
 

generator/final_processor_intermediate_mwm.cpp

@@ -120,7 +120,7 @@ void Sort(std::vector<FeatureBuilder> & fbs)
   });
 }
 
-std::string GetCountryNameFormTmpMwmPath(std::string filename)
+std::string GetCountryNameFromTmpMwmPath(std::string filename)
 {
   strings::ReplaceLast(filename, DATA_FILE_EXTENSION_TMP, "");
   return filename;
@@ -128,7 +128,7 @@ std::string GetCountryNameFormTmpMwmPath(std::string filename)
 
 bool FilenameIsCountry(std::string const & filename, AffiliationInterface const & affiliation)
 {
-  return affiliation.HasRegionByName(GetCountryNameFormTmpMwmPath(filename));
+  return affiliation.HasRegionByName(GetCountryNameFromTmpMwmPath(filename));
 }
 
 class PlaceHelper
@@ -644,7 +644,7 @@ void ComplexFinalProcessor::Process()
   std::vector<std::future<std::vector<HierarchyEntry>>> futures;
   ForEachCountry(m_mwmTmpPath, [&](auto const & filename) {
     auto future = pool.Submit([&, filename]() {
-      auto countryName = GetCountryNameFormTmpMwmPath(filename);
+      auto countryName = GetCountryNameFromTmpMwmPath(filename);
       // https://wiki.openstreetmap.org/wiki/Simple_3D_buildings
       // An object with tag 'building:part' is a part of a relation with outline 'building' or
       // is contained in an object with tag 'building'. We will split data and work with

indexer/feature_data.hpp

@@ -313,6 +313,9 @@ public:
   feature::Metadata const & GetMetadata() const { return m_metadata; }
   feature::Metadata & GetMetadata() { return m_metadata; }
 
+  void SetMetadata(feature::Metadata && metadata) { m_metadata = std::move(metadata); }
+  void ClearMetadata() { SetMetadata({}); }
+
   template <class Sink>
   void Write(Sink & sink) const
   {

indexer/feature_visibility.cpp

@@ -223,13 +223,16 @@ namespace
 
   /// Add here all exception classificator types: needed for algorithms,
   /// but don't have drawing rules.
+  /// Warning: Geometry of features with always existing types will be indexed in mwm on all
+  /// zoom levels. If you add an always existing type to drawing types, the displacement of icons
+  /// may work not correctly.
   bool TypeAlwaysExists(uint32_t type, GeomType g = GeomType::Undefined)
   {
     if (!classif().IsTypeValid(type))
       return false;
 
     static uint32_t const internet = classif().GetTypeByPath({"internet_access"});
-    static uint32_t const complex_entry = classif().GetTypeByPath({"complex_entry"});
+    static uint32_t const complexEntry = classif().GetTypeByPath({"complex_entry"});
 
     if ((g == GeomType::Line || g == GeomType::Undefined) && HasRoutingExceptionType(type))
       return true;
@@ -238,7 +241,7 @@ namespace
     if (g != GeomType::Line && type == internet)
       return true;
 
-    if (type == complex_entry)
+    if (type == complexEntry)
       return true;
 
     return false;
@@ -288,6 +291,7 @@ namespace
     // Reserved for custom event processing, e.g. fc2018.
     // if (event == type)
     //   return true;
+
     return false;
   }
 }  // namespace