Skip to content

Commit

Permalink
Issue #30
Browse files Browse the repository at this point in the history
  • Loading branch information
clarisma committed Jan 4, 2024
1 parent 6e84eab commit 7ff47f2
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion src/filter/WithinFilter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -91,9 +91,13 @@ int WithinPolygonFilter::locateMembers(FeatureStore* store, RelationRef relation

bool WithinPolygonFilter::acceptAreaRelation(FeatureStore* store, RelationRef relation) const
{
RecursionGuard guard(relation);
// We only check ways (i.e. ignore label nodes and sub-areas)

if (relation.id() == 16367304)
{
LOG("DEBUG!!!");
}

FastMemberIterator iter(store, relation);
for (;;)
{
Expand All @@ -106,6 +110,13 @@ bool WithinPolygonFilter::acceptAreaRelation(FeatureStore* store, RelationRef re
if (locateWayNodes(memberWay) < 0) return false;
}
}

// TODO: Currently, areas without ways will be accepted (we should
// return false instead if there were no non-placeholder ways);
// however, this needs to be fixed in the GOL Tool and therefore
// won't be an issue
// See geodesk-py#30, gol-tool#107

return true;
}

Expand Down

0 comments on commit 7ff47f2

Please sign in to comment.