From 81c09cc979c646ec890e65603553ad559dedeebc Mon Sep 17 00:00:00 2001 From: Vladimir Byko-Ianko Date: Fri, 9 Feb 2018 15:34:48 +0300 Subject: [PATCH] Review fixes. --- transit/transit_graph_data.cpp | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/transit/transit_graph_data.cpp b/transit/transit_graph_data.cpp index 7ae84c9879..3d71976354 100644 --- a/transit/transit_graph_data.cpp +++ b/transit/transit_graph_data.cpp @@ -186,14 +186,13 @@ void DeserializerFromJson::operator()(FeatureIdentifiers & id, char const * name auto const it = m_osmIdToFeatureIds.find(osmId); if (it != m_osmIdToFeatureIds.cend()) { - CHECK_GREATER_OR_EQUAL(it->second.size(), 1, - ("Osm id:", osmId, "(encoded", osmId.EncodedId(), - ") from transit graph does not correspond to any feature.")); + CHECK(!it->second.empty(), ("Osm id:", osmId, "(encoded", osmId.EncodedId(), + ") from transit graph does not correspond to any feature.")); if (it->second.size() != 1) { - // Note. |osmId| corresponds to several feature ids. It may happen in case of stops; - // if a stop is present as relation. It's a rare case. - LOG(LWARNING, ("Osm id:", osmId, "(encoded", osmId.EncodedId(), "corresponds to", + // Note. |osmId| corresponds to several feature ids. It may happen in case of stops, + // if a stop is present as a relation. It's a rare case. + LOG(LWARNING, ("Osm id:", osmId, "( encoded", osmId.EncodedId(), ") corresponds to", it->second.size(), "feature ids.")); } id.SetFeatureId(it->second[0]);