From 214aa559a9d96aeecbd377e9fc3f976541c4f957 Mon Sep 17 00:00:00 2001 From: Konstantin Pastbin Date: Tue, 26 Sep 2023 12:11:49 +0300 Subject: [PATCH 1/5] [drape] Refactor apply_feature_functors Optimizations: - don't parse a road ref if there is no shield drule - don't process a shield drule if there are no road shields in a ref - call ClipSplineByRect() once per linear feature (not for each line drule) - calculate areaDepth for BG-by-size areas only ...and many minor things like replace redundant checks with asserts, less params wrapping and passing, simpler structure, etc. Signed-off-by: Konstantin Pastbin --- drape_frontend/apply_feature_functors.cpp | 446 ++++++++++------------ drape_frontend/apply_feature_functors.hpp | 71 ++-- drape_frontend/line_shape.cpp | 4 +- drape_frontend/rule_drawer.cpp | 94 ++--- drape_frontend/rule_drawer.hpp | 5 +- drape_frontend/shape_view_params.hpp | 9 +- drape_frontend/stylist.cpp | 71 ++-- drape_frontend/stylist.hpp | 21 +- drape_frontend/visual_params.cpp | 14 - drape_frontend/visual_params.hpp | 27 +- geometry/rect2d.hpp | 3 +- indexer/drawing_rule_def.hpp | 3 - indexer/drawing_rules.cpp | 83 +--- indexer/drawing_rules.hpp | 8 +- map/transit/transit_reader.cpp | 6 +- 15 files changed, 365 insertions(+), 500 deletions(-) diff --git a/drape_frontend/apply_feature_functors.cpp b/drape_frontend/apply_feature_functors.cpp index a5d922add0..faccc9d2ff 100644 --- a/drape_frontend/apply_feature_functors.cpp +++ b/drape_frontend/apply_feature_functors.cpp @@ -13,7 +13,6 @@ #include "editor/osm_editor.hpp" -#include "indexer/drawing_rules.hpp" #include "indexer/drules_include.hpp" #include "indexer/feature_source.hpp" #include "indexer/map_style_reader.hpp" @@ -38,7 +37,6 @@ #include #include - namespace df { dp::Color ToDrapeColor(uint32_t src) @@ -119,11 +117,10 @@ private: }; #endif -void ExtractLineParams(::LineRuleProto const * lineRule, df::LineViewParams & params) +void ExtractLineParams(LineRuleProto const * lineRule, LineViewParams & params) { double const scale = df::VisualParams::Instance().GetVisualScale(); params.m_color = ToDrapeColor(lineRule->color()); - ASSERT_GREATER(lineRule->width(), 0, ("Zero width line (no pathsym)")); params.m_width = static_cast(std::max(lineRule->width() * scale, 1.0)); if (lineRule->has_dashdot()) @@ -217,13 +214,18 @@ bool IsSymbolRoadShield(ftypes::RoadShield const & shield) std::string GetRoadShieldSymbolName(ftypes::RoadShield const & shield, double fontScale) { + ASSERT(IsSymbolRoadShield(shield), ()); std::string result = ""; if (shield.m_type == ftypes::RoadShieldType::US_Interstate) result = shield.m_name.size() <= 2 ? "shield-us-i-thin" : "shield-us-i-wide"; else if (shield.m_type == ftypes::RoadShieldType::US_Highway) result = shield.m_name.size() <= 2 ? "shield-us-hw-thin" : "shield-us-hw-wide"; + else + { + ASSERT(false, ()); + } - if (!result.empty() && fontScale > 1.0) + if (fontScale > 1.0) result += "-scaled"; return result; @@ -306,6 +308,7 @@ dp::Anchor GetShieldAnchor(uint8_t shieldIndex, uint8_t shieldCount) else if (shieldIndex == 2) return dp::RightTop; return dp::LeftTop; } + // A single shield. return dp::Center; } @@ -355,10 +358,22 @@ BaseApplyFeature::BaseApplyFeature(TileKey const & tileKey, TInsertShapeFn const ASSERT(m_insertShape != nullptr, ()); } -void BaseApplyFeature::ExtractCaptionParams(CaptionDefProto const * primaryProto, +void BaseApplyFeature::FillCommonParams(CommonOverlayViewParams & p) const +{ + p.m_rank = m_f.GetRank(); + p.m_tileCenter = m_tileRect.Center(); + p.m_featureId = m_f.GetID(); +} + +void ApplyPointFeature::ExtractCaptionParams(CaptionDefProto const * primaryProto, CaptionDefProto const * secondaryProto, TextViewParams & params) const { + FillCommonParams(params); + params.m_depthLayer = DepthLayer::OverlayLayer; + params.m_depthTestEnabled = false; + params.m_posZ = m_posZ; + auto & titleDecl = params.m_titleDecl; dp::FontDecl decl; @@ -421,93 +436,38 @@ double BaseApplyFeature::PriorityToDepth(int priority, drule::rule_type_t ruleTy } ApplyPointFeature::ApplyPointFeature(TileKey const & tileKey, TInsertShapeFn const & insertShape, FeatureType & f, - CaptionDescription const & captions, float posZ) + CaptionDescription const & captions) : TBase(tileKey, insertShape, f, captions) - , m_posZ(posZ) - , m_symbolDepth(dp::kMinDepth) {} -void ApplyPointFeature::operator()(m2::PointD const & point, bool hasArea) +void ApplyPointFeature::ProcessPointRules(SymbolRuleProto const * symbolRule, CaptionRuleProto const * captionRule, + CaptionRuleProto const * houseNumberRule, m2::PointD const & centerPoint, + ref_ptr texMng) { - m_hasArea = hasArea; - m_centerPoint = point; - // TODO: This is only one place of cross-dependency with Editor. auto const & editor = osm::Editor::Instance(); auto const featureStatus = editor.GetFeatureStatus(m_f.GetID()); - m_createdByEditor = featureStatus == FeatureStatus::Created; - m_obsoleteInEditor = featureStatus == FeatureStatus::Obsolete; -} + bool const createdByEditor = featureStatus == FeatureStatus::Created; + bool const obsoleteInEditor = featureStatus == FeatureStatus::Obsolete; -void ApplyPointFeature::ProcessPointRule(drule::BaseRule const * rule, bool isHouseNumber) -{ - SymbolRuleProto const * symRule = rule->GetSymbol(); - if (symRule) - { - m_symbolDepth = PriorityToDepth(symRule->priority(), drule::symbol, 0); - m_symbolRule = symRule; - return; - } - - CaptionDefProto const * capRule = rule->GetCaption(0); - if (capRule) - { - CaptionDefProto const * auxRule = rule->GetCaption(1); - TextViewParams params; - - if (isHouseNumber) - params.m_titleDecl.m_primaryText = m_captions.GetHouseNumberText(); - else - { - params.m_titleDecl.m_primaryText = m_captions.GetMainText(); - if (auxRule != nullptr) - params.m_titleDecl.m_secondaryText = m_captions.GetAuxText(); - } - ASSERT(!params.m_titleDecl.m_primaryText.empty(), ()); - - ExtractCaptionParams(capRule, auxRule, params); - params.m_depth = PriorityToDepth(rule->GetCaptionsPriority(), drule::caption, 0); - params.m_featureId = m_f.GetID(); - params.m_tileCenter = m_tileRect.Center(); - params.m_depthLayer = DepthLayer::OverlayLayer; - params.m_depthTestEnabled = false; - params.m_rank = m_f.GetRank(); - params.m_posZ = m_posZ; - params.m_hasArea = m_hasArea; - params.m_createdByEditor = m_createdByEditor; - - if (isHouseNumber) - m_hnParams = params; - else - m_textParams = params; - } -} - -void ApplyPointFeature::Finish(ref_ptr texMng) -{ m2::PointF symbolSize(0.0f, 0.0f); - bool const hasIcon = m_symbolRule != nullptr; - auto const & visualParams = df::VisualParams::Instance(); - double const mainScale = visualParams.GetVisualScale(); - if (hasIcon) + if (symbolRule) { - double const poiExtendScale = visualParams.GetPoiExtendScale(); - PoiSymbolViewParams params; - params.m_featureId = m_f.GetID(); - params.m_tileCenter = m_tileRect.Center(); - params.m_depthTestEnabled = false; - params.m_depth = m_symbolDepth; + FillCommonParams(params); params.m_depthLayer = DepthLayer::OverlayLayer; - params.m_rank = m_f.GetRank(); - params.m_symbolName = m_symbolRule->name(); - ASSERT_GREATER_OR_EQUAL(m_symbolRule->min_distance(), 0, ()); - params.m_extendingSize = static_cast(mainScale * m_symbolRule->min_distance() * poiExtendScale); + params.m_depthTestEnabled = false; + params.m_depth = PriorityToDepth(symbolRule->priority(), drule::symbol, 0); + params.m_symbolName = symbolRule->name(); + ASSERT_GREATER_OR_EQUAL(symbolRule->min_distance(), 0, ()); + auto const & vp = df::VisualParams::Instance(); + params.m_extendingSize = static_cast(vp.GetVisualScale() * symbolRule->min_distance() * + vp.GetPoiExtendScale()); params.m_posZ = m_posZ; - params.m_hasArea = m_hasArea; - params.m_prioritized = m_createdByEditor; - if (m_obsoleteInEditor) + params.m_hasArea = HasArea(); + params.m_prioritized = createdByEditor; + if (obsoleteInEditor) params.m_maskColor = kPoiDeletedMaskColor; dp::TextureManager::SymbolRegion region; @@ -515,43 +475,66 @@ void ApplyPointFeature::Finish(ref_ptr texMng) symbolSize = region.GetPixelSize(); if (region.IsValid()) - m_insertShape(make_unique_dp(m_centerPoint, params, m_tileKey, 0 /* textIndex */)); + m_insertShape(make_unique_dp(centerPoint, params, m_tileKey, 0 /* textIndex */)); else LOG(LERROR, ("Style error. Symbol name must be valid for feature", m_f.GetID())); } - bool const hasText = !m_textParams.m_titleDecl.m_primaryText.empty(); - bool const hasHouseNumber = !m_hnParams.m_titleDecl.m_primaryText.empty(); - if (hasText) + if (captionRule) { + TextViewParams params; + CaptionDefProto const * capRule = &captionRule->primary(); + CaptionDefProto const * auxRule = captionRule->has_secondary() ? &captionRule->secondary() : nullptr; + + params.m_titleDecl.m_primaryText = m_captions.GetMainText(); + if (auxRule) + params.m_titleDecl.m_secondaryText = m_captions.GetAuxText(); + ASSERT(!params.m_titleDecl.m_primaryText.empty(), ()); + + ExtractCaptionParams(capRule, auxRule, params); + params.m_depth = PriorityToDepth(captionRule->priority(), drule::caption, 0); + params.m_hasArea = HasArea(); + params.m_createdByEditor = createdByEditor; + /// @todo Hardcoded styles-bug patch. The patch is ok, but probably should enhance (or fire assert) styles? /// @see https://github.com/organicmaps/organicmaps/issues/2573 - if ((hasIcon || hasHouseNumber) && m_textParams.m_titleDecl.m_anchor == dp::Anchor::Center) + if ((symbolRule || houseNumberRule) && params.m_titleDecl.m_anchor == dp::Anchor::Center) { - ASSERT(!hasIcon, ("A `text-offset: *` is not set in styles.", m_f.GetID(), m_textParams.m_titleDecl.m_primaryText)); - m_textParams.m_titleDecl.m_anchor = GetAnchor(0, 1); + ASSERT(!symbolRule, ("A `text-offset: *` is not set in styles.", m_f.GetID(), params.m_titleDecl.m_primaryText)); + params.m_titleDecl.m_anchor = GetAnchor(0, 1); } - m_textParams.m_startOverlayRank = hasIcon ? dp::OverlayRank1 : dp::OverlayRank0; - auto shape = make_unique_dp(m_centerPoint, m_textParams, m_tileKey, symbolSize, + params.m_startOverlayRank = symbolRule ? dp::OverlayRank1 : dp::OverlayRank0; + auto shape = make_unique_dp(centerPoint, params, m_tileKey, symbolSize, m2::PointF(0.0f, 0.0f) /* symbolOffset */, dp::Center /* symbolAnchor */, 0 /* textIndex */); m_insertShape(std::move(shape)); } - if (hasHouseNumber) + if (houseNumberRule) { + TextViewParams params; + CaptionDefProto const * capRule = &houseNumberRule->primary(); + + params.m_titleDecl.m_primaryText = m_captions.GetHouseNumberText(); + ASSERT(!params.m_titleDecl.m_primaryText.empty(), ()); + + ExtractCaptionParams(capRule, nullptr, params); + params.m_depth = PriorityToDepth(houseNumberRule->priority(), drule::caption, 0); + params.m_hasArea = HasArea(); + params.m_createdByEditor = createdByEditor; + // If icon or main text exists then put housenumber above them. - if (hasIcon || hasText) + if (symbolRule || captionRule) { - m_hnParams.m_titleDecl.m_anchor = GetAnchor(0, -1); - if (hasIcon) + params.m_titleDecl.m_anchor = GetAnchor(0, -1); + if (symbolRule) { - m_hnParams.m_titleDecl.m_primaryOptional = true; - m_hnParams.m_startOverlayRank = dp::OverlayRank1; + params.m_titleDecl.m_primaryOptional = true; + params.m_startOverlayRank = dp::OverlayRank1; } } - m_insertShape(make_unique_dp(m_centerPoint, m_hnParams, m_tileKey, symbolSize, + m_insertShape(make_unique_dp(centerPoint, params, m_tileKey, symbolSize, m2::PointF(0.0f, 0.0f) /* symbolOffset */, dp::Center /* symbolAnchor */, 0 /* textIndex */)); } @@ -561,12 +544,14 @@ ApplyAreaFeature::ApplyAreaFeature(TileKey const & tileKey, TInsertShapeFn const FeatureType & f, double currentScaleGtoP, bool isBuilding, bool skipAreaGeometry, float minPosZ, float posZ, CaptionDescription const & captions) - : TBase(tileKey, insertShape, f, captions, posZ) + : TBase(tileKey, insertShape, f, captions) , m_minPosZ(minPosZ) , m_isBuilding(isBuilding) , m_skipAreaGeometry(skipAreaGeometry) , m_currentScaleGtoP(currentScaleGtoP) -{} +{ + m_posZ = posZ; +} void ApplyAreaFeature::operator()(m2::PointD const & p1, m2::PointD const & p2, m2::PointD const & p3) { @@ -579,11 +564,18 @@ void ApplyAreaFeature::operator()(m2::PointD const & p1, m2::PointD const & p2, m2::PointD const v1 = p2 - p1; m2::PointD const v2 = p3 - p1; + //TODO : degenerate triangles filtering should be done in the generator. + // ASSERT(!v1.IsAlmostZero() && !v2.IsAlmostZero(), ()); if (v1.IsAlmostZero() || v2.IsAlmostZero()) return; double const crossProduct = m2::CrossProduct(v1.Normalize(), v2.Normalize()); double constexpr kEps = 1e-7; + // ASSERT_GREATER_OR_EQUAL(fabs(crossProduct), kEps, (fabs(crossProduct), p1, p2, p3, m_f.DebugString(19, true))); + // TODO : e.g. a landuse-meadow has a following triangle with two identical points: + // m2::Point(8.5829683287662987823, 53.929641499591184584) + // m2::Point(8.5830675705005887721, 53.930025055483156393) + // m2::Point(8.5830675705005887721, 53.930025055483156393) if (fabs(crossProduct) < kEps) return; @@ -605,6 +597,7 @@ void ApplyAreaFeature::ProcessBuildingPolygon(m2::PointD const & p1, m2::PointD { // For building we must filter degenerate polygons because now we have to reconstruct // building outline by bunch of polygons. + // TODO : filter degenerates in the generator.(see a TODO above). m2::PointD const v1 = p2 - p1; m2::PointD const v2 = p3 - p1; if (v1.IsAlmostZero() || v2.IsAlmostZero()) @@ -643,6 +636,7 @@ int ApplyAreaFeature::GetIndex(m2::PointD const & pt) { for (size_t i = 0; i < m_points.size(); i++) { + // TODO : should be possible to use exact match. if (pt.EqualDxDy(m_points[i], mercator::kPointEqualityEps)) return static_cast(i); } @@ -678,6 +672,7 @@ m2::PointD ApplyAreaFeature::CalculateNormal(m2::PointD const & p1, m2::PointD c void ApplyAreaFeature::BuildEdges(int vertexIndex1, int vertexIndex2, int vertexIndex3, bool twoSide) { // Check if triangle is degenerate. + // TODO : filter degenerates in the generator. if (vertexIndex1 == vertexIndex2 || vertexIndex2 == vertexIndex3 || vertexIndex1 == vertexIndex3) return; @@ -726,57 +721,70 @@ void ApplyAreaFeature::CalculateBuildingOutline(bool calculateNormals, BuildingO } } -void ApplyAreaFeature::ProcessAreaRule(drule::BaseRule const * rule, double areaDepth, bool isHatching) +void ApplyAreaFeature::ProcessAreaRules(AreaRuleProto const * areaRule, AreaRuleProto const * hatchingRule) { - AreaRuleProto const * areaRule = rule->GetArea(); - ASSERT(areaRule != nullptr, ()); - if (!m_triangles.empty()) + ASSERT(areaRule || hatchingRule, ()); + ASSERT(HasGeometry(), ()); + + double areaDepth = drule::kBaseDepthBgBySize - 1; + + if (hatchingRule) { - AreaViewParams params; - params.m_tileCenter = m_tileRect.Center(); - params.m_depth = PriorityToDepth(areaRule->priority(), drule::area, areaDepth); - params.m_color = ToDrapeColor(areaRule->color()); - params.m_rank = m_f.GetRank(); - params.m_minPosZ = m_minPosZ; - params.m_posZ = m_posZ; - params.m_hatching = isHatching; - params.m_baseGtoPScale = static_cast(m_currentScaleGtoP); + ASSERT_GREATER_OR_EQUAL(hatchingRule->priority(), drule::kBasePriorityFg, ()); + ProcessRule(hatchingRule, areaDepth, true); + } - BuildingOutline outline; - if (m_isBuilding && !params.m_hatching) - { - /// @todo Make borders work for non-building areas too. - outline.m_generateOutline = areaRule->has_border() && - areaRule->color() != areaRule->border().color() && - areaRule->border().width() > 0.0; - if (outline.m_generateOutline) - params.m_outlineColor = ToDrapeColor(areaRule->border().color()); - - bool const calculateNormals = m_posZ > 0.0; - if (calculateNormals || outline.m_generateOutline) - CalculateBuildingOutline(calculateNormals, outline); - - params.m_is3D = !outline.m_indices.empty() && calculateNormals; - } - - m_insertShape(make_unique_dp(params.m_hatching ? m_triangles : std::move(m_triangles), - std::move(outline), params)); + if (areaRule) + { + // Calculate areaDepth for BG-by-size areas only. + if (areaRule->priority() < drule::kBasePriorityBgTop) + areaDepth = drule::CalcAreaBySizeDepth(m_f); + ProcessRule(areaRule, areaDepth, false); } } +void ApplyAreaFeature::ProcessRule(AreaRuleProto const * areaRule, double areaDepth, bool isHatching) +{ + AreaViewParams params; + params.m_tileCenter = m_tileRect.Center(); + params.m_depth = PriorityToDepth(areaRule->priority(), drule::area, areaDepth); + params.m_color = ToDrapeColor(areaRule->color()); + params.m_rank = m_f.GetRank(); + params.m_minPosZ = m_minPosZ; + params.m_posZ = m_posZ; + params.m_hatching = isHatching; + params.m_baseGtoPScale = m_currentScaleGtoP; + + BuildingOutline outline; + if (m_isBuilding && !isHatching) + { + /// @todo Make borders work for non-building areas too. + outline.m_generateOutline = areaRule->has_border() && + areaRule->color() != areaRule->border().color() && + areaRule->border().width() > 0.0; + if (outline.m_generateOutline) + params.m_outlineColor = ToDrapeColor(areaRule->border().color()); + + bool const calculateNormals = m_posZ > 0.0; + if (calculateNormals || outline.m_generateOutline) + CalculateBuildingOutline(calculateNormals, outline); + + params.m_is3D = !outline.m_indices.empty() && calculateNormals; + } + + m_insertShape(make_unique_dp(m_triangles, std::move(outline), params)); +} + ApplyLineFeatureGeometry::ApplyLineFeatureGeometry(TileKey const & tileKey, TInsertShapeFn const & insertShape, - FeatureType & f, double currentScaleGtoP, - size_t pointsCount, bool smooth) + FeatureType & f, double currentScaleGtoP) : TBase(tileKey, insertShape, f, CaptionDescription()) - , m_currentScaleGtoP(static_cast(currentScaleGtoP)) + , m_currentScaleGtoP(currentScaleGtoP) + // TODO: calculate just once in the RuleDrawer. , m_minSegmentSqrLength(base::Pow2(4.0 * df::VisualParams::Instance().GetVisualScale() / currentScaleGtoP)) , m_simplify(tileKey.m_zoomLevel >= 10 && tileKey.m_zoomLevel <= 12) - , m_smooth(smooth) - , m_initialPointsCount(pointsCount) -#ifdef LINES_GENERATION_CALC_FILTERED_POINTS - , m_readCount(0) -#endif -{} +{ + m_spline.Reset(new m2::Spline(f.GetPointsCount())); +} void ApplyLineFeatureGeometry::operator() (m2::PointD const & point) { @@ -784,9 +792,6 @@ void ApplyLineFeatureGeometry::operator() (m2::PointD const & point) ++m_readCount; #endif - if (m_spline.IsNull()) - m_spline.Reset(new m2::Spline(m_initialPointsCount)); - if (m_spline->IsEmpty()) { m_spline->AddPoint(point); @@ -808,38 +813,33 @@ void ApplyLineFeatureGeometry::operator() (m2::PointD const & point) } } -bool ApplyLineFeatureGeometry::HasGeometry() const -{ - return m_spline->IsValid(); -} - -void ApplyLineFeatureGeometry::ProcessLineRule(drule::BaseRule const * rule) +void ApplyLineFeatureGeometry::ProcessLineRules(Stylist::LineRulesT const & lineRules) { + ASSERT(!lineRules.empty(), ()); ASSERT(HasGeometry(), ()); - LineRuleProto const * pLineRule = rule->GetLine(); - ASSERT(pLineRule != nullptr, ()); - - if (!m_smooth) + if (!ftypes::IsIsolineChecker::Instance()(m_f)) { // A line crossing the tile several times will be split in several parts. + // TODO : use feature's pre-calculated limitRect when possible. m_clippedSplines = m2::ClipSplineByRect(m_tileRect, m_spline); } else { // Isolines smoothing. + ASSERT_EQUAL(lineRules.size(), 1, ()); m2::GuidePointsForSmooth guidePointsForSmooth; std::vector> clippedPaths; auto extTileRect = m_tileRect; extTileRect.Inflate(m_tileRect.SizeX() * 0.3, m_tileRect.SizeY() * 0.3); - m2::ClipPathByRectBeforeSmooth(extTileRect, m_spline->GetPath(), guidePointsForSmooth, - clippedPaths); + m2::ClipPathByRectBeforeSmooth(extTileRect, m_spline->GetPath(), guidePointsForSmooth, clippedPaths); + if (clippedPaths.empty()) return; m2::SmoothPaths(guidePointsForSmooth, 4 /* newPointsPerSegmentCount */, m2::kCentripetalAlpha, clippedPaths); - m_clippedSplines.clear(); + ASSERT(m_clippedSplines.empty(), ()); std::function inserter = base::MakeBackInsertFunctor(m_clippedSplines); for (auto & path : clippedPaths) m2::ClipPathByRect(m_tileRect, std::move(path), inserter); @@ -848,11 +848,21 @@ void ApplyLineFeatureGeometry::ProcessLineRule(drule::BaseRule const * rule) if (m_clippedSplines.empty()) return; - double const depth = PriorityToDepth(pLineRule->priority(), drule::line, 0); + for (auto const & r : lineRules) + ProcessRule(r); - if (pLineRule->has_pathsym()) +#ifdef LINES_GENERATION_CALC_FILTERED_POINTS + LinesStat::Get().InsertLine(m_f.GetID(), m_tileKey.m_zoomLevel, m_readCount, static_cast(m_spline->GetSize())); +#endif +} + +void ApplyLineFeatureGeometry::ProcessRule(LineRuleProto const * lineRule) +{ + double const depth = PriorityToDepth(lineRule->priority(), drule::line, 0); + + if (lineRule->has_pathsym()) { - PathSymProto const & symRule = pLineRule->pathsym(); + PathSymProto const & symRule = lineRule->pathsym(); PathSymbolViewParams params; params.m_tileCenter = m_tileRect.Center(); params.m_depth = depth; @@ -870,7 +880,7 @@ void ApplyLineFeatureGeometry::ProcessLineRule(drule::BaseRule const * rule) { LineViewParams params; params.m_tileCenter = m_tileRect.Center(); - ExtractLineParams(pLineRule, params); + ExtractLineParams(lineRule, params); params.m_depth = depth; params.m_rank = m_f.GetRank(); params.m_baseGtoPScale = m_currentScaleGtoP; @@ -881,43 +891,15 @@ void ApplyLineFeatureGeometry::ProcessLineRule(drule::BaseRule const * rule) } } -void ApplyLineFeatureGeometry::Finish() -{ -#ifdef LINES_GENERATION_CALC_FILTERED_POINTS - LinesStat::Get().InsertLine(m_f.GetID(), m_tileKey.m_zoomLevel, m_readCount, static_cast(m_spline->GetSize())); -#endif -} - ApplyLineFeatureAdditional::ApplyLineFeatureAdditional(TileKey const & tileKey, TInsertShapeFn const & insertShape, FeatureType & f, double currentScaleGtoP, CaptionDescription const & captions, std::vector const & clippedSplines) : TBase(tileKey, insertShape, f, captions) , m_clippedSplines(clippedSplines) - , m_currentScaleGtoP(static_cast(currentScaleGtoP)) - , m_captionDepth(0.0f) - , m_shieldDepth(0.0f) - , m_captionRule(nullptr) - , m_shieldRule(nullptr) -{} - -void ApplyLineFeatureAdditional::ProcessLineRule(drule::BaseRule const * rule) + , m_currentScaleGtoP(currentScaleGtoP) { ASSERT(!m_clippedSplines.empty(), ()); - - ShieldRuleProto const * pShieldRule = rule->GetShield(); - if (pShieldRule != nullptr) - { - m_shieldRule = pShieldRule; - m_shieldDepth = PriorityToDepth(pShieldRule->priority(), drule::shield, 0); - } - - CaptionDefProto const * pCaptionRule = rule->GetCaption(0); - if (pCaptionRule != nullptr && pCaptionRule->height() > 2 && !m_captions.GetMainText().empty()) - { - m_captionRule = pCaptionRule; - m_captionDepth = PriorityToDepth(rule->GetCaptionsPriority(), drule::pathtext, 0); - } } void ApplyLineFeatureAdditional::GetRoadShieldsViewParams(ref_ptr texMng, @@ -928,7 +910,7 @@ void ApplyLineFeatureAdditional::GetRoadShieldsViewParams(ref_ptr texMng, - ftypes::RoadShieldsSetT const & roadShields, - GeneratedRoadShields & generatedRoadShields) +void ApplyLineFeatureAdditional::ProcessAdditionalLineRules(PathTextRuleProto const * pathtextRule, + ShieldRuleProto const * shieldRule, + ref_ptr texMng, + ftypes::RoadShieldsSetT const & roadShields, + GeneratedRoadShields & generatedRoadShields) { - if (m_clippedSplines.empty()) - return; + ASSERT(pathtextRule || shieldRule, ()); auto const vs = static_cast(df::VisualParams::Instance().GetVisualScale()); std::vector shieldPositions; - if (m_shieldRule != nullptr && !roadShields.empty()) - shieldPositions.reserve(m_clippedSplines.size() * 3); - - if (m_captionRule != nullptr) + ASSERT((shieldRule && !roadShields.empty()) || !(shieldRule && !roadShields.empty()), + (roadShields.empty(), shieldRule == nullptr)); + if (shieldRule) { + m_shieldRule = shieldRule; + m_shieldDepth = PriorityToDepth(shieldRule->priority(), drule::shield, 0); + shieldPositions.reserve(m_clippedSplines.size() * 3); + } + + if (pathtextRule) + { + ASSERT(!m_captions.GetMainText().empty(), ()); + m_captionRule = &pathtextRule->primary(); + ASSERT_GREATER_OR_EQUAL(m_captionRule->height(), kMinVisibleFontSize / df::kMaxVisualScale, ()); + m_captionDepth = PriorityToDepth(pathtextRule->priority(), drule::pathtext, 0); + dp::FontDecl fontDecl; CaptionDefProtoToFontDecl(m_captionRule, fontDecl); PathTextViewParams params; - params.m_tileCenter = m_tileRect.Center(); - params.m_featureId = m_f.GetID(); + FillCommonParams(params); + params.m_depthLayer = DepthLayer::OverlayLayer; + params.m_depthTestEnabled = false; params.m_depth = m_captionDepth; - params.m_rank = m_f.GetRank(); params.m_mainText = m_captions.GetMainText(); params.m_auxText = m_captions.GetAuxText(); params.m_textFont = fontDecl; @@ -1095,14 +1068,14 @@ void ApplyLineFeatureAdditional::Finish(ref_ptr texMng, // Position shields inbetween captions. // If there is only one center position then the shield and the caption will compete for it. - if (m_shieldRule != nullptr && !roadShields.empty()) + if (m_shieldRule) CalculateRoadShieldPositions(shape->GetOffsets(), spline, shieldPositions); m_insertShape(std::move(shape)); textIndex++; } } - else if (m_shieldRule != nullptr && !roadShields.empty()) + else if (m_shieldRule) { // Position shields without captions. for (auto const & spline : m_clippedSplines) @@ -1120,7 +1093,6 @@ void ApplyLineFeatureAdditional::Finish(ref_ptr texMng, return; // Set default shield's icon min distance. - ASSERT(m_shieldRule != nullptr, ()); int minDistance = m_shieldRule->min_distance(); ASSERT_GREATER_OR_EQUAL(minDistance, 0, ()); if (minDistance <= 0) diff --git a/drape_frontend/apply_feature_functors.hpp b/drape_frontend/apply_feature_functors.hpp index 2819217f1e..d9f4793933 100644 --- a/drape_frontend/apply_feature_functors.hpp +++ b/drape_frontend/apply_feature_functors.hpp @@ -8,6 +8,7 @@ #include "drape/pointers.hpp" #include "indexer/drawing_rule_def.hpp" +#include "indexer/drawing_rules.hpp" #include "indexer/road_shields_parser.hpp" #include "geometry/point2d.hpp" @@ -17,8 +18,6 @@ #include class CaptionDefProto; -class ShieldRuleProto; -class SymbolRuleProto; namespace dp { @@ -43,9 +42,7 @@ public: virtual ~BaseApplyFeature() = default; protected: - void ExtractCaptionParams(CaptionDefProto const * primaryProto, - CaptionDefProto const * secondaryProto, - TextViewParams & params) const; + void FillCommonParams(CommonOverlayViewParams & p) const; double PriorityToDepth(int priority, drule::rule_type_t ruleType, double areaDepth) const; TInsertShapeFn m_insertShape; @@ -62,24 +59,19 @@ class ApplyPointFeature : public BaseApplyFeature public: ApplyPointFeature(TileKey const & tileKey, TInsertShapeFn const & insertShape, FeatureType & f, - CaptionDescription const & captions, float posZ); + CaptionDescription const & captions); - void operator()(m2::PointD const & point, bool hasArea); - void ProcessPointRule(drule::BaseRule const * rule, bool isHouseNumber); - void Finish(ref_ptr texMng); + void ProcessPointRules(SymbolRuleProto const * symbolRule, CaptionRuleProto const * captionRule, + CaptionRuleProto const * houseNumberRule, m2::PointD const & centerPoint, + ref_ptr texMng); protected: - float const m_posZ; - + void ExtractCaptionParams(CaptionDefProto const * primaryProto, + CaptionDefProto const * secondaryProto, + TextViewParams & params) const; + float m_posZ = 0.0f; private: - TextViewParams m_textParams; - TextViewParams m_hnParams; - m2::PointD m_centerPoint; - SymbolRuleProto const * m_symbolRule = nullptr; - float m_symbolDepth; - bool m_hasArea = false; - bool m_createdByEditor = false; - bool m_obsoleteInEditor = false; + virtual bool HasArea() const { return false; } }; class ApplyAreaFeature : public ApplyPointFeature @@ -92,10 +84,9 @@ public: bool skipAreaGeometry, float minPosZ, float posZ, CaptionDescription const & captions); - using TBase::operator (); - void operator()(m2::PointD const & p1, m2::PointD const & p2, m2::PointD const & p3); - void ProcessAreaRule(drule::BaseRule const * rule, double areaDepth, bool isHatching); + bool HasGeometry() const { return !m_triangles.empty(); } + void ProcessAreaRules(AreaRuleProto const * areaRule, AreaRuleProto const * hatchingRule); struct Edge { @@ -127,6 +118,9 @@ public: }; private: + bool HasArea() const override { return true; } + + void ProcessRule(AreaRuleProto const * areaRule, double areaDepth, bool isHatching); void ProcessBuildingPolygon(m2::PointD const & p1, m2::PointD const & p2, m2::PointD const & p3); void CalculateBuildingOutline(bool calculateNormals, BuildingOutline & outline); int GetIndex(m2::PointD const & pt); @@ -150,27 +144,26 @@ class ApplyLineFeatureGeometry : public BaseApplyFeature public: ApplyLineFeatureGeometry(TileKey const & tileKey, TInsertShapeFn const & insertShape, - FeatureType & f, double currentScaleGtoP, size_t pointsCount, bool smooth); + FeatureType & f, double currentScaleGtoP); void operator() (m2::PointD const & point); - bool HasGeometry() const; - void ProcessLineRule(drule::BaseRule const * rule); - void Finish(); + bool HasGeometry() const { return m_spline->IsValid(); } + void ProcessLineRules(Stylist::LineRulesT const & lineRules); std::vector const & GetClippedSplines() const { return m_clippedSplines; } private: + void ProcessRule(LineRuleProto const * lineRule); + m2::SharedSpline m_spline; std::vector m_clippedSplines; - float m_currentScaleGtoP; - double m_minSegmentSqrLength; + double const m_currentScaleGtoP; + double const m_minSegmentSqrLength; m2::PointD m_lastAddedPoint; - bool m_simplify; - bool m_smooth; - size_t m_initialPointsCount; + bool const m_simplify; #ifdef LINES_GENERATION_CALC_FILTERED_POINTS - int m_readCount; + int m_readCount = 0; #endif }; @@ -184,9 +177,9 @@ public: FeatureType & f, double currentScaleGtoP, CaptionDescription const & captions, std::vector const & clippedSplines); - void ProcessLineRule(drule::BaseRule const * rule); - void Finish(ref_ptr texMng, ftypes::RoadShieldsSetT const & roadShields, - GeneratedRoadShields & generatedRoadShields); + void ProcessAdditionalLineRules(PathTextRuleProto const * pathtextRule, ShieldRuleProto const * shieldRule, + ref_ptr texMng, ftypes::RoadShieldsSetT const & roadShields, + GeneratedRoadShields & generatedRoadShields); private: void GetRoadShieldsViewParams(ref_ptr texMng, @@ -202,10 +195,10 @@ private: std::vector & shields); std::vector m_clippedSplines; - float m_currentScaleGtoP; - float m_captionDepth, m_shieldDepth; - CaptionDefProto const * m_captionRule; - ShieldRuleProto const * m_shieldRule; + double const m_currentScaleGtoP; + float m_captionDepth = 0.0f, m_shieldDepth = 0.0f; + CaptionDefProto const * m_captionRule = nullptr; + ShieldRuleProto const * m_shieldRule = nullptr; }; extern dp::Color ToDrapeColor(uint32_t src); diff --git a/drape_frontend/line_shape.cpp b/drape_frontend/line_shape.cpp index eca756196b..c76c236082 100644 --- a/drape_frontend/line_shape.cpp +++ b/drape_frontend/line_shape.cpp @@ -294,7 +294,6 @@ public: struct BuilderParams : BaseBuilderParams { dp::TextureManager::StippleRegion m_stipple; - float m_glbHalfWidth; float m_baseGtoP; }; @@ -538,8 +537,7 @@ void LineShape::Prepare(ref_ptr textures) const DashedLineBuilder::BuilderParams p; commonParamsBuilder(p); p.m_stipple = maskRegion; - p.m_baseGtoP = m_params.m_baseGtoPScale; - p.m_glbHalfWidth = p.m_pxHalfWidth / m_params.m_baseGtoPScale; + p.m_baseGtoP = static_cast(m_params.m_baseGtoPScale); auto builder = std::make_unique(p, m_spline->GetPath().size()); Construct(*builder); diff --git a/drape_frontend/rule_drawer.cpp b/drape_frontend/rule_drawer.cpp index 2cfb0426d6..b09c8f2c27 100644 --- a/drape_frontend/rule_drawer.cpp +++ b/drape_frontend/rule_drawer.cpp @@ -13,7 +13,6 @@ #include "indexer/feature_visibility.hpp" #include "indexer/ftypes_matcher.hpp" #include "indexer/map_style_reader.hpp" -#include "indexer/road_shields_parser.hpp" #include "indexer/scales.hpp" #include "geometry/clipping.hpp" @@ -168,7 +167,6 @@ RuleDrawer::RuleDrawer(TCheckCancelledCallback const & checkCancelled, , m_context(engineContext) , m_customFeaturesContext(engineContext->GetCustomFeaturesContext().lock()) , m_deviceLang(deviceLang) - , m_wasCancelled(false) { ASSERT(m_checkCancelled != nullptr, ()); @@ -286,8 +284,7 @@ void RuleDrawer::ProcessAreaAndPointStyle(FeatureType & f, Stylist const & s, TI areaMinHeight = static_cast((rectMercator.SizeX() + rectMercator.SizeY()) * 0.5); } - bool applyPointStyle = s.m_symbolRule != nullptr || s.m_captionRule != nullptr || - s.m_houseNumberRule != nullptr; + bool applyPointStyle = s.m_symbolRule || s.m_captionRule || s.m_houseNumberRule; if (applyPointStyle) { if (!is3dBuilding) @@ -298,63 +295,35 @@ void RuleDrawer::ProcessAreaAndPointStyle(FeatureType & f, Stylist const & s, TI applyPointStyle = m_globalRect.IsPointInside(featureCenter); } - double areaDepth = drule::kBaseDepthBgBySize - 1; ApplyAreaFeature apply(m_context->GetTileKey(), insertShape, f, m_currentScaleGtoP, isBuilding, m_context->Is3dBuildingsEnabled() && isBuildingOutline /* skipAreaGeometry */, areaMinHeight /* minPosZ */, areaHeight /* posZ */, s.GetCaptionDescription()); - if (s.m_areaRule != nullptr || s.m_hatchingRule != nullptr) - { + if (s.m_areaRule || s.m_hatchingRule) f.ForEachTriangle(apply, m_zoomLevel); - /// @todo areaDepth is not needed for FG and BG-top areas. - areaDepth = drule::CalcAreaBySizeDepth(f); - } - if (applyPointStyle) - apply(featureCenter, true /* hasArea */); // ApplyPointFeature::operator()() - if (CheckCancelled()) - return; - - if (s.m_hatchingRule != nullptr) - apply.ProcessAreaRule(s.m_hatchingRule, areaDepth, true); - if (s.m_areaRule != nullptr) - apply.ProcessAreaRule(s.m_areaRule, areaDepth, false); + if (apply.HasGeometry()) + apply.ProcessAreaRules(s.m_areaRule, s.m_hatchingRule); /// @todo Can we put this check in the beginning of this function? if (applyPointStyle && !IsDiscardCustomFeature(f.GetID())) { - // Process point style. - if (s.m_symbolRule != nullptr) - apply.ProcessPointRule(s.m_symbolRule, false); - if (s.m_captionRule != nullptr) - apply.ProcessPointRule(s.m_captionRule, false); - if (s.m_houseNumberRule != nullptr) - apply.ProcessPointRule(s.m_houseNumberRule, true); - apply.Finish(m_context->GetTextureManager()); // ApplyPointFeature::Finish() + apply.ProcessPointRules(s.m_symbolRule, s.m_captionRule, s.m_houseNumberRule, + featureCenter, m_context->GetTextureManager()); } } void RuleDrawer::ProcessLineStyle(FeatureType & f, Stylist const & s, TInsertShapeFn const & insertShape) { - bool const smooth = ftypes::IsIsolineChecker::Instance()(f); - - ApplyLineFeatureGeometry applyGeom(m_context->GetTileKey(), insertShape, f, m_currentScaleGtoP, - f.GetPointsCount(), smooth); + ApplyLineFeatureGeometry applyGeom(m_context->GetTileKey(), insertShape, f, m_currentScaleGtoP); f.ForEachPoint(applyGeom, m_zoomLevel); - if (CheckCancelled()) - return; - if (applyGeom.HasGeometry()) - { - for (auto const & r : s.m_lineRules) - applyGeom.ProcessLineRule(r); - } - applyGeom.Finish(); + applyGeom.ProcessLineRules(s.m_lineRules); std::vector clippedSplines; - bool needAdditional = s.m_pathtextRule != nullptr || s.m_shieldRule != nullptr; + bool needAdditional = s.m_pathtextRule || s.m_shieldRule; if (needAdditional) { auto const metalineSpline = m_context->GetMetalineManager()->GetMetaline(f.GetID()); @@ -380,12 +349,9 @@ void RuleDrawer::ProcessLineStyle(FeatureType & f, Stylist const & s, TInsertSha { ApplyLineFeatureAdditional applyAdditional(m_context->GetTileKey(), insertShape, f, m_currentScaleGtoP, s.GetCaptionDescription(), clippedSplines); - if (s.m_pathtextRule != nullptr) - applyAdditional.ProcessLineRule(s.m_pathtextRule); - if (s.m_shieldRule != nullptr) - applyAdditional.ProcessLineRule(s.m_shieldRule); - applyAdditional.Finish(m_context->GetTextureManager(), ftypes::GetRoadShields(f), - m_generatedRoadShields); + applyAdditional.ProcessAdditionalLineRules(s.m_pathtextRule, s.m_shieldRule, + m_context->GetTextureManager(), s.m_roadShields, + m_generatedRoadShields); } if (m_context->IsTrafficEnabled() && m_zoomLevel >= kRoadClass0ZoomLevel) @@ -431,19 +397,9 @@ void RuleDrawer::ProcessPointStyle(FeatureType & f, Stylist const & s, TInsertSh if (IsDiscardCustomFeature(f.GetID())) return; - ApplyPointFeature apply(m_context->GetTileKey(), insertShape, f, s.GetCaptionDescription(), 0.0f /* posZ */); - apply(f.GetCenter(), false /* hasArea */); - - if (CheckCancelled()) - return; - - if (s.m_symbolRule != nullptr) - apply.ProcessPointRule(s.m_symbolRule, false); - if (s.m_captionRule != nullptr) - apply.ProcessPointRule(s.m_captionRule, false); - if (s.m_houseNumberRule != nullptr) - apply.ProcessPointRule(s.m_houseNumberRule, true); - apply.Finish(m_context->GetTextureManager()); + ApplyPointFeature apply(m_context->GetTileKey(), insertShape, f, s.GetCaptionDescription()); + apply.ProcessPointRules(s.m_symbolRule, s.m_captionRule, s.m_houseNumberRule, + f.GetCenter(), m_context->GetTextureManager()); } void RuleDrawer::operator()(FeatureType & f) @@ -457,27 +413,25 @@ void RuleDrawer::operator()(FeatureType & f) !ftypes::IsBuildingChecker::Instance()(types))) return; + if (ftypes::IsCoastlineChecker::Instance()(types) && !CheckCoastlines(f)) + return; + Stylist const s(f, m_zoomLevel, m_deviceLang); // No drawing rules. - if (s.m_symbolRule == nullptr && s.m_captionRule == nullptr && s.m_houseNumberRule == nullptr && - s.m_lineRules.empty() && s.m_areaRule == nullptr && s.m_hatchingRule == nullptr) - return; - - if (s.m_isCoastline && !CheckCoastlines(f)) + if (!s.m_symbolRule && !s.m_captionRule && !s.m_houseNumberRule && + s.m_lineRules.empty() && !s.m_areaRule && !s.m_hatchingRule) return; #ifdef DEBUG // Validate mixing of feature styles. bool const hasLine = !s.m_lineRules.empty(); - bool const hasLineAdd = s.m_shieldRule != nullptr || s.m_pathtextRule != nullptr; - bool const hasPoint = s.m_symbolRule != nullptr || s.m_captionRule != nullptr || s.m_houseNumberRule != nullptr; - bool const hasArea = s.m_areaRule != nullptr || s.m_hatchingRule != nullptr; + bool const hasLineAdd = s.m_shieldRule || s.m_pathtextRule; + bool const hasPoint = s.m_symbolRule || s.m_captionRule || s.m_houseNumberRule; + bool const hasArea = s.m_areaRule || s.m_hatchingRule; ASSERT(!((hasLine || hasLineAdd) && (hasPoint || hasArea)), ("Line drules mixed with point/area ones", f.DebugString(m_zoomLevel, true))); - - // TODO : validate in the styles generator. ASSERT(!hasLineAdd || hasLine, ("Pathtext/shield without a line drule", f.DebugString(m_zoomLevel, true))); #endif @@ -510,7 +464,7 @@ void RuleDrawer::operator()(FeatureType & f) } else { - ASSERT(s.m_symbolRule != nullptr || s.m_captionRule != nullptr || s.m_houseNumberRule != nullptr, ()); + ASSERT(s.m_symbolRule || s.m_captionRule || s.m_houseNumberRule, ()); ASSERT(geomType == feature::GeomType::Point, ()); ProcessPointStyle(f, s, insertShape); } diff --git a/drape_frontend/rule_drawer.hpp b/drape_frontend/rule_drawer.hpp index 100e5b06d1..4abc34f14c 100644 --- a/drape_frontend/rule_drawer.hpp +++ b/drape_frontend/rule_drawer.hpp @@ -72,15 +72,16 @@ private: std::unordered_set m_usedMetalines; m2::RectD m_globalRect; - uint8_t m_zoomLevel; double m_currentScaleGtoP; double m_trafficScalePtoG; TrafficSegmentsGeometry m_trafficGeometry; std::array m_mapShapes; - bool m_wasCancelled; GeneratedRoadShields m_generatedRoadShields; + + uint8_t m_zoomLevel = 0; + bool m_wasCancelled = false; }; } // namespace df diff --git a/drape_frontend/shape_view_params.hpp b/drape_frontend/shape_view_params.hpp index bbe2a917d6..7298258887 100644 --- a/drape_frontend/shape_view_params.hpp +++ b/drape_frontend/shape_view_params.hpp @@ -74,7 +74,7 @@ struct AreaViewParams : CommonViewParams float m_posZ = 0.0f; bool m_is3D = false; bool m_hatching = false; - float m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0f; }; struct LineViewParams : CommonViewParams @@ -85,7 +85,7 @@ struct LineViewParams : CommonViewParams dp::LineCap m_cap; dp::LineJoin m_join; dp::PenPatternT m_pattern; - float m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0f; int m_zoomLevel = -1; }; @@ -105,16 +105,15 @@ struct PathTextViewParams : CommonOverlayViewParams dp::FontDecl m_textFont; std::string m_mainText; std::string m_auxText; - float m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0f; }; struct PathSymbolViewParams : CommonViewParams { - FeatureID m_featureID; std::string m_symbolName; float m_offset = 0.0f; float m_step = 0.0f; - float m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0f; }; struct ColoredSymbolViewParams : CommonOverlayViewParams diff --git a/drape_frontend/stylist.cpp b/drape_frontend/stylist.cpp index 1d335d5589..8899f4613f 100644 --- a/drape_frontend/stylist.cpp +++ b/drape_frontend/stylist.cpp @@ -1,6 +1,7 @@ #include "drape_frontend/stylist.hpp" #include "indexer/classificator.hpp" +#include "indexer/drules_include.hpp" #include "indexer/feature.hpp" #include "indexer/feature_visibility.hpp" #include "indexer/scales.hpp" @@ -97,54 +98,50 @@ void Stylist::ProcessKey(FeatureType & f, drule::Key const & key) switch (key.m_type) { case drule::symbol: - ASSERT(dRule->GetSymbol() != nullptr && m_symbolRule == nullptr && + ASSERT(dRule->GetSymbol() && !m_symbolRule && (geomType == GeomType::Point || geomType == GeomType::Area), - (m_symbolRule == nullptr, geomType, f.DebugString(0, true))); - m_symbolRule = dRule; + (m_symbolRule == nullptr, geomType, f.DebugString(19, true))); + m_symbolRule = dRule->GetSymbol(); + break; + case drule::caption: + ASSERT(dRule->GetCaption() && dRule->GetCaption()->has_primary() && !m_captionRule && + (geomType == GeomType::Point || geomType == GeomType::Area), + (m_captionRule == nullptr, f.DebugString(19, true))); + m_captionRule = dRule->GetCaption(); break; case drule::pathtext: - case drule::caption: - ASSERT(dRule->GetCaption(0) != nullptr, (f.DebugString(0, true))); - if (key.m_type == drule::caption) - { - ASSERT(m_captionRule == nullptr && (geomType == GeomType::Point || geomType == GeomType::Area), - (geomType, f.DebugString(0, true))); - m_captionRule = dRule; - } - else - { - ASSERT(m_pathtextRule == nullptr && geomType == GeomType::Line, - (geomType, f.DebugString(0, true))); - m_pathtextRule = dRule; - } + ASSERT(dRule->GetPathtext() && dRule->GetPathtext()->has_primary() && !m_pathtextRule && + geomType == GeomType::Line, + (m_pathtextRule == nullptr, geomType, f.DebugString(19, true))); + m_pathtextRule = dRule->GetPathtext(); break; case drule::shield: - ASSERT(dRule->GetShield() != nullptr && m_shieldRule == nullptr && geomType == GeomType::Line, - (m_shieldRule == nullptr, geomType, f.DebugString(0, true))); - m_shieldRule = dRule; + ASSERT(dRule->GetShield() && !m_shieldRule && geomType == GeomType::Line, + (m_shieldRule == nullptr, geomType, f.DebugString(19, true))); + m_shieldRule = dRule->GetShield(); break; case drule::line: - ASSERT(dRule->GetLine() != nullptr && geomType == GeomType::Line, (geomType, f.DebugString(0, true))); - m_lineRules.push_back(dRule); + ASSERT(dRule->GetLine() && geomType == GeomType::Line, (geomType, f.DebugString(19, true))); + m_lineRules.push_back(dRule->GetLine()); break; case drule::area: - ASSERT(dRule->GetArea() != nullptr && geomType == GeomType::Area, (geomType, f.DebugString(0, true))); + ASSERT(dRule->GetArea() && geomType == GeomType::Area, (geomType, f.DebugString(19, true))); if (key.m_hatching) { - ASSERT(m_hatchingRule == nullptr, (f.DebugString(0, true))); - m_hatchingRule = dRule; + ASSERT(!m_hatchingRule, (f.DebugString(19, true))); + m_hatchingRule = dRule->GetArea(); } else { - ASSERT(m_areaRule == nullptr, (f.DebugString(0, true))); - m_areaRule = dRule; + ASSERT(!m_areaRule, (f.DebugString(19, true))); + m_areaRule = dRule->GetArea(); } break; // TODO : check if circle/waymarker support exists still (not used in styles ATM). case drule::circle: case drule::waymarker: default: - ASSERT(false, (key.m_type, f.DebugString(0, true))); + ASSERT(false, (key.m_type, f.DebugString(19, true))); return; } } @@ -208,10 +205,10 @@ Stylist::Stylist(FeatureType & f, uint8_t zoomLevel, int8_t deviceLang) for (auto const & key : keys) ProcessKey(f, key); - if (m_captionRule != nullptr || m_pathtextRule != nullptr) + if (m_captionRule || m_pathtextRule) { - bool const auxExists = (m_captionRule != nullptr && m_captionRule->GetCaption(1) != nullptr) || - (m_pathtextRule != nullptr && m_pathtextRule->GetCaption(1) != nullptr); + bool const auxExists = (m_captionRule && m_captionRule->has_secondary()) || + (m_pathtextRule && m_pathtextRule->has_secondary()); m_captionDescriptor.Init(f, deviceLang, zoomLevel, geomType, auxExists); if (m_captionDescriptor.IsHouseNumberExists()) @@ -240,7 +237,7 @@ Stylist::Stylist(FeatureType & f, uint8_t zoomLevel, int8_t deviceLang) if (mainOverlayType == addressType) { // Optimization: just duplicate the drule if the main type is building-address. - ASSERT(m_captionRule != nullptr, ()); + ASSERT(m_captionRule, ()); m_houseNumberRule = m_captionRule; } else @@ -252,7 +249,8 @@ Stylist::Stylist(FeatureType & f, uint8_t zoomLevel, int8_t deviceLang) // A caption drule exists for this zoom level. ASSERT(addressKeys.size() == 1 && addressKeys[0].m_type == drule::caption, ("building-address should contain a caption drule only")); - m_houseNumberRule = drule::rules().Find(addressKeys[0]); + ASSERT(m_houseNumberRule == nullptr, ()); + m_houseNumberRule = drule::rules().Find(addressKeys[0])->GetCaption(); } } } @@ -265,7 +263,12 @@ Stylist::Stylist(FeatureType & f, uint8_t zoomLevel, int8_t deviceLang) } } - m_isCoastline = types.Has(cl.GetCoastType()); + if (m_shieldRule) + { + m_roadShields = ftypes::GetRoadShields(f); + if (m_roadShields.empty()) + m_shieldRule = nullptr; + } } } // namespace df diff --git a/drape_frontend/stylist.hpp b/drape_frontend/stylist.hpp index ef8929dad7..21a7e971a5 100644 --- a/drape_frontend/stylist.hpp +++ b/drape_frontend/stylist.hpp @@ -3,6 +3,7 @@ #include "indexer/ftypes_matcher.hpp" #include "indexer/drawing_rule_def.hpp" #include "indexer/drawing_rules.hpp" +#include "indexer/road_shields_parser.hpp" #include "base/buffer_vector.hpp" @@ -47,16 +48,18 @@ private: class Stylist { public: - bool m_isCoastline = false; + SymbolRuleProto const * m_symbolRule = nullptr; + CaptionRuleProto const * m_captionRule = nullptr; + CaptionRuleProto const * m_houseNumberRule = nullptr; + PathTextRuleProto const * m_pathtextRule = nullptr; + ShieldRuleProto const * m_shieldRule = nullptr; + AreaRuleProto const * m_areaRule = nullptr; + AreaRuleProto const * m_hatchingRule = nullptr; - drule::BaseRule const * m_symbolRule = nullptr; - drule::BaseRule const * m_captionRule = nullptr; - drule::BaseRule const * m_houseNumberRule = nullptr; - drule::BaseRule const * m_pathtextRule = nullptr; - drule::BaseRule const * m_shieldRule = nullptr; - drule::BaseRule const * m_areaRule = nullptr; - drule::BaseRule const * m_hatchingRule = nullptr; - buffer_vector m_lineRules; + using LineRulesT = buffer_vector; + LineRulesT m_lineRules; + + ftypes::RoadShieldsSetT m_roadShields; Stylist(FeatureType & f, uint8_t zoomLevel, int8_t deviceLang); diff --git a/drape_frontend/visual_params.cpp b/drape_frontend/visual_params.cpp index 97933d4377..33d2b9d3a6 100644 --- a/drape_frontend/visual_params.cpp +++ b/drape_frontend/visual_params.cpp @@ -27,20 +27,6 @@ static bool g_isInited = false; #define ASSERT_INITED #endif -double const VisualParams::kMdpiScale = 1.0; -double const VisualParams::kHdpiScale = 1.5; -double const VisualParams::kXhdpiScale = 2.0; -double const VisualParams::k6plusScale = 2.4; -double const VisualParams::kXxhdpiScale = 3.0; -double const VisualParams::kXxxhdpiScale = 3.5; - -VisualParams::VisualParams() - : m_tileSize(0) - , m_visualScale(0.0) - , m_poiExtendScale(0.1) // found empirically - , m_fontScale(1.0) -{} - VisualParams & VisualParams::Instance() { static VisualParams vizParams; diff --git a/drape_frontend/visual_params.hpp b/drape_frontend/visual_params.hpp index 2f0089b418..be9ca123bc 100644 --- a/drape_frontend/visual_params.hpp +++ b/drape_frontend/visual_params.hpp @@ -19,12 +19,16 @@ double constexpr kMaxVisualScale = 4.0; class VisualParams { public: - static double const kMdpiScale; - static double const kHdpiScale; - static double const kXhdpiScale; - static double const k6plusScale; - static double const kXxhdpiScale; - static double const kXxxhdpiScale; + static double constexpr kMdpiScale = 1.0; + static double constexpr kHdpiScale = 1.5; + static double constexpr kXhdpiScale = 2.0; + static double constexpr k6plusScale = 2.4; + static double constexpr kXxhdpiScale = 3.0; + static double constexpr kXxxhdpiScale = 3.5; + + static_assert(kMdpiScale < kHdpiScale && kHdpiScale < kXhdpiScale && + kXhdpiScale < k6plusScale && k6plusScale < kXxhdpiScale && + kXxhdpiScale < kXxxhdpiScale && kXxxhdpiScale <= kMaxVisualScale); static void Init(double vs, uint32_t tileSize); static VisualParams & Instance(); @@ -64,13 +68,14 @@ public: void SetVisualScale(double visualScale); private: - VisualParams(); + VisualParams() = default; - uint32_t m_tileSize; - double m_visualScale; - double m_poiExtendScale; GlyphVisualParams m_glyphVisualParams; - std::atomic m_fontScale; + + uint32_t m_tileSize = 0; + double m_visualScale = 0.0f; + double m_poiExtendScale = 0.1f; // Found empirically. + std::atomic m_fontScale = 1.0f; DISALLOW_COPY_AND_MOVE(VisualParams); }; diff --git a/geometry/rect2d.hpp b/geometry/rect2d.hpp index 00bbc94e0f..1978a2ee53 100644 --- a/geometry/rect2d.hpp +++ b/geometry/rect2d.hpp @@ -42,8 +42,7 @@ public: Rect(T minX, T minY, T maxX, T maxY) : m_minX(minX), m_minY(minY), m_maxX(maxX), m_maxY(maxY) { - ASSERT(minX <= maxX, ()); - ASSERT(minY <= maxY, ()); + ASSERT(minX <= maxX && minY <= maxY, (minX, maxX, minY, maxY)); } Rect(Point const & p1, Point const & p2) diff --git a/indexer/drawing_rule_def.hpp b/indexer/drawing_rule_def.hpp index 2803dace54..b7a0e9277e 100644 --- a/indexer/drawing_rule_def.hpp +++ b/indexer/drawing_rule_def.hpp @@ -80,9 +80,6 @@ static double constexpr kBaseDepthFg = 0, /// drawing type of rule - can be one of ... enum rule_type_t { line, area, symbol, caption, circle, pathtext, waymarker, shield, count_of_rules }; - /// @todo not used. - enum text_type_t { text_type_name = 0, text_type_housename, text_type_housenumber }; - typedef buffer_vector KeysT; void MakeUnique(KeysT & keys); double CalcAreaBySizeDepth(FeatureType & f); diff --git a/indexer/drawing_rules.cpp b/indexer/drawing_rules.cpp index 664a3c1cf6..4f8810760c 100644 --- a/indexer/drawing_rules.cpp +++ b/indexer/drawing_rules.cpp @@ -23,16 +23,6 @@ using namespace std; namespace { uint32_t const DEFAULT_BG_COLOR = 0xEEEEDD; - - drule::text_type_t GetTextType(string const & text) - { - if (text == "addr:housename") - return drule::text_type_housename; - else if (text == "addr:housenumber") - return drule::text_type_housenumber; - - return drule::text_type_name; - } } // namespace LineRuleProto const * BaseRule::GetLine() const @@ -50,17 +40,12 @@ SymbolRuleProto const * BaseRule::GetSymbol() const return 0; } -CaptionDefProto const * BaseRule::GetCaption(int) const +CaptionRuleProto const * BaseRule::GetCaption() const { return 0; } -text_type_t BaseRule::GetCaptionTextType(int) const -{ - return text_type_name; -} - -int BaseRule::GetCaptionsPriority() const +PathTextRuleProto const * BaseRule::GetPathtext() const { return 0; } @@ -159,7 +144,11 @@ namespace { LineRuleProto m_line; public: - explicit Line(LineRuleProto const & r) : m_line(r) {} + explicit Line(LineRuleProto const & r) + : m_line(r) + { + CHECK(r.has_pathsym() || r.width() > 0, ("Zero width line w/o path symbol)")); //pastk + } virtual LineRuleProto const * GetLine() const { return &m_line; } }; @@ -182,57 +171,27 @@ namespace virtual SymbolRuleProto const * GetSymbol() const { return &m_symbol; } }; - template class CaptionT : public BaseRule + class Caption : public BaseRule { - T m_caption; - - text_type_t m_textTypePrimary; - text_type_t m_textTypeSecondary; - + CaptionRuleProto m_caption; public: - explicit CaptionT(T const & r) - : m_caption(r), m_textTypePrimary(text_type_name), m_textTypeSecondary(text_type_name) + explicit Caption(CaptionRuleProto const & r) + : m_caption(r) { - if (!m_caption.primary().text().empty()) - m_textTypePrimary = GetTextType(m_caption.primary().text()); - - if (m_caption.has_secondary() && !m_caption.secondary().text().empty()) - m_textTypeSecondary = GetTextType(m_caption.secondary().text()); + CHECK(r.has_primary(),()); //pastk } - virtual CaptionDefProto const * GetCaption(int i) const - { - if (i == 0) - return &m_caption.primary(); - else - { - ASSERT_EQUAL ( i, 1, () ); - if (m_caption.has_secondary()) - return &m_caption.secondary(); - else - return 0; - } - } - - virtual int GetCaptionsPriority() const - { - return m_caption.priority(); - } - - virtual text_type_t GetCaptionTextType(int i) const - { - if (i == 0) - return m_textTypePrimary; - else - { - ASSERT_EQUAL ( i, 1, () ); - return m_textTypeSecondary; - } - } + virtual CaptionRuleProto const * GetCaption() const { return &m_caption; } }; - typedef CaptionT Caption; - typedef CaptionT PathText; + class PathText : public BaseRule + { + PathTextRuleProto m_pathtext; + public: + explicit PathText(PathTextRuleProto const & r) : m_pathtext(r) {} + + virtual PathTextRuleProto const * GetPathtext() const { return &m_pathtext; } + }; class Shield : public BaseRule { diff --git a/indexer/drawing_rules.hpp b/indexer/drawing_rules.hpp index 371af8363b..e41c1df4ba 100644 --- a/indexer/drawing_rules.hpp +++ b/indexer/drawing_rules.hpp @@ -19,7 +19,8 @@ class LineRuleProto; class AreaRuleProto; class SymbolRuleProto; -class CaptionDefProto; +class CaptionRuleProto; +class PathTextRuleProto; class ShieldRuleProto; class ContainerProto; class FeatureType; @@ -36,9 +37,8 @@ namespace drule virtual LineRuleProto const * GetLine() const; virtual AreaRuleProto const * GetArea() const; virtual SymbolRuleProto const * GetSymbol() const; - virtual CaptionDefProto const * GetCaption(int) const; - virtual text_type_t GetCaptionTextType(int) const; - virtual int GetCaptionsPriority() const; + virtual CaptionRuleProto const * GetCaption() const; + virtual PathTextRuleProto const * GetPathtext() const; virtual ShieldRuleProto const * GetShield() const; // Test feature by runtime feature style selector diff --git a/map/transit/transit_reader.cpp b/map/transit/transit_reader.cpp index 1208dc568d..055a86d379 100644 --- a/map/transit/transit_reader.cpp +++ b/map/transit/transit_reader.cpp @@ -168,11 +168,7 @@ void ReadTransitTask::Do() //TODO: there should be a simpler way to just get a symbol name. df::Stylist stylist(ft, 19, 0); if (stylist.m_symbolRule != nullptr) - { - auto const * symRule = stylist.m_symbolRule->GetSymbol(); - ASSERT(symRule != nullptr, ()); - featureInfo.m_gateSymbolName = symRule->name(); - } + featureInfo.m_gateSymbolName = stylist.m_symbolRule->name(); } featureInfo.m_point = feature::GetCenter(ft); }, features); -- 2.45.3 From b965a89992bdda79d93e9c1590b5b7b299d433d2 Mon Sep 17 00:00:00 2001 From: Alexander Borsuk Date: Sun, 17 Sep 2023 01:22:26 +0200 Subject: [PATCH 2/5] Enable CTest memcheck with valgrind on Linux Signed-off-by: Alexander Borsuk --- CMakeLists.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 01464196b8..a506404a0a 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -209,9 +209,9 @@ if (PLATFORM_LINUX OR PLATFORM_ANDROID) endif() if (NOT SKIP_TESTS) - set(SKIP_TESTS FALSE) enable_testing() - set(CTEST_CONFIGURATION_TYPE ${CMAKE_BUILD_TYPE}) + # Enables ctest -T memcheck with valgrind + include(CTest) endif() if (NOT PYTHON_VERSION) -- 2.45.3 From da3b51a78fde391d3b96aabca70b9b5fbe113b73 Mon Sep 17 00:00:00 2001 From: Sergiy Kozyr Date: Mon, 9 Oct 2023 22:15:29 +0300 Subject: [PATCH 3/5] Added Ukrainian translation to built-in FAQ page (#6252) --- data/faq.html | 365 ++++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 356 insertions(+), 9 deletions(-) diff --git a/data/faq.html b/data/faq.html index 61f8cb348e..b09609d941 100644 --- a/data/faq.html +++ b/data/faq.html @@ -90,9 +90,9 @@ } // TODO: Update this list with a new translation. - var translations = ['en', 'ru', 'de', 'es', 'fr', 'pt', 'pl', 'tr']; + var translations = ['en', 'ru', 'de', 'es', 'fr', 'pt', 'pl', 'tr', 'uk']; // Show Russian for browsers with this language codes. - var canReadRussian = ['ab', 'be', 'kk', 'ky', 'tg', 'uk', 'uz']; + var canReadRussian = ['ab', 'be', 'kk', 'ky', 'tg', 'uz']; // TODO: Properly handle be-EN, he-RU or similar cases. function showLanguage() { @@ -124,6 +124,7 @@

Perguntas frequentes

FAQ

SSS

+

Часті питання

@@ -369,6 +377,10 @@ organicmaps.app/tr/ web sitemizi ziyaret edin. +

+ Для отримання додаткової інформації, будь ласка, відвідайте наш веб-сайт + organicmaps.app +

@@ -400,6 +412,9 @@

Haritada bazı yerler eksik veya yanlış adlara sahip

+

+ Деякі місця відсутні на карті або мають неправильні назви +

@@ -826,6 +841,52 @@
+ +
+

+ Джерелом наших картографічних даних є + OpenStreetMap (OSM). + Це картографічний проект, подібний до Вікіпедії, але для мап, де + будь-хто може створювати та редагувати мапи. +

+ +

+ Якщо ви побачили невірну інформацію або виявили, що деякі об'єкти + відсутні на карті, ви можете + залишити повідомлення для + редакторів OSM або + реєструватися і редагувати + мапу. +

+ +

+ Чим більше людей долучається, тим детальніші стають карти. Ми віримо, + що найдетальніша карта всього світу, створена відкритою + спільнотою - це лише питання часу. +

+ +

Примітки:

+ +
    +
  • +

    + Ви також можете додавати нові місця, редагувати існуючі POI та інформацію про будівлі + (адреси, години роботи, назви) безпосередньо в Organic Maps. Після того, як + ви ввійдете за допомогою облікового запису OSM, ваші правки будуть автоматично + завантажені на OSM. Будь ласка, будьте обережні при редагуванні, оскільки ваші правки + будуть видимі всім іншим користувачам. +

    +
    • +
  • +

    + База даних OpenStreetMap змінюється щохвилини. Ми прагнемо + оновлювати карти в додатку 1-4 рази на місяць. Якщо ви змінили + щось в OSM, ваші правки з'являться в наступних оновленнях мап. +

    +
    • +
+
+
@@ -839,6 +900,7 @@

O aplicativo não pode determinar minha localização

Aplikacja nie może określić mojej lokalizacji

Uygulama konumumu haritada bulamıyor

+

Додаток не може визначити моє місцезнаходження на карті

@@ -1412,6 +1474,65 @@ + +
+

+ Будь ласка, переконайтеся, що на вашому пристрої ввімкнено GPS та активовані + налаштування місцезнаходження. +

+

Android

+

+ На вашому пристрої відкрийте Налаштування → Місцезнаходження. Краще + увімкнути режим високої точності. +

+

+ Якщо у вас виникають труднощі з визначенням вашого місцезнаходження за допомогою GPS, увімкніть + (вимкніть, якщо увімкнено) "Сервіси Google Play" в налаштуваннях програми. +

+

+ Примітка: ви можете побачити його, лише якщо на вашому пристрої встановлені + та увімкнені служби Google Play Android. Ці служби використовуються для + більш точного визначення місцезнаходження. Якщо у вас виникли проблеми з + з точністю визначення місцезнаходження після вимкнення цієї опції, + увімкніть її назад. +

+

iOS

+ Якщо ви користувач iPhone або iPad, перевірте налаштування iOS → Конфіденційність → + Служби визначення місцезнаходження. Обмін геолокаційними даними слід увімкнути для + Organic Maps. +

Примітки:

+
    +
  • +

    + Щоб уникнути небажаних даних у роумінгу, ви можете вимкнути всі мобільні + дані, активувати режим польоту або вимкнути мобільні дані для Organic + Maps у налаштуваннях вашого пристрою. Пристрої на Android та iOS можуть використовувати GPS + в режимі польоту. +

    • +
  • +

    + Деякі мобільні пристрої не мають вбудованих GPS-приймачів, наприклад + iPod Touch, iPad з підтримкою Wi-Fi, Amazon Kindle Fire/Kindle Fire HD + 7 та деякі планшети Android. На цих пристроях наш додаток буде + показуватиме ваше приблизне місцезнаходження за умови підключення до + до Інтернету. +

    +
    • +
  • +

    + Насамкінець, будь ласка, пам'ятайте, що визначення місцезнаходження + за допомогою GPS (з використанням WiFi але без мобільної мережи) може + зайняти деякий час. Чим довше GPS не використовувався, тим більше часу це займе. Швидкість + визначення місцезнаходження залежить від пристрою, а не від програми. + На роботу GPS приймача також впливає погода - він працює + найкраще на відкритому повітрі, коли небо чисте. Проблеми можуть виникнути при + при спробі визначити своє місцезнаходження в приміщенні, на вузькій вулиці + або за кермом автомобіля. +

    +
    • +
+
+
@@ -1424,6 +1545,7 @@

Não consigo baixar (atualizar) mapas

Nie mogę pobrać (zaktualizować) map

Haritaları indiremiyorum/güncelleyemiyorum

+

Я не можу завантажити (оновити) карти

@@ -1609,6 +1731,26 @@ eski haritaları silip yeniden indirmek yardımcı olabilir.

+ +
+

+ Завантаження може завершитися невдало через тимчасову помилку мережі + або певні проблеми провайдера/маршрутизатора. Будь ласка, повторіть + спробу пізніше або використовуйте іншу точку доступу Wi-Fi. Крім того, + переконайтеся, що у вас є достатньо вільного місця для завантаження карт. +

+ +

+ На Android, будь ласка, переконайтеся, що ви надали доступ до мережі для + Organic Maps і системний менеджер завантажень (провайдер завантажень). +

+ +

+ Крім того, якщо у вас дуже застарілі карти, і програма не може їх оновити, + вам може допомогти видалення всіх карт і повторне їх завантаження. +

+
+

Search cannot find a place on the map

@@ -1620,6 +1762,7 @@

A pesquisa não pode encontrar um lugar no mapa

Wyszukiwanie nie może znaleźć miejsca na mapie

Arama, haritada bir yer bulamıyor

+

Пошук не може знайти місце на карті

Remember that to search for a place in a specific area, you will need to @@ -1848,6 +1991,31 @@ bize bildirin.

+
+

+ Пам'ятайте, що для пошуку місця в певному районі вам потрібно + завантажити відповідну мапу і збільшити масштаб карти до цієї області. + Або ж ваше місцезнаходження має бути поблизу цієї області, перш ніж + ви почнете пошук. Наприклад, якщо ви хочете знайти місце у В'єтнамі, + а ваше місцезнаходження десь в іншому місці, вам слід спочатку + завантажити і відкрити мапу В'єтнаму. +

+ +

+ Крім того, місце може бути ще не додано на мапу в базі + OpenStreetMap.org, нашому + джерелі картографічних даних. Якщо ви хочете допомогти і покращити мапу, перевірте + цей путівник. +

+ +

+ Якщо місце відображається на мапі в Organic Maps, але наш пошук + не може його знайти, + повідомте нам координати + місця та приклад вашого пошукового запиту. +

+
+
@@ -1862,10 +2030,9 @@

Pourquoi l'application s'arrête ou crash

O que fazer se a aplicação parar de funcionar (bloquear)

O que posso fazer se o aplicativo parar (travar)

-

- Co mogę zrobić, jeśli aplikacja nie odpowiada/uległa awarii -

-

Uygulama durdurulursa/çökerse ne yapabilirim

+

Co mogę zrobić, jeśli aplikacja nie odpowiada/uległa awarii

+

Uygulama durdurulursa/çökerse ne yapabilirim

+

Що робити, якщо додаток несподівано закрився або вийшов з ладу

@@ -2122,6 +2289,32 @@
  • cihaz modeli ve İşletim Sistemi sürümü (Android veya iOS).
    • + +
    +

    + Ймовірно, це наша помилка, і ми будемо раді виправити її в наступному + оновленні. +

    + +

    + Для Android пристроїв, якщо ви зберігаєте карти на SD-карті, найімовірнішою + причиною є несправна SD-карта. Ви можете відформатувати SD-карту, а ще краще + замінити її на нову. Якщо ви перемістили саму програму на SD-карту, + будь ласка, поверніть його назад у внутрішню пам'ять пристрою. +

    + +

    + Якщо проблема не зникне, будь ласка + зв'яжіться з нами і повідомте: +

    + +
      +
    • короткий опис проблеми
      • +
    • версію Organic Maps
      • +
    • модель пристрою та версія ОС (Android або iOS).
      • +
    +
    +
    @@ -2134,6 +2327,7 @@

    Como criar uma rota

    Jak utworzyć trasę

    Nasıl rota oluşturulur

    +

    Як побудувати маршрут

    @@ -2554,7 +2748,50 @@ ayarlarını açın → Sürüş seçenekleri → gerekli seçenekleri açın.

    + +
    +

    + Після того, як ваше місцезнаходження визначено на карті, оберіть пункт призначення. + Ви можете скористатися одним із наступних способів: +

    +
      +
    • торкніться кнопки пошуку
      • +
    • торкніться кнопки закладок
      • +
    • торкніться будь-якого місця на карті (торкніться і утримуйте для порожніх областей)
      • +

      + +

      + Після того, як ви обрали пункт призначення, натисніть кнопку "Прокласти маршрут" на + нижній панелі. Маршрут буде створено, і ви побачите відстань + та приблизний час у дорозі. Ви можете змінити тип маршруту, натиснувши + іконку Автомобіль, Пішохід, Метро, Велосипед або Лінійка у верхній частині + екрана. Щоб почати рух за маршрутом, натисніть кнопку "Почати". Під час навігацію + натисніть на значок у правому нижньому куті екрана і натисніть кнопку "Зупинити" + щоб завершити маршрут. +

      + +

      + На iOS пристроях щоб спланувати маршрут, ви можете натиснути іконку "Маршрут" на + нижній панелі меню. Такоже ви можете вибрати іншу початкову точку (кнопка + "Маршрут від") для попереднього перегляду маршруту, але навігація доступна лише + з вашого поточного місцезнаходження. +

      + +

      + До маршруту можна додати до 100 проміжних пунктів. Щоб додати + проміжний пункт: створіть маршрут між пунктом відправлення та пунктом + призначення → потім торкніться точки на мапі і натисніть "Додати зупинку". +

      + +

      + Ви можете змінити налаштування автомобільного маршруту і вибрати типи доріг + які ви хотіли б уникати (платні дороги, ґрунтові дороги, автомагістралі, + пороми). Відкрийте налаштування програми → Налаштування обʼїзду → увімкніть + потрібні опції. +

      +
    +
    @@ -2567,6 +2804,7 @@

    Não consigo ouvir instruções de voz

    Nie słyszę instrukcji głosowych

    Sesli yönlendirmeyi duyamıyorum

    +

    Я не чую голосових підказок

    @@ -2963,6 +3201,67 @@ Slovakça, Tayca, Yunanca.

    + +
    +

    + Зауваження: голосові підказки доступні для автомобільних та велосипедних + маршрутів. Наразі ви можете чути голосові підказки лише під час руху + і коли екран увімкнено. +

    + +

    Якщо ви не чуєте голосові підказки:

    + +
      +
    • +

      + Переконайтеся, що на вашому пристрої не вимкнено звук. Ви можете + змінити рівень гучності за допомогою кнопок гучності вашого пристрою. + Щоб це зробити вам, можливо, доведеться вимкнути опцію "Змінювати кнопками" + (якщо її увімкнено) у Налаштуваннях пристрою → Звуки. +

      +
      • + +
    • +

      + Коли ви вмикаєте Bluetooth, він не повинен відображати голосові + інструкції. Однак ми не тестували голосові підказки з автомобільним + аудіо, це може спричинити проблеми. У такому разі ви можете вимкнути + Bluetooth. +

      +
      • + +
    • +

      + Якщо ви користувач Android і ця опція вимкнена (або деякі з + з підтримуваних мов недоступні), перевірте системні налаштування TTS. +

      +
      • +
    + +

    + На Android голосові інструкції доступні 27 мовами: Англійська, + арабська, китайська (традиційна та спрощена), чеська, данська, голландська, чеська, датська, + фінська, французька, німецька, грецька, хінді, угорська, індонезійська, італійська, німецька, італійська, угорська, хінді, + японська, корейська, перська, польська, португальська, румунська, російська, французька, чеська, японська, + іспанська, тайська, турецька, українська, французька, в'єтнамська. +

    + +

    + Google TTS підтримує всі перелічені мови, окрім арабської та перської + (фарсі). Для цих мов вам може знадобитися встановити сторонній TTS + (наприклад, eSpeak TTS, Vocalizer TTS або SVOX Classic TTS) і + мовний пакет з магазину додатків (Google Play Store, Galaxy Store тощо). +

    + +

    + На iOS голосові інструкції доступні 26 мовами: Англійська, + арабська, китайська (традиційна та спрощена), чеська, данська, голландська, чеська, датська, + фінська, французька, німецька, грецька, хінді, угорська, індонезійська, італійська, німецька, хорватська, чеська, шведська, японська, + японська, корейська, польська, португальська, румунська, російська, словацька, словацька, + іспанська, шведська, тайська, українсьска, турецька. +

    +
    +
    @@ -2976,6 +3275,7 @@

    Como compartilhar (exportar) favoritos

    Jak udostępniać (eksportować) zakładki

    Yer imleri nasıl paylaşılır/dışa aktarılır

    +

    Як поділитися (експортувати) мітками

    @@ -3060,6 +3360,14 @@ önündeki üç noktaya dokunun ve açılır pencereden Dosyayı Dışa Aktar'a dokunun.
    +
    + Виберіть мітку на карті, торкніться її, а потім натисніть кнопку "Поділитися" + в нижній панели. Щоб поділитися відразу всіма мітками зі списку, торкніться + символу зірки на головному екрані, потім торкніться значка з трьома крапками після + назви списку з мітками і натисніть "Експортувати файл" у спливаючому вікні. +
    +
    +

    How to import bookmarks

    @@ -3071,6 +3379,7 @@

    Como importar favoritos

    Jak importować zakładki

    Yer imleri nasıl içe aktarılır

    +

    Як імпортувати мітки

    @@ -3464,6 +3773,43 @@
    + +
    + Ви можете імпортувати мітки, надіслані з Organic Maps або зі сторонніх додатків, + які підтримують експорт у формати KML, KMZ або GPX: + +
      +
    1. +

      + На Android ви можете імпортувати мітки з папок і програм (включно з Maps.Me). + Натисніть кнопку із зірочкою, щоб відкрити мітки, натисніть + "Імпортувати мітки" і виберіть диск або папку з файлами KML, KMZ, GPX. +

      +
      2. + +
    2. +

      + Відкрийте файл KML, KMZ, GPX із мітками, який ви отримали електронною поштою, + в месенджері або через хмарним сховище, наприклад, iCloud або Google Drive. +

      +
      3. +
    3. +

      + Натисніть один раз або натисніть і утримуйте файл KML, KMZ, GPX із мітками та + виберіть "Відкрити за допомогою Organic Maps" (Android) або "Імпортувати за допомогою Organic + Maps" (iOS) у спливаючому вікні. +

      +
      4. + +
    4. +

      + Її буде відкрито у програмі Organic Maps, і ви побачите повідомлення "Закладки завантажено + успішно!". Ви можете побачити їх на мапі або в меню Мітки на головному екрані. +

      +
      5. +
    +
    +
    Go to Top Наверх @@ -3474,5 +3820,6 @@ Ir para o início Idź na górę Başa Git + До початку -- 2.45.3 From de0ad7f7559634c6af32501103710e85e1059ec1 Mon Sep 17 00:00:00 2001 From: Konstantin Pastbin Date: Tue, 10 Oct 2023 12:36:38 +0300 Subject: [PATCH 4/5] review fixes --- drape_frontend/apply_feature_functors.cpp | 24 +++++++++++------------ drape_frontend/rule_drawer.cpp | 2 +- drape_frontend/shape_view_params.hpp | 8 ++++---- drape_frontend/stylist.cpp | 8 ++++---- drape_frontend/visual_params.hpp | 6 +++--- indexer/drawing_rules.cpp | 4 ++-- map/transit/transit_reader.cpp | 2 +- 7 files changed, 27 insertions(+), 27 deletions(-) diff --git a/drape_frontend/apply_feature_functors.cpp b/drape_frontend/apply_feature_functors.cpp index faccc9d2ff..d897a474c7 100644 --- a/drape_frontend/apply_feature_functors.cpp +++ b/drape_frontend/apply_feature_functors.cpp @@ -222,7 +222,7 @@ std::string GetRoadShieldSymbolName(ftypes::RoadShield const & shield, double fo result = shield.m_name.size() <= 2 ? "shield-us-hw-thin" : "shield-us-hw-wide"; else { - ASSERT(false, ()); + ASSERT(false, ("This shield type doesn't support symbols:", shield.m_type)); } if (fontScale > 1.0) @@ -366,8 +366,8 @@ void BaseApplyFeature::FillCommonParams(CommonOverlayViewParams & p) const } void ApplyPointFeature::ExtractCaptionParams(CaptionDefProto const * primaryProto, - CaptionDefProto const * secondaryProto, - TextViewParams & params) const + CaptionDefProto const * secondaryProto, + TextViewParams & params) const { FillCommonParams(params); params.m_depthLayer = DepthLayer::OverlayLayer; @@ -380,7 +380,7 @@ void ApplyPointFeature::ExtractCaptionParams(CaptionDefProto const * primaryProt CaptionDefProtoToFontDecl(primaryProto, decl); titleDecl.m_primaryTextFont = decl; titleDecl.m_anchor = GetAnchor(primaryProto->offset_x(), primaryProto->offset_y()); - // TODO: remove offsets processing as de-facto "text-offset: *" is used to define anchors only. + // TODO(pastk) : remove offsets processing as de-facto "text-offset: *" is used to define anchors only. titleDecl.m_primaryOffset = GetOffset(primaryProto->offset_x(), primaryProto->offset_y()); titleDecl.m_primaryOptional = primaryProto->is_optional(); @@ -424,7 +424,7 @@ double BaseApplyFeature::PriorityToDepth(int priority, drule::rule_type_t ruleTy { // Note we don't adjust priorities of "point-styles" according to layer=*, // because their priorities are used for displacement logic only. - /// @todo we might want to hide e.g. a trash bin under man_made=bridge or a bench on underground railway station? + /// @todo(pastk) we might want to hide e.g. a trash bin under man_made=bridge or a bench on underground railway station? // Check overlays priorities range. ASSERT(-drule::kOverlaysMaxPriority <= depth && depth < drule::kOverlaysMaxPriority, (depth, m_f.GetID())); @@ -564,7 +564,7 @@ void ApplyAreaFeature::operator()(m2::PointD const & p1, m2::PointD const & p2, m2::PointD const v1 = p2 - p1; m2::PointD const v2 = p3 - p1; - //TODO : degenerate triangles filtering should be done in the generator. + //TODO(pastk) : degenerate triangles filtering should be done in the generator. // ASSERT(!v1.IsAlmostZero() && !v2.IsAlmostZero(), ()); if (v1.IsAlmostZero() || v2.IsAlmostZero()) return; @@ -572,7 +572,7 @@ void ApplyAreaFeature::operator()(m2::PointD const & p1, m2::PointD const & p2, double const crossProduct = m2::CrossProduct(v1.Normalize(), v2.Normalize()); double constexpr kEps = 1e-7; // ASSERT_GREATER_OR_EQUAL(fabs(crossProduct), kEps, (fabs(crossProduct), p1, p2, p3, m_f.DebugString(19, true))); - // TODO : e.g. a landuse-meadow has a following triangle with two identical points: + // TODO(pastk) : e.g. a landuse-meadow has a following triangle with two identical points: // m2::Point(8.5829683287662987823, 53.929641499591184584) // m2::Point(8.5830675705005887721, 53.930025055483156393) // m2::Point(8.5830675705005887721, 53.930025055483156393) @@ -597,7 +597,7 @@ void ApplyAreaFeature::ProcessBuildingPolygon(m2::PointD const & p1, m2::PointD { // For building we must filter degenerate polygons because now we have to reconstruct // building outline by bunch of polygons. - // TODO : filter degenerates in the generator.(see a TODO above). + // TODO(pastk) : filter degenerates in the generator.(see a TODO above). m2::PointD const v1 = p2 - p1; m2::PointD const v2 = p3 - p1; if (v1.IsAlmostZero() || v2.IsAlmostZero()) @@ -636,7 +636,7 @@ int ApplyAreaFeature::GetIndex(m2::PointD const & pt) { for (size_t i = 0; i < m_points.size(); i++) { - // TODO : should be possible to use exact match. + // TODO(pastk) : should be possible to use exact match. if (pt.EqualDxDy(m_points[i], mercator::kPointEqualityEps)) return static_cast(i); } @@ -672,7 +672,7 @@ m2::PointD ApplyAreaFeature::CalculateNormal(m2::PointD const & p1, m2::PointD c void ApplyAreaFeature::BuildEdges(int vertexIndex1, int vertexIndex2, int vertexIndex3, bool twoSide) { // Check if triangle is degenerate. - // TODO : filter degenerates in the generator. + // TODO(pastk) : filter degenerates in the generator. if (vertexIndex1 == vertexIndex2 || vertexIndex2 == vertexIndex3 || vertexIndex1 == vertexIndex3) return; @@ -779,7 +779,7 @@ ApplyLineFeatureGeometry::ApplyLineFeatureGeometry(TileKey const & tileKey, TIns FeatureType & f, double currentScaleGtoP) : TBase(tileKey, insertShape, f, CaptionDescription()) , m_currentScaleGtoP(currentScaleGtoP) - // TODO: calculate just once in the RuleDrawer. + // TODO(pastk) : calculate just once in the RuleDrawer. , m_minSegmentSqrLength(base::Pow2(4.0 * df::VisualParams::Instance().GetVisualScale() / currentScaleGtoP)) , m_simplify(tileKey.m_zoomLevel >= 10 && tileKey.m_zoomLevel <= 12) { @@ -821,7 +821,7 @@ void ApplyLineFeatureGeometry::ProcessLineRules(Stylist::LineRulesT const & line if (!ftypes::IsIsolineChecker::Instance()(m_f)) { // A line crossing the tile several times will be split in several parts. - // TODO : use feature's pre-calculated limitRect when possible. + // TODO(pastk) : use feature's pre-calculated limitRect when possible. m_clippedSplines = m2::ClipSplineByRect(m_tileRect, m_spline); } else diff --git a/drape_frontend/rule_drawer.cpp b/drape_frontend/rule_drawer.cpp index b09c8f2c27..2bcb6c48a7 100644 --- a/drape_frontend/rule_drawer.cpp +++ b/drape_frontend/rule_drawer.cpp @@ -446,7 +446,7 @@ void RuleDrawer::operator()(FeatureType & f) size_t const index = shape->GetType(); ASSERT_LESS(index, m_mapShapes.size(), ()); - // TODO: MinZoom was used for optimization in RenderGroup::UpdateCanBeDeletedStatus(), but is long time broken. + // TODO(pastk) : MinZoom was used for optimization in RenderGroup::UpdateCanBeDeletedStatus(), but is long time broken. // See https://github.com/organicmaps/organicmaps/pull/5903 for details. shape->SetFeatureMinZoom(0); m_mapShapes[index].push_back(std::move(shape)); diff --git a/drape_frontend/shape_view_params.hpp b/drape_frontend/shape_view_params.hpp index 7298258887..12e8d545ee 100644 --- a/drape_frontend/shape_view_params.hpp +++ b/drape_frontend/shape_view_params.hpp @@ -74,7 +74,7 @@ struct AreaViewParams : CommonViewParams float m_posZ = 0.0f; bool m_is3D = false; bool m_hatching = false; - double m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0; }; struct LineViewParams : CommonViewParams @@ -85,7 +85,7 @@ struct LineViewParams : CommonViewParams dp::LineCap m_cap; dp::LineJoin m_join; dp::PenPatternT m_pattern; - double m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0; int m_zoomLevel = -1; }; @@ -105,7 +105,7 @@ struct PathTextViewParams : CommonOverlayViewParams dp::FontDecl m_textFont; std::string m_mainText; std::string m_auxText; - double m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0; }; struct PathSymbolViewParams : CommonViewParams @@ -113,7 +113,7 @@ struct PathSymbolViewParams : CommonViewParams std::string m_symbolName; float m_offset = 0.0f; float m_step = 0.0f; - double m_baseGtoPScale = 1.0f; + double m_baseGtoPScale = 1.0; }; struct ColoredSymbolViewParams : CommonOverlayViewParams diff --git a/drape_frontend/stylist.cpp b/drape_frontend/stylist.cpp index 8899f4613f..b3c6d89a9e 100644 --- a/drape_frontend/stylist.cpp +++ b/drape_frontend/stylist.cpp @@ -45,7 +45,7 @@ void CaptionDescription::Init(FeatureType & f, int8_t deviceLang, int zoomLevel, feature::GeomType geomType, bool auxCaptionExists) { feature::NameParamsOut out; - // TODO: remove forced secondary text for all lines and set it via styles for major roads and rivers only. + // TODO(pastk) : remove forced secondary text for all lines and set it via styles for major roads and rivers only. // ATM even minor paths/streams/etc use secondary which makes their pathtexts take much more space. if (zoomLevel > scales::GetUpperWorldScale() && (auxCaptionExists || geomType == feature::GeomType::Line)) { @@ -75,13 +75,13 @@ void CaptionDescription::Init(FeatureType & f, int8_t deviceLang, int zoomLevel, if (m_mainText.size() > kMaxTextSize) m_mainText = m_mainText.substr(0, kMaxTextSize) + "..."; - // TODO : its better to determine housenumbers minZoom once upon drules load and cache it, + // TODO(pastk) : its better to determine housenumbers minZoom once upon drules load and cache it, // but it'd mean a lot of housenumbers-specific logic in otherwise generic RulesHolder.. uint8_t constexpr kHousenumbersMinZoom = 16; if (geomType != feature::GeomType::Line && zoomLevel >= kHousenumbersMinZoom && (auxCaptionExists || m_mainText.empty())) { - // TODO: its not obvious that a housenumber display is dependent on a secondary caption drule existance in styles. + // TODO(pastk) : its not obvious that a housenumber display is dependent on a secondary caption drule existance in styles. m_houseNumberText = f.GetHouseNumber(); if (!m_houseNumberText.empty() && !m_mainText.empty() && m_houseNumberText.find(m_mainText) != std::string::npos) m_mainText.clear(); @@ -137,7 +137,7 @@ void Stylist::ProcessKey(FeatureType & f, drule::Key const & key) m_areaRule = dRule->GetArea(); } break; - // TODO : check if circle/waymarker support exists still (not used in styles ATM). + // TODO(pastk) : check if circle/waymarker support exists still (not used in styles ATM). case drule::circle: case drule::waymarker: default: diff --git a/drape_frontend/visual_params.hpp b/drape_frontend/visual_params.hpp index be9ca123bc..85e67a0346 100644 --- a/drape_frontend/visual_params.hpp +++ b/drape_frontend/visual_params.hpp @@ -73,9 +73,9 @@ private: GlyphVisualParams m_glyphVisualParams; uint32_t m_tileSize = 0; - double m_visualScale = 0.0f; - double m_poiExtendScale = 0.1f; // Found empirically. - std::atomic m_fontScale = 1.0f; + double m_visualScale = 0.0; + double m_poiExtendScale = 0.1; // Found empirically. + std::atomic m_fontScale = 1.0; DISALLOW_COPY_AND_MOVE(VisualParams); }; diff --git a/indexer/drawing_rules.cpp b/indexer/drawing_rules.cpp index 4f8810760c..65db28ca68 100644 --- a/indexer/drawing_rules.cpp +++ b/indexer/drawing_rules.cpp @@ -147,7 +147,7 @@ namespace explicit Line(LineRuleProto const & r) : m_line(r) { - CHECK(r.has_pathsym() || r.width() > 0, ("Zero width line w/o path symbol)")); //pastk + ASSERT(r.has_pathsym() || r.width() > 0, ("Zero width line w/o path symbol)")); } virtual LineRuleProto const * GetLine() const { return &m_line; } @@ -178,7 +178,7 @@ namespace explicit Caption(CaptionRuleProto const & r) : m_caption(r) { - CHECK(r.has_primary(),()); //pastk + ASSERT(r.has_primary(),()); } virtual CaptionRuleProto const * GetCaption() const { return &m_caption; } diff --git a/map/transit/transit_reader.cpp b/map/transit/transit_reader.cpp index 055a86d379..256e06daf6 100644 --- a/map/transit/transit_reader.cpp +++ b/map/transit/transit_reader.cpp @@ -165,7 +165,7 @@ void ReadTransitTask::Do() if (featureInfo.m_isGate) { - //TODO: there should be a simpler way to just get a symbol name. + //TODO(pastk): there should be a simpler way to just get a symbol name. df::Stylist stylist(ft, 19, 0); if (stylist.m_symbolRule != nullptr) featureInfo.m_gateSymbolName = stylist.m_symbolRule->name(); -- 2.45.3 From d4e5b2f623162ff23fd468fc27e8ff45b298dc4c Mon Sep 17 00:00:00 2001 From: Konstantin Pastbin Date: Tue, 10 Oct 2023 17:08:39 +0300 Subject: [PATCH 5/5] change non-null ptrs passing to refs --- drape_frontend/apply_feature_functors.cpp | 44 +++++++++++------------ drape_frontend/apply_feature_functors.hpp | 4 +-- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drape_frontend/apply_feature_functors.cpp b/drape_frontend/apply_feature_functors.cpp index d897a474c7..b5dfe5a632 100644 --- a/drape_frontend/apply_feature_functors.cpp +++ b/drape_frontend/apply_feature_functors.cpp @@ -117,15 +117,15 @@ private: }; #endif -void ExtractLineParams(LineRuleProto const * lineRule, LineViewParams & params) +void ExtractLineParams(LineRuleProto const & lineRule, LineViewParams & params) { double const scale = df::VisualParams::Instance().GetVisualScale(); - params.m_color = ToDrapeColor(lineRule->color()); - params.m_width = static_cast(std::max(lineRule->width() * scale, 1.0)); + params.m_color = ToDrapeColor(lineRule.color()); + params.m_width = static_cast(std::max(lineRule.width() * scale, 1.0)); - if (lineRule->has_dashdot()) + if (lineRule.has_dashdot()) { - DashDotProto const & dd = lineRule->dashdot(); + DashDotProto const & dd = lineRule.dashdot(); int const count = dd.dd_size(); params.m_pattern.reserve(count); @@ -133,7 +133,7 @@ void ExtractLineParams(LineRuleProto const * lineRule, LineViewParams & params) params.m_pattern.push_back(dp::PatternFloat2Pixel(dd.dd(i) * scale)); } - switch (lineRule->cap()) + switch (lineRule.cap()) { case ::ROUNDCAP : params.m_cap = dp::RoundCap; break; @@ -145,7 +145,7 @@ void ExtractLineParams(LineRuleProto const * lineRule, LineViewParams & params) CHECK(false, ()); } - switch (lineRule->join()) + switch (lineRule.join()) { case ::NOJOIN : params.m_join = dp::MiterJoin; break; @@ -731,7 +731,7 @@ void ApplyAreaFeature::ProcessAreaRules(AreaRuleProto const * areaRule, AreaRule if (hatchingRule) { ASSERT_GREATER_OR_EQUAL(hatchingRule->priority(), drule::kBasePriorityFg, ()); - ProcessRule(hatchingRule, areaDepth, true); + ProcessRule(*hatchingRule, areaDepth, true); } if (areaRule) @@ -739,16 +739,16 @@ void ApplyAreaFeature::ProcessAreaRules(AreaRuleProto const * areaRule, AreaRule // Calculate areaDepth for BG-by-size areas only. if (areaRule->priority() < drule::kBasePriorityBgTop) areaDepth = drule::CalcAreaBySizeDepth(m_f); - ProcessRule(areaRule, areaDepth, false); + ProcessRule(*areaRule, areaDepth, false); } } -void ApplyAreaFeature::ProcessRule(AreaRuleProto const * areaRule, double areaDepth, bool isHatching) +void ApplyAreaFeature::ProcessRule(AreaRuleProto const & areaRule, double areaDepth, bool isHatching) { AreaViewParams params; params.m_tileCenter = m_tileRect.Center(); - params.m_depth = PriorityToDepth(areaRule->priority(), drule::area, areaDepth); - params.m_color = ToDrapeColor(areaRule->color()); + params.m_depth = PriorityToDepth(areaRule.priority(), drule::area, areaDepth); + params.m_color = ToDrapeColor(areaRule.color()); params.m_rank = m_f.GetRank(); params.m_minPosZ = m_minPosZ; params.m_posZ = m_posZ; @@ -759,11 +759,11 @@ void ApplyAreaFeature::ProcessRule(AreaRuleProto const * areaRule, double areaDe if (m_isBuilding && !isHatching) { /// @todo Make borders work for non-building areas too. - outline.m_generateOutline = areaRule->has_border() && - areaRule->color() != areaRule->border().color() && - areaRule->border().width() > 0.0; + outline.m_generateOutline = areaRule.has_border() && + areaRule.color() != areaRule.border().color() && + areaRule.border().width() > 0.0; if (outline.m_generateOutline) - params.m_outlineColor = ToDrapeColor(areaRule->border().color()); + params.m_outlineColor = ToDrapeColor(areaRule.border().color()); bool const calculateNormals = m_posZ > 0.0; if (calculateNormals || outline.m_generateOutline) @@ -848,21 +848,21 @@ void ApplyLineFeatureGeometry::ProcessLineRules(Stylist::LineRulesT const & line if (m_clippedSplines.empty()) return; - for (auto const & r : lineRules) - ProcessRule(r); + for (LineRuleProto const * r : lineRules) + ProcessRule(*r); #ifdef LINES_GENERATION_CALC_FILTERED_POINTS LinesStat::Get().InsertLine(m_f.GetID(), m_tileKey.m_zoomLevel, m_readCount, static_cast(m_spline->GetSize())); #endif } -void ApplyLineFeatureGeometry::ProcessRule(LineRuleProto const * lineRule) +void ApplyLineFeatureGeometry::ProcessRule(LineRuleProto const & lineRule) { - double const depth = PriorityToDepth(lineRule->priority(), drule::line, 0); + double const depth = PriorityToDepth(lineRule.priority(), drule::line, 0); - if (lineRule->has_pathsym()) + if (lineRule.has_pathsym()) { - PathSymProto const & symRule = lineRule->pathsym(); + PathSymProto const & symRule = lineRule.pathsym(); PathSymbolViewParams params; params.m_tileCenter = m_tileRect.Center(); params.m_depth = depth; diff --git a/drape_frontend/apply_feature_functors.hpp b/drape_frontend/apply_feature_functors.hpp index d9f4793933..b98c5d5250 100644 --- a/drape_frontend/apply_feature_functors.hpp +++ b/drape_frontend/apply_feature_functors.hpp @@ -120,7 +120,7 @@ public: private: bool HasArea() const override { return true; } - void ProcessRule(AreaRuleProto const * areaRule, double areaDepth, bool isHatching); + void ProcessRule(AreaRuleProto const & areaRule, double areaDepth, bool isHatching); void ProcessBuildingPolygon(m2::PointD const & p1, m2::PointD const & p2, m2::PointD const & p3); void CalculateBuildingOutline(bool calculateNormals, BuildingOutline & outline); int GetIndex(m2::PointD const & pt); @@ -153,7 +153,7 @@ public: std::vector const & GetClippedSplines() const { return m_clippedSplines; } private: - void ProcessRule(LineRuleProto const * lineRule); + void ProcessRule(LineRuleProto const & lineRule); m2::SharedSpline m_spline; std::vector m_clippedSplines; -- 2.45.3