Blacklist polygon rules should be treated as "all" in whitelist pass

[jwheare]

My understanding of blacklist rules is they are effectively "all with exceptions". The exceptions are handled in the first blacklist pass, so the second pass should treat them as all.

Example that should be a polygon (natural=water) https://www.openstreetmap.org/way/9641206

there is a natural=* tag and its value is not any of: no, coastline, cliff, ridge, arete nor tree_row;
https://wiki.openstreetmap.org/wiki/Overpass_turbo/Polygon_Features

[jwheare]

The failing test appears to be due to highway being both a blacklist and a whitelist. The blacklist seems redundant if there's already a whitelist.

[jwheare]

I see now why there's the extra blacklist for highway. I've pushed a better fix with test for the issue I'm seeing. All tests should now pass.

This involves an extra loop through the polygon features list in the second pass for blacklists.

[rapkin]

Thank you for your work, this is looking good!