stop rendering historic=citywalls

[matkoniecz]

I am not sure is it a good idea to display historic=citywalls. Maybe it is expecting too much, but city walls may be mapped as building=*.

Current historic=citywalls rendering is not intuitive, especially in places with building=* - see http://www.openstreetmap.org/#map=19/41.90367/12.45770

Citywalls are visible early but it is not clear what is represented

http://www.openstreetmap.org/#map=14/53.3186/13.8433

see also #1333

[imagico]

Actually the more common (19k vs. 6k uses) and more logical tagging is barrier=city_wall. A simple city wall, in particular a onesided, embankment-like one - like this, should not be mapped as a building.

[nebulon42]

Related to #1333.

[dieterdreist]

I used to put both, barrier=city_wall and historic=citywalls, as its both in my case. There is also historic=city_gate http://taginfo.openstreetmap.org/tags/historic=city_gate (which would be nice to render the name btw.) --- barrier=city_gate is negligible. In past discussions the issue of historic citywalls that aren't a barrier now has been raised.

in my opinion we should render both tags.