kirylkaveryn/organicmaps
1
Fork 0
forked from organicmaps/organicmaps-tmp
Code Pull requests Activity

Minor comments fixes.

Browse source
This commit is contained in:
Alex Zolotarev 2016-03-10 15:25:23 +03:00 • committed by Sergey Yershov
parent 0566981599
commit 65aeeac668
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
search/v2/features_layer_matcher.hpp
View file

@ -335,9 +335,11 @@ private:
inline void GetByIndex(uint32_t id, FeatureType & ft) const
{
/// @todo Add Cache for feature id -> (point, name / house number).
/// TODO(vng): GetFeature below can retur false if feature was deleted by user in the Editor.
/// TODO(vng): GetFeature below can return false if feature was deleted by user in the Editor.
/// This code should be fixed to take that into an account.
/// Until we don't show "Delete" button to our users, this code will work correctly.
/// Correct fix would be injection into ForEachInIntervalAndScale, so deleted features will never
/// be emitted and used in other code.
UNUSED_VALUE(m_context->GetFeature(id, ft));
}

1
search/v2/mwm_context.hpp
View file

@ -68,7 +68,6 @@ private:
template <class TFn> void ForEachIndexImpl(covering::IntervalsT const & intervals,
uint32_t scale, TFn && fn) const
{
// TODO(vng): checkUnique is not used in this code. Do we really need it?
CheckUniqueIndexes checkUnique(m_value.GetHeader().GetFormat() >= version::Format::v5);
for (auto const & i : intervals)
m_index.ForEachInIntervalAndScale([&] (uint32_t index) { fn(index); }, i.first, i.second, scale);