From 446c095a02789e499907a5b53e1298ad3945d6d7 Mon Sep 17 00:00:00 2001 From: vng Date: Tue, 9 Oct 2012 19:47:23 +0300 Subject: [PATCH] Some minor comments. --- map/address_finder.cpp | 3 +++ yg/overlay_element.cpp | 4 ++++ 2 files changed, 7 insertions(+) diff --git a/map/address_finder.cpp b/map/address_finder.cpp index 157dc5a361..5c829db461 100644 --- a/map/address_finder.cpp +++ b/map/address_finder.cpp @@ -384,6 +384,9 @@ namespace if (info.m_name.empty()) { + /// @todo Make logic better. + /// Now we skip linear objects to get only POI's here (don't mix with streets or roads). + /// But there are linear types that may be interesting for POI (rivers). if (m_cont[i].m_types.GetGeoType() != feature::GEOM_LINE) { info.m_name = m_cont[i].m_name; diff --git a/yg/overlay_element.cpp b/yg/overlay_element.cpp index 1e1d5b0ca6..a305457347 100644 --- a/yg/overlay_element.cpp +++ b/yg/overlay_element.cpp @@ -186,6 +186,10 @@ namespace yg m2::PointD const OverlayElement::point(EPosition pos) const { + /// @todo It's better to call roughBoundRect(), or place ASSERT(!m_isDirtyRoughRect, ()) here. + /// In general there is no need to store m_roughBoundRect at all. + /// It's calculating time is fast, because elements already cache vector. + m2::PointD res = m_roughBoundRect.Center(); if (pos & EPosLeft)