<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/openstreetmap/osm2pgsql/pull/1176'>https://github.com/openstreetmap/osm2pgsql/pull/1176</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>Remove unused pending_threaded_processor::ids_done</li>
  <li>ids_queued doesn't need to be a member variable</li>
  <li>Remove unneeded copy of vectors of output clones</li>
  <li>pending_threaded_processor is always used in append mode</li>
  <li>Remove second parameter of pending_way/relation</li>
  <li>Iterate over outputs in threads making queue simpler and shorter</li>
  <li>Refactor do_jobs</li>
  <li>Refactor out common code from process_ways/relations</li>
  <li>Rename member variables: Add m_ prefix</li>
  <li>Move commiting of clones into process_queue()</li>
  <li>Move more common code into process_queue()</li>
  <li>Simplify expire tree merging</li>
  <li>Use idlist_t (vector<osmid_t>) instead of stack for queue</li>
  <li>Disentangle processor from dependency manager</li>
  <li>Reorder parts of the pending_threaded_processor class</li>
  <li>Various cleanups and docs</li>
  <li>User ptr instead of non-const refs for parameters</li>
  <li>Adapt parameter type to use case</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-bf02c3c419d3ddb36412d1341769d7de">src/dependency-manager.cpp</a>
    (19)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-fe39fe48c76cd95051f3c9849b955772">src/dependency-manager.hpp</a>
    (73)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-945b0338af91c31d5230475d7275513d">src/osmdata.cpp</a>
    (345)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-71bd4d24701983b83cd8beedb90c03c8">src/osmdata.hpp</a>
    (2)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-1ec007a8e654cf27e1889bdc6518924c">src/output-flex.cpp</a>
    (13)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-950d59556601894012e34f978f1d521f">src/output-flex.hpp</a>
    (4)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-ef67686615712ffbbed7b2db3266ad35">src/output-gazetteer.hpp</a>
    (4)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-a3e65ae6b86635f909c8ecd8ad1a12c1">src/output-multi.cpp</a>
    (8)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-732ab59463507f38b2debac03c50d4af">src/output-multi.hpp</a>
    (4)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-9ebcac4894bdff4b53d2df484b573935">src/output-null.hpp</a>
    (4)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-2775989ebde2a0178e22e1329ff70ee1">src/output-pgsql.cpp</a>
    (14)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-e2bf981c05025c58fb0ea12788f41fd0">src/output-pgsql.hpp</a>
    (4)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-328a9b7d06dabd86e513eeb757c3b39c">src/output.hpp</a>
    (4)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176/files#diff-534e625287cf03f1406cbacfbc2d30ba">tests/test-middle.cpp</a>
    (15)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/openstreetmap/osm2pgsql/pull/1176.patch'>https://github.com/openstreetmap/osm2pgsql/pull/1176.patch</a></li>
  <li><a href='https://github.com/openstreetmap/osm2pgsql/pull/1176.diff'>https://github.com/openstreetmap/osm2pgsql/pull/1176.diff</a></li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/osm2pgsql/pull/1176">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AA6353XGHQLIOEVZ6GNYWWTRSOIRFANCNFSM4NFW5W5A">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AA6353W7OBXYDEXQSTFRPRLRSOIRFA5CNFSM4NFW5W5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4JIMMEXQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/osm2pgsql/pull/1176",
"url": "https://github.com/openstreetmap/osm2pgsql/pull/1176",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>