[Tile-serving] [osm2pgsql] Cleanup based on the output of LLVM's static analyzer (#299)

Sarah Hoffmann notifications at github.com
Sat Feb 21 19:24:21 UTC 2015


>          int nodec;
>  
> +        assert(ndc > 0 && "no nodes to proccess");

The current code actually produces empty geometries for 0-node ways. Simply ignoring them might leave zombies when running in append mode. Better to dump empty ways at the very beginning of the function as follows:
~~~
  if (ndc <= 0) {
    delete_unused_full('W', id);
    return 0;
  }
~~~

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/299/files#r25124866
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20150221/d5b837b4/attachment.html>


More information about the Tile-serving mailing list