aspectumapp / osm2geojson

Convert OSM and Overpass XML/JSON to GeoJSON
MIT License
97 stars 15 forks source link

Converting to list/dict comprehensions #18

Closed dericke closed 3 years ago

dericke commented 3 years ago

PR changes the for...append() pattern to list comprehensions where equivalent, does the same for dicts, makes exception handling slightly more specific (does not catch KeyboardInterrupt for example), uses setdefault() to replace a LBYL, and simplifies a couple of conditionals.

rapkin commented 3 years ago

Good job!